Skip to content

rustdoc: improve stability mark arrows #67959

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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
Collaborator

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
# for free to join this conversation on GitHub. Already have an account? # 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