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

rustdoc: improve stability mark arrows #67959

Merged
merged 2 commits into from
Jan 13, 2020
Merged

rustdoc: improve stability mark arrows #67959

merged 2 commits into from
Jan 13, 2020

Conversation

liigo
Copy link
Contributor

@liigo liigo commented Jan 7, 2020

current

old-stability-arrow

new

new-stability-arrow

new dark

dark-stability-arrow

@liigo
Copy link
Contributor Author

liigo commented Jan 7, 2020

cc #51387

@JohnTitor JohnTitor added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 7, 2020
@Dylan-DPC-zz
Copy link

r? @GuillaumeGomez

Copy link
Member

@GuillaumeGomez GuillaumeGomez left a comment

Choose a reason for hiding this comment

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

Just one change to make, otherwise looks good to me.

left: -13px;
top: -6px;
left: -15px;
color: #999;
Copy link
Member

Choose a reason for hiding this comment

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

Everything color-linked should be put inside light.css and dark.css files (even if it's the same in both files.

@liigo
Copy link
Contributor Author

liigo commented Jan 8, 2020

@GuillaumeGomez done.

content: '˪';
font-size: 30px;
content: '';
transform:rotate(90deg);
Copy link
Member

Choose a reason for hiding this comment

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

Also I'm not a fan of this. I'm pretty sure the expected character exists as is.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I failed to find it. Will check that again.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed to another arrow: ⬑

rotate-vs-nonrotate

@liigo
Copy link
Contributor Author

liigo commented Jan 9, 2020

Now we use the 'Leftwards Arrow with Tip Upwards': ⬑ (U+2B11) (Not the rotated ⤶)
@GuillaumeGomez

rotate-vs-nonrotate

@GuillaumeGomez
Copy link
Member

Thanks a lot!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jan 11, 2020

📌 Commit ae3a53f has been approved by GuillaumeGomez

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 11, 2020
bors added a commit that referenced this pull request Jan 13, 2020
Rollup of 8 pull requests

Successful merges:

 - #67313 (Document more use cases of dataflow)
 - #67959 (rustdoc: improve stability mark arrows)
 - #68097 (Specify units for test timeout environment variables)
 - #68135 (restore some rustc_parse visibilities for rustfmt)
 - #68145 (Expose `context::CheckLintNameResult`)
 - #68156 (Fix crate paths in comments)
 - #68157 (Clean up E0186 explanation)
 - #68161 (Fix system call docs for time::Instant)

Failed merges:

r? @ghost
@bors bors merged commit ae3a53f into rust-lang:master Jan 13, 2020
@liigo liigo deleted the patch-13 branch January 14, 2020 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants