Skip to content
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

Download page #1484

Merged
merged 2 commits into from
Jul 21, 2018
Merged

Download page #1484

merged 2 commits into from
Jul 21, 2018

Conversation

RunDevelopment
Copy link
Member

This PR eliminates awkward word breaks.

It also better aligns checkboxes and text for a cleaner look.
I also disabled hyphens for the owners of plugins because I don't want to see these names chopped into pieces.

I have tested it on Chrome and FireFox.

Before

image

After

image

Copy link
Member

@mAAdhaTTah mAAdhaTTah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes to the CSS needed, but this is definitely a nice improvement. Thank you.

form label input {
margin-right: .7em;
margin-left: -1.7em;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CSS you added appears indented further than the code around it.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, because it is about elements inside form label, so it gets indented.
This practice can also be seen further down below.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, that works.

download.html Outdated
form input {
margin-right: .7em;
}
form label > *:nth-child(2) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This selector is really brittle. Is there a better way of styling this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mAAdhaTTah Fixed

@mAAdhaTTah mAAdhaTTah merged commit b1c2f4d into PrismJS:master Jul 21, 2018
@mAAdhaTTah
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants