fix: path map hops banner#354
Conversation
|
This path distance banner pill is shared with the Trace Path Map results. Moving it to the bottom like this will hide it behind the "Run Trace" button. What about keeping the distance banner at the top, but disabling the map ruler by setting |
|
I wonder if just replacing the "Path Map" sheet title with the details would suffice. You tapped on "Path Map" to get here so the title is kinda redundant |
|
I'm partial to the glass pill. I like seeing the map refract under it, haha. Is the issue that it seems too crowded at the top with the pill, even without the map scale? |
|
Yeah I think it would be too crowded still. We can still do the glass pill at the top in the title bar (see the chat title in #355) |
|
Ah yes, I like that idea. |
90b7731 to
65099da
Compare
|
Just updated the bubble to be in the top bar (screenshot also updated). Let me know what you think! |
|
Looks great, thanks! |
Description
Moves the path map banner pill to the bottom so it doesnt conflict with the scale element or compete with top bar
Overview of Changes
Testing
Simulator
Tested on
Checklist
CONTRIBUTING.md