Skip to content

fix: added cursor pointer to .btn class for better UX#41602

Closed
Nafisa-tamboli wants to merge 1 commit intotwbs:mainfrom
Nafisa-tamboli:main
Closed

fix: added cursor pointer to .btn class for better UX#41602
Nafisa-tamboli wants to merge 1 commit intotwbs:mainfrom
Nafisa-tamboli:main

Conversation

@Nafisa-tamboli
Copy link
Copy Markdown

Added cursor: pointer; to the .btn class in _buttons.scss for consistent UX across browsers. Enhances accessibility and clarity of interactive elements.

@Nafisa-tamboli Nafisa-tamboli requested a review from a team as a code owner July 20, 2025 14:58
@Nafisa-tamboli
Copy link
Copy Markdown
Author

Hey maintainers! 👋
This PR adds cursor: pointer; to the .btn class in _buttons.scss to ensure all buttons consistently show the pointer cursor across browsers. Improves UX and interaction clarity, especially in custom setups.

Let me know if any changes are needed. Happy to contribute! 😊✨

@julien-deramond
Copy link
Copy Markdown
Member

Thanks for your contribution, @Nafisa-tamboli. However, the Sass code provided in this PR is currently invalid. More importantly, the proposed change lacks context on the specific use cases it aims to address, especially since the existing implementation already works as expected in most browsers. For example, PRs like #41577 are designed to override this behavior only when necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants