Skip to content

Fix release description hover jitter - #3392

Merged
GuillaumeGomez merged 2 commits into
rust-lang:mainfrom
Herrtian:fix/recent-releases-descriptions
Jul 23, 2026
Merged

Fix release description hover jitter#3392
GuillaumeGomez merged 2 commits into
rust-lang:mainfrom
Herrtian:fix/recent-releases-descriptions

Conversation

@Herrtian

@Herrtian Herrtian commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • keep release descriptions truncated on hover so release rows do not change height
  • add title text so full descriptions remain available without layout shifts
  • extend the long description GUI test to cover hover row height

Fixes #173

Validation

  • cargo check -p docs_rs_web
  • cargo fmt --all --check
  • git diff --check

@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Jul 2, 2026
Comment thread crates/bin/docs_rs_web/templates/core/home.html Outdated

@GuillaumeGomez GuillaumeGomez 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.

Test looks good, however like mentioned, we cannot simply add the description like that in case it contains double quotes.

View changes since this review

@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks!

@GuillaumeGomez
GuillaumeGomez enabled auto-merge (rebase) July 23, 2026 20:03
@GuillaumeGomez
GuillaumeGomez merged commit c7165b9 into rust-lang:main Jul 23, 2026
13 checks passed
@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 Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recent Releases shaking when descriptions are too long

2 participants