Skip to content

Make the blogs highlight style inline with the website #631

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

Merged
merged 2 commits into from
Jul 13, 2020

Conversation

XAMPPRocky
Copy link
Member

Companion PR to rust-lang/www.rust-lang.org#1165

Production

Screenshot 2020-07-01 at 10 17 27

Proposed

Screenshot 2020-07-01 at 10 17 15

@rust-highfive
Copy link

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@tesuji
Copy link
Contributor

tesuji commented Jul 1, 2020

Does the change on phone still produce the same ouptput?

@XAMPPRocky
Copy link
Member Author

Screenshot 2020-07-01 at 10 55 47

@nikomatsakis
Copy link
Contributor

I don't think I'm a good assignee here!

r? @Manishearth perhaps?

@Manishearth
Copy link
Member

As I mentioned in rust-lang/www.rust-lang.org#1165 , I believe the highlight style chosen here was a deliberate choice. I've seen this kind of styling before across the web.

@XAMPPRocky XAMPPRocky requested a review from a team as a code owner July 4, 2020 15:35
@skade
Copy link
Contributor

skade commented Jul 6, 2020

Given that there's positive review on the website topic (rust-lang/www.rust-lang.org#1165 (comment)), I assume this one is good, too.

@XAMPPRocky
Copy link
Member Author

Yep, but this shouldn’t be merged yet, the www PR needs to merge and then the website needs to be deployed so that the change happens simultaneously. I also have to make a PR to thanks.

@Manishearth Manishearth merged commit 04334d9 into rust-lang:master Jul 13, 2020
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.

6 participants