-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Update documentation CSS #11411
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
Update documentation CSS #11411
Conversation
doc/_static/mpl.css
Outdated
} | ||
|
||
/* function and class description */ | ||
dl.function, dl.method, dl.attribute { | ||
dl.method, dl.classmethod, dl.staticmethod dl.attribute { |
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.
Comma missing, dl.staticmethod, dl.attribute
This looks great in general! However, Not a huge fan of the signature font size change. It’s already highlighted, italics and a different indent. Making larger makes it wrap more for the longer signatures. |
Currently the font-size of the signatures is 13.3px. Would changing it to 14px instead of 15 px not suffice? |
Ok, changed font-size to 14px. Side remark: I'm not quite sure all the font-size settings in the CSS are consistent/intentional. But that's something to look at at another time. |
Update: Table styles (see above). |
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.
Overall this is a nice improvement. Will wait for another approval since its kind of a big change...
There seem to be a conflict, please backport manually |
PR Summary
This updates the CSS of the documentation. In particular this
Following are some examples:
Signature font size
before:
after:
Deprecation
before:
after:
See Also
before:
after:
Warning
before:
after:
Note
before:
after:
Hint
before
after:
Table of contents
before:
after: