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

[Draw area task] Location card obscures location lock button #2442

Open
gino-m opened this issue Apr 12, 2024 · 7 comments
Open

[Draw area task] Location card obscures location lock button #2442

gino-m opened this issue Apr 12, 2024 · 7 comments
Assignees
Labels
type: bug Something isn't working

Comments

@gino-m
Copy link
Collaborator

gino-m commented Apr 12, 2024

Screenshot 2024-04-12 at 8 24 28 AM

Screenshot 2024-04-12 at 8 25 41 AM

@rawbzz Is looking at this UI element now, may want to wait for final designs before fixing.

@gino-m gino-m added the type: bug Something isn't working label Apr 12, 2024
@gino-m gino-m added this to the 4. OSS launch (21-Jul-24) milestone Apr 12, 2024
@gino-m
Copy link
Collaborator Author

gino-m commented Apr 17, 2024

To consider in context of new popup card design

@rawbzz
Copy link

rawbzz commented Apr 19, 2024

Screenshot 2024-04-19 at 1 30 26 PM

@gino-m
Copy link
Collaborator Author

gino-m commented Apr 19, 2024

I like it! @rawbzz Should we make the styling of the line more consistent with the blue dot styling (e.g., white?)

Also, Can you update the text of current location to show Lat Lng without Altitude, and with Accuracy being shown when location lock is enabled?

@rawbzz
Copy link

rawbzz commented Apr 19, 2024

yes! how is accuracy measured? Also would i need to show lat lng if accuracy is shown? or just one or the other shows?

@gino-m
Copy link
Collaborator Author

gino-m commented Apr 19, 2024

For now we can just show meters (m). In the future we will probably want to add a setting for other units.

Lat/lng would always be shown, since it applies both to manual selection and auto/current location. Accuracy only applies when location lock is enabled.

@rawbzz
Copy link

rawbzz commented Apr 19, 2024

Maybe something like this??

Screenshot 2024-04-19 at 3 59 28 PM Screenshot 2024-04-19 at 3 59 34 PM

@gino-m
Copy link
Collaborator Author

gino-m commented Apr 19, 2024

Looks great! One nit: Can the card w/accuracy be made to fit in two lines, perhaps by showing Accuracy right aligned?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Status: No status
Development

No branches or pull requests

2 participants