-
Notifications
You must be signed in to change notification settings - Fork 9
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
[style] break-word on screen max-width: 680px #8
[style] break-word on screen max-width: 680px #8
Conversation
static/styles/jsdoc.css
Outdated
@@ -661,6 +661,11 @@ html[data-search-mode] .level-hide { | |||
margin: 1em 0; | |||
} | |||
|
|||
#main h4.name { | |||
padding: .75em 10px .75em 10px; |
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.
Instead this, how about we put padding-right: 10px
in global rule. I cannot see any reason we should put 0px for right padding in wide screen.
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.
Actually, I couldn't see any history too.
I fixed your feedback. I added padding-right: 10px in global rule and removed padding in media-query.
Thanks for your comment.
Thanks! This is probably fine, but we should probably configure netlify to build PRs here against the |
should rebase this onto |
OK, you can now see a preview of this change here |
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.
LGTM, thank you!
To fix #4412, I added word-break, padding of #main h4.name.
I think this is easy to read on mobile. It makes line breaks appear wherever the text would otherwise overflow like the examples below:
(The example width is 360px.)
AS-IS
TO-BE