Skip to content

Conversation

@TkDodo
Copy link
Collaborator

@TkDodo TkDodo commented Oct 28, 2025

in UI2, only use the hover effects of the transparent button instead of styling the surrounding container. No changes in UI2, only UI2:

before after
Screenshot 2025-10-28 at 13 28 19 Screenshot 2025-10-28 at 13 28 43
Screenshot 2025-10-28 at 13 28 36 Screenshot 2025-10-28 at 13 28 12

in UI2, only use the hover effects of the transparent button instead of styling the surrounding container
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 28, 2025
@TkDodo TkDodo marked this pull request as ready for review October 28, 2025 12:29
@TkDodo TkDodo requested a review from a team as a code owner October 28, 2025 12:29
cursor[bot]

This comment was marked as outdated.

@TkDodo TkDodo merged commit 7a0f8c1 into master Oct 28, 2025
48 checks passed
@TkDodo TkDodo deleted the tkdodo/fix/disclosure-hover branch October 28, 2025 16:16
priscilawebdev pushed a commit that referenced this pull request Oct 29, 2025
in UI2, only use the hover effects of the transparent button instead of
styling the surrounding container. No changes in UI2, only UI2:

| before | after |
|--------|--------|
| <img width="454" height="85" alt="Screenshot 2025-10-28 at 13 28 19"
src="https://github.com/user-attachments/assets/d34482bb-ad8e-4be6-abe0-ca41d6bd65c7"
/> | <img width="466" height="91" alt="Screenshot 2025-10-28 at 13 28
43"
src="https://github.com/user-attachments/assets/2cf558fc-9435-4a21-a05e-ac4dcbd8e261"
/> |
| <img width="463" height="86" alt="Screenshot 2025-10-28 at 13 28 36"
src="https://github.com/user-attachments/assets/4571a5cb-f197-487b-ad55-73da83afed93"
/> | <img width="460" height="81" alt="Screenshot 2025-10-28 at 13 28
12"
src="https://github.com/user-attachments/assets/a84c6794-a816-4761-aff3-873e567814a7"
/> |
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.

3 participants