Suggestion for 2 slight changes to docs CSS (for better clarity/readability) #10925
Conversation
|
Do you mind attaching here before and after screenshots? 😅 |
|
Here the desired before/after screenshots. However, from my point of view the changes become especially relevant when dynamically scrolling through a long page, for example through the 35 properties and methods of AnimationAction.html. This is the current layout:
Here the same with the suggested changes:
Note: The descriptions (div class="desc") are not affected by the suggested indentations:
|
|
Nice! |
|
Thanks! |
| } | ||
|
|
||
| div { | ||
| /* padding-left: 30px; */ | ||
| padding-left: 30px; |
There was a problem hiding this comment.
I reverted this change though. It messed up the manual pages...
There was a problem hiding this comment.
Oh - sorry!
The manual pages seem to have a partly inconsistent CSS, and when testing the effects of my CSS changes on the manual pages I must have accidentially chosen those that looked good, since they use <div class=”desc“> (such as “Detecting WebGL”) or <p> (such as “Matrix transformations”).
My suggestion:
I would like to unify the “Manual” and “Constant” pages’ CSS by assigning div <class=”desc”> to all paragraphs that shall not be indented. That would be not much work because only a few pages are affected. In this way my CSS changes would have no negative side effects.
Would that be okay?
I reverted this from my work on the animation docs and made this extra PR instead (as @looeee suggested (1020fda#r104101289).
I think, clarity/readability becomes relevant when a page contains many and relative long paragraphs (as for example the AnimationAction.html in #10911 ).