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 code block information icon position #62238

Merged

Conversation

GuillaumeGomez
Copy link
Member

Fixes #62118.

A screenshot of the fix:

Screenshot 2019-06-29 at 18 28 59

r? @rust-lang/rustdoc

@GuillaumeGomez GuillaumeGomez added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 29, 2019
@GuillaumeGomez
Copy link
Member Author

As usual, small fix, a few days passed and still no comments so I'll just r+ it... I'll add the corresponding test in rust-docs-test-ui repository.

@bors: r=rustdoc rollup

@bors
Copy link
Contributor

bors commented Jul 4, 2019

📌 Commit e861efd has been approved by rustdoc

@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 Jul 4, 2019
@QuietMisdreavus
Copy link
Member

@bors r-

Can you post a before/after screenshot for actual bulleted/numbered lists?

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 4, 2019
@QuietMisdreavus
Copy link
Member

Never mind, i just tested this myself. In the future, please refrain from approving your own PRs, no matter how small. If you have several small PRs waiting for review, please ping me directly, on Discord, so i can read through them. I have too many GitHub notifications now to keep up with everything.

@bors r+

@bors
Copy link
Contributor

bors commented Jul 4, 2019

📌 Commit e861efd has been approved by QuietMisdreavus

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 4, 2019
@GuillaumeGomez
Copy link
Member Author

The before was in the linked issue. And sure, will do that next time!

Centril added a commit to Centril/rust that referenced this pull request Jul 5, 2019
…mation-icon-pos, r=QuietMisdreavus

Fix code block information icon position

Fixes rust-lang#62118.

A screenshot of the fix:

<img width="720" alt="Screenshot 2019-06-29 at 18 28 59" src="https://user-images.githubusercontent.com/3050060/60386900-edb23b80-9a9b-11e9-9f4f-0f343674348c.png">

r? @rust-lang/rustdoc
Centril added a commit to Centril/rust that referenced this pull request Jul 5, 2019
…mation-icon-pos, r=QuietMisdreavus

Fix code block information icon position

Fixes rust-lang#62118.

A screenshot of the fix:

<img width="720" alt="Screenshot 2019-06-29 at 18 28 59" src="https://user-images.githubusercontent.com/3050060/60386900-edb23b80-9a9b-11e9-9f4f-0f343674348c.png">

r? @rust-lang/rustdoc
bors added a commit that referenced this pull request Jul 5, 2019
Rollup of 10 pull requests

Successful merges:

 - #62123 ( Remove needless lifetimes (std))
 - #62150 (Implement mem::{zeroed,uninitialized} in terms of MaybeUninit.)
 - #62169 (Derive which queries to save using the proc macro)
 - #62238 (Fix code block information icon position)
 - #62292 (Move `async || ...` closures into `#![feature(async_closure)]`)
 - #62323 (Clarify unaligned fields in ptr::{read,write}_unaligned)
 - #62324 (Reduce reliance on `await!(...)` macro)
 - #62371 (Add tracking issue for Box::into_pin)
 - #62383 (Improve error span for async type inference error)
 - #62388 (Break out of the correct number of scopes in loops)

Failed merges:

r? @ghost
@bors bors merged commit e861efd into rust-lang:master Jul 5, 2019
@GuillaumeGomez GuillaumeGomez deleted the fix-code-block-information-icon-pos branch July 5, 2019 20:35
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. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid position of code block warning/error sign in a list
3 participants