Skip to content

Conversation

hasinhayder
Copy link
Contributor

Summary

This PR adds the missing cursor-pointer class to the Settings and Log out links in the dropdown menu. Previously, these links did not visually indicate interactivity, which could lead to a slightly confusing user experience.

Changes

Added cursor-pointer to both components inside the dropdown menu.
Ensured consistency with other interactive elements in the UI.

Why This Fix?

Without cursor-pointer, users may not realize that these links are clickable, especially since they are styled as buttons. This small update improves usability by aligning with common UI behavior.

Impact

  • Improved user experience by providing a clearer visual cue for interactive elements.
  • No breaking changes—only a minor class addition for better styling.

Testing

  • Manually verified that the cursor changes appropriately on hover.
  • Checked that the links still function as expected.

Without cursor-pointer, users may not realize that these links are clickable, especially since they are styled as buttons. This small update improves usability by aligning with common UI behavior.
Updated the "Current Password" label to use sentence case ("Current password") for consistency with the "New password" and "Confirm password" labels. This ensures a uniform and visually balanced UI.
@taylorotwell
Copy link
Member

This is an intentional design choice I believe.

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