Refine device info layout and add collapsible map legend#164
Conversation
|
Only tested on Android |
|
Oh good I was trying to do that to the legends, I failed. I have some new map interface I'm adding so I'll need to have this PR in before I can merge my changes. |
|
I can also add a feature that can use carto DB for folks who like black maps and could be applied in settings or as an overlays button on the map screen |
|
It would nice to have a map selection for topo and others. |
|
Ok I'll work on that later I can wait until you have added your features. If that helps |
|
Why does the github app double message.... I'll just merge my code into yours. |
|
10 4 |
|
Not sure why it sends 2 times sometimes |
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Looks good to me. Ill merge it in thanks! |
|
You're welcome |
This PR improves UI clarity and interaction consistency in two areas.
Device Info (Settings)
Device Info Screenshots
###OLD
###NEW
Map Legend
Map Legend Screenshots
No behavioral changes.
No dependency changes.
Only affects:
lib/screens/settings_screen.dartlib/screens/map_screen.dart