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

fix: ayu theme meta highlighting #1133

Merged

Conversation

EverlastingBugstopper
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper commented Jan 30, 2020

Was previously highlighting #[derive(Clone)] like it was a comment. This updates the css to match the grouping of other themes.

For an example see https://rust-lang.github.io/async-book/02_execution/04_executor.html. Note the #[derive(Clone)] highlighting color in line 7 of the following screenshots

For reference, here is the regular theme:

image

And here is the ayu theme today:

image

And here is the ayu theme after the change proposed in this PR:

image

@Byron
Copy link
Member

Byron commented Jan 31, 2020

Failure seems related to the issue fixed via #1131 .

@EverlastingBugstopper
Copy link
Contributor Author

@Byron - I don't think so? Willing to be corrected but this seems like an issue with the css. I've updated my submission to include reference screenshots.

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Yea, it looks better to me. Thanks!

@ehuss ehuss merged commit c4a64ab into rust-lang:master Feb 1, 2020
Ruin0x11 pushed a commit to Ruin0x11/mdBook that referenced this pull request Aug 30, 2020
…x-derive-highlighting

fix: ayu theme meta highlighting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants