Skip to content

Fix display of down arrow in topbar when crate name is too long - #3569

Merged
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
GuillaumeGomez:arrow-crate-name
Sep 30, 2026
Merged

GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
GuillaumeGomez:arrow-crate-name

Conversation

@GuillaumeGomez

Copy link
Copy Markdown
Member

Fixes #3566.

With this fix it looks like this:

image

@GuillaumeGomez
GuillaumeGomez requested a review from a team as a code owner September 29, 2026 12:52
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Sep 29, 2026
Comment thread gui-tests/crate-topbar-dropdown-menu.goml

@syphar syphar left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice!

can you rebase so the gui tests run in CI?

Then you can merge when green

View changes since this review

@rustbot

rustbot commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@GuillaumeGomez
GuillaumeGomez merged commit e27fd1f into rust-lang:main Sep 30, 2026
10 checks passed
@GuillaumeGomez
GuillaumeGomez deleted the arrow-crate-name branch September 30, 2026 09:46
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Sep 30, 2026
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Sep 30, 2026

This branch was successfully deployed

1 active deployment
github-pages — bf729e40 Deployed Sep 30, 2026 by GuillaumeGomez via deploy #576
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.

Long crate names hide dropdown arrow

3 participants