-
Notifications
You must be signed in to change notification settings - Fork 120
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
[Add point/polygon] Prompt user whether to add point, draw perimeter, or capture location as allowed by organizer #2469
Comments
@shobhitagarwal1612 Are you able to take one this issue? |
I have a couple of tasks already ongoing. We can take this one up once the proto migration is done. |
@anandwana001 can you please coordinate with @rfontanarosa, who is working on google/ground-platform#1744? You only need to show to pictured UI if more than one value is specified in |
@gino-m could you please help me with the figma access for this design? earlier password no more working |
@anandwana001 did you get access to the figma? Anything else needed for this one? |
@anandwana001 Gentle ping! |
Sorry for the late response Yes, got the figma access. Got all doubts cleared now. Thanks. |
@anandwana001 We spoke. Let's try to implement this first, but eventually start creating tasks dynamically as they're needed rather than at the start of the data collection flow. Related to #2829 and likely other data collection issues. @shobhitagarwal1612 FYI |
@rawbzz A few edge case questions:
All of this considered, should we be showing this selector directly on the task screen somehow, eg in place of the map? @amegantz Can we think of a better prompt than "How will you map this site?" Note we already show "Map a new site" above, so we could just remove the title all together and just show the buttons where the map is currently? |
@rawbzz Looks great, thank you! Added a few variants to the Figma for your input when you have a sec 🙏 |
@rawbzz, thanks for your feedback on Figma. @anandwana001 Latest tweaked designs: |
@rawbzz @vittorino FYI
The text was updated successfully, but these errors were encountered: