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 moving text in search tabs headers #59026

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

GuillaumeGomez
Copy link
Member

Fixes #59005.

Now, the text in the search tabs headers isn't moving anymore.

r? @QuietMisdreavus

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 8, 2019
@Dylan-DPC-zz
Copy link

ping from triage @QuietMisdreavus waiting for your review on this

@QuietMisdreavus
Copy link
Member

Where is this issue occurring? What problem is this fixing?

@Dylan-DPC-zz Dylan-DPC-zz 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-review Status: Awaiting review from the assignee but also interested parties. labels Mar 18, 2019
@GuillaumeGomez
Copy link
Member Author

I can't make a screenshot for it so you'll have to test it yourself: once you have search results (or not results, it doesn't matter), just switch between tabs and look at the tabs' headers' text when siwtch (doing it with tab is more obvious). In current rendering, the text moves a bit, this PR fixes this move.

@QuietMisdreavus
Copy link
Member

Ah, now i see it. Thanks for the explanation!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Mar 26, 2019

📌 Commit 6a4cced 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 Mar 26, 2019
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Mar 26, 2019
…r=QuietMisdreavus

Fix moving text in search tabs headers

Fixes rust-lang#59005.

Now, the text in the search tabs headers isn't moving anymore.

r? @QuietMisdreavus
bors added a commit that referenced this pull request Mar 26, 2019
Rollup of 7 pull requests

Successful merges:

 - #59004 ([rustdoc] Improve "in parameters" search and search more generally)
 - #59026 (Fix moving text in search tabs headers)
 - #59197 (Exclude old book redirect stubs from search engines)
 - #59330 (Improve the documentation for std::convert (From, Into, AsRef and AsMut))
 - #59424 (Fix code block display in portability element in dark theme)
 - #59427 (Link to PhantomData in NonNull documentation)
 - #59432 (Improve some compiletest documentation)

Failed merges:

r? @ghost
bors added a commit that referenced this pull request Mar 27, 2019
Rollup of 7 pull requests

Successful merges:

 - #59004 ([rustdoc] Improve "in parameters" search and search more generally)
 - #59026 (Fix moving text in search tabs headers)
 - #59197 (Exclude old book redirect stubs from search engines)
 - #59330 (Improve the documentation for std::convert (From, Into, AsRef and AsMut))
 - #59424 (Fix code block display in portability element in dark theme)
 - #59427 (Link to PhantomData in NonNull documentation)
 - #59432 (Improve some compiletest documentation)

Failed merges:

r? @ghost
@bors bors merged commit 6a4cced into rust-lang:master Mar 27, 2019
@GuillaumeGomez GuillaumeGomez deleted the search-tabs-header branch March 27, 2019 09:10
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.

5 participants