Git checkout b feature/UI icon tabs#357
Closed
nngeek195 wants to merge 2 commits intovalhalla:masterfrom
Closed
Conversation
|
Preview is ready! 🚀 You can view it here: https://valhalla-app-tests.gis-ops.com/357 |
Member
|
Pure AI. Sorry dude, he that readeth not, shall not write. |
Author
It was easy that's why I use, I gave the nessary icons and just did that, If you propose, sure sir I will continue other manually. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses the UI enhancement requested in #353. It moves the primary navigation from a hidden sheet trigger to a persistent, high-visibility vertical icon bar in the top-left corner. Additionally, it refactors the "Open OSM" action into a standardized floating icon button to improve map real estate and consistency.
Changes
src/components/route-planner.tsx
Vertical Tab Controls: Replaced the SheetTrigger with a persistent floating div in the upper-left corner containing three high-contrast icon buttons:
src/components/map/index.tsx
OSM Button : Updated the "Open OSM" button with using OSM original logo(add it in svg)
Related Issue
Closes #353