Skip to content

svelte: Pass keywords to CrateHeader on all crate sub-pages#13370

Merged
Turbo87 merged 2 commits intorust-lang:mainfrom
Turbo87:sv-crate-header-keywords
Apr 8, 2026
Merged

svelte: Pass keywords to CrateHeader on all crate sub-pages#13370
Turbo87 merged 2 commits intorust-lang:mainfrom
Turbo87:sv-crate-header-keywords

Conversation

@Turbo87
Copy link
Copy Markdown
Member

@Turbo87 Turbo87 commented Apr 8, 2026

The versions, dependencies, reverse dependencies, and security pages were rendering CrateHeader without passing the keywords prop, causing keywords to not appear in the header on those pages.

This PR adds regression tests for this behavior and fixes it by passing in the necessary props.

Related

Turbo87 added 2 commits April 8, 2026 13:10
The Svelte app's crate sub-pages (versions, dependencies, reverse
dependencies, security) don't pass keywords to the `CrateHeader`
component, unlike the Ember app. These tests reproduce the issue.
The versions, dependencies, reverse dependencies, and security pages
were rendering `CrateHeader` without passing the `keywords` prop,
causing keywords to not appear in the header on those pages.
@Turbo87 Turbo87 added the C-bug 🐞 Category: unintended, undesired behavior label Apr 8, 2026
@Turbo87 Turbo87 requested a review from a team April 8, 2026 11:13
Copy link
Copy Markdown
Contributor

@eth3lbert eth3lbert left a comment

Choose a reason for hiding this comment

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

Sorry, I overlooked it earlier. It’s always great to see more tests added, as this makes the system more robust and reduces human error.

View changes since this review

@Turbo87 Turbo87 merged commit facff05 into rust-lang:main Apr 8, 2026
14 checks passed
@Turbo87 Turbo87 deleted the sv-crate-header-keywords branch April 8, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-frontend 🐹 A-svelte C-bug 🐞 Category: unintended, undesired behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants