Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: search improvements #67

Closed

Conversation

juandiegocv27
Copy link
Contributor

Enhancements to Search UI

  • Query Lock: Interaction is now disabled while a search query is pending, preventing duplicate queries and enhancing user experience.

  • UI Simplification: Removed tabs from the search dropdown to simplify the interface, making the search results cleaner and more straightforward.

  • Search Bar Styling: Updated the homepage search bar styling to align with the Optimistic Etherscan's layout, maintaining thematic consistency with the OP Mainnet Explorer.

image

note: update the code format with prettier in base with guidelines:

pnpm prettier

This update addresses various components of the search UI to improve functionality and aesthetics as outlined in Issue #47.

Copy link

vercel bot commented Aug 6, 2024

@juandiegocv27 is attempting to deploy a commit to the Walnut Team on Vercel.

A member of the Team first needs to authorize it.

@juandiegocv27
Copy link
Contributor Author

@saimeunt PR just fresh to merge!🚀🇨🇷

Copy link
Collaborator

@saimeunt saimeunt left a comment

Choose a reason for hiding this comment

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

@juandiegocv27 LGTM overall, please add the required dependency cmdk in package.json before we merge.

@juandiegocv27
Copy link
Contributor Author

@saimeunt already add cmdk dependency🫡

@juandiegocv27
Copy link
Contributor Author

@saimeunt I´ll continue trying to resolve the conflicts 🙌🏻

@juandiegocv27
Copy link
Contributor Author

@saimeunt I resolved the confilcts

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