Skip to content

Fix Compound Link elements not having an underline.#31583

Merged
Half-Shot merged 8 commits intodevelopfrom
hs/prevent-overriding-compound-links
Jan 5, 2026
Merged

Fix Compound Link elements not having an underline.#31583
Half-Shot merged 8 commits intodevelopfrom
hs/prevent-overriding-compound-links

Conversation

@Half-Shot
Copy link
Member

@Half-Shot Half-Shot commented Dec 19, 2025

Blocked on element-hq/compound-web#418

Specifically this is fixing the "Messages you send will be shared with new members invited to this room" banner not having a proper underline on Learn more.

Checklist

@Half-Shot Half-Shot requested a review from a team as a code owner December 19, 2025 12:33
<Link>.
*/
a:not([data-kind]) {
&:hover,
Copy link
Member Author

Choose a reason for hiding this comment

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

while I'm here, I was struggling to really figure out what this is doing. I hate that we perform blanket CSS changes links across the app, but I think this can only now be fixed by replacing our bare tags with proper components.

Copy link
Member Author

@Half-Shot Half-Shot left a comment

Choose a reason for hiding this comment

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

@Half-Shot Half-Shot requested a review from a team as a code owner January 2, 2026 17:08
Copy link
Member

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

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

Fine from a crypto point of view.

@Half-Shot Half-Shot added this pull request to the merge queue Jan 5, 2026
Merged via the queue into develop with commit cf7bf71 Jan 5, 2026
36 checks passed
@Half-Shot Half-Shot deleted the hs/prevent-overriding-compound-links branch January 5, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants