-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[core] Batch small changes #20980
[core] Batch small changes #20980
Conversation
oliviertassinari
commented
May 10, 2020
•
edited
Loading
edited
- [test] Remove legacy data-mui-test ec9858d: the fewer, the better.
- [docs] Update mui-treasury link 68279a4.
- [docs] Fix wrong link 38db8e2.
- [docs] Add lost flags from [docs] Parse markdown on mount #20601 d5958d8: spotted working at the sponsor page.
- [docs] Remove leftover from [docs] Add "reset focus" control to demo tools #20724 ad9e3eb: spotted testing the demos in IE 11.
- [test] Remove all assert.x API a528ab4: we were missing a few. I believe we are 100% done now.
- [core] Avoid over-transpiling 4a42017: Raised in [Slider] Cannot read property 'focus' of null at focusThumb #20896 (comment). I'm curious about the bundle size reduction, it seems to be interesting.
- [core] Use shorthand unit when possible f5de90f.
- [docs] Better visually group tocs items b149905.
- [docs] Fix Crowdin parsing 02b430a.
Details of bundle changes.Comparing: d74bd5d...656f0b8 Details of page changes
|
303263d
to
4a42017
Compare
79ef426
to
b149905
Compare
Could you include before and after? It isn't clear what you're trying to achieve. |
@eps1lon Sure, here are the 3 objectives, by order of importance: Before After
(
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make the TOCs style less impacted by spacing value changes.
Why? This is the point of spacing.
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
@eps1lon Yeah ok, we would need to think something at a systemic level for the documentation. I'm reverting. My concern was that it doesn't look great with too different values (e.g. 2 or 16), but it's true for the rest of the documentation layout. Thanks |
0e0e6f3
to
656f0b8
Compare