Skip to content

fix(ui): Disable links in project selector avatars#114775

Merged
scttcper merged 1 commit intomasterfrom
scttcper/project-avatar-links
May 4, 2026
Merged

fix(ui): Disable links in project selector avatars#114775
scttcper merged 1 commit intomasterfrom
scttcper/project-avatar-links

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented May 4, 2026

Project selector chips and dropdown rows use project badges just for the avatar, but those badges still wrapped the avatar in a project details link.

Pass disableLink for both the selected multi-value chips and the dropdown options so the selector does not hide links inside its UI.

Project selector chips and dropdown rows use IdBadge with hideName, but the badge still wrapped the avatar in a project details link. Pass disableLink in both places so the select behaves like a select instead of hiding links inside it.

Co-authored-by: Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.40%

@scttcper scttcper marked this pull request as ready for review May 4, 2026 20:54
@scttcper scttcper requested a review from a team as a code owner May 4, 2026 20:54
@scttcper scttcper enabled auto-merge (squash) May 4, 2026 20:56
@scttcper scttcper merged commit fa197ca into master May 4, 2026
74 checks passed
@scttcper scttcper deleted the scttcper/project-avatar-links branch May 4, 2026 20:56
cleptric pushed a commit that referenced this pull request May 5, 2026
Project selector chips and dropdown rows use project badges just for the
avatar, but those badges still wrapped the avatar in a project details
link.

Pass `disableLink` for both the selected multi-value chips and the
dropdown options so the selector does not hide links inside its UI.

Co-authored-by: Codex <noreply@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants