Skip to content

Fix/button priority UI#150

Open
Muskankr wants to merge 2 commits into
aditya-ai00:mainfrom
Muskankr:fix/button-priority-ui
Open

Fix/button priority UI#150
Muskankr wants to merge 2 commits into
aditya-ai00:mainfrom
Muskankr:fix/button-priority-ui

Conversation

@Muskankr

Copy link
Copy Markdown

## Which issue does this PR close?


## Rationale for this change

The action buttons on the weather dashboard previously had very similar visual emphasis, which made it difficult to distinguish the primary action from secondary utility actions.

This update improves the visual hierarchy by making the Search button more prominent while reducing the emphasis of secondary controls like geolocation, theme toggle, unit toggle, and AQ-check.

The changes were kept minimal and focused only on button prioritization as requested.


## What changes are included in this PR?

  • Enhanced the visual prominence of the Search button
  • Reduced visual dominance of secondary action buttons
  • Improved button spacing, shadows, and hover states
  • Removed unnecessary attention-grabbing animation from AQ-check button
  • Preserved existing functionality and responsiveness
  • Maintained dark mode compatibility

## Are these changes tested?

Yes.

Tested:

  • Search functionality
  • Geolocation button
  • Unit toggle
  • Dark mode toggle
  • Responsive layout on smaller screens

All existing button functionality remains unchanged.


## Are there any user-facing changes?

Yes.

Users will now see clearer visual prioritization between primary and secondary actions, improving overall usability and UI clarity.


## Screenshots

Before

Screenshot 2026-05-17 001341

After

Screenshot 2026-05-17 001606

Thank you for reviewing my PR!

@vercel

vercel Bot commented May 16, 2026

Copy link
Copy Markdown

@Muskankr is attempting to deploy a commit to the aditya-ai00's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Muskankr

Copy link
Copy Markdown
Author

Hi @aditya-ai00, I created PR , Please review and let me know if any changes required further.

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.

Button Priority Is Unclear

1 participant