NWACus / avy

Mobile-Native Viewing of NAC Avalanche Forecasts
MIT License
10 stars 6 forks source link

Add clearing of obs location #708

Closed yuliadub closed 3 months ago

yuliadub commented 5 months ago

attempt to solve https://github.com/NWACus/avy/issues/656. This changes the behavior a bit, where to select a location you now push a "check" location and "x" to clear it. The part that is not perfect is clearing the coordinates fully, you can spot that if you clear and then come back to set it again - the point will briefly be visible at the previous location if there was one.

https://nwactechcommittee.slack.com/archives/C051KR96B2L/p1711086267002079?thread_ts=1711086244.167219&cid=C051KR96B2L

stevekuznetsov commented 4 months ago

Looks good!

yuliadub commented 4 months ago

/preview

github-actions[bot] commented 4 months ago

Preview in Expo Go

iOS Android
qrcode qrcode
exp://u.expo.dev/update/9de2cf12-5fc5-48d1-8210-61969dbaf7be exp://u.expo.dev/update/0b35487f-98c8-4cad-94dd-31efab2de566
yuliadub commented 4 months ago

/preview

github-actions[bot] commented 4 months ago

expo_preview: succeeded ✅ https://github.com/NWACus/avy/actions/runs/8729826774

yuliadub commented 4 months ago

/preview

github-actions[bot] commented 4 months ago

expo_preview: succeeded ✅ https://github.com/NWACus/avy/actions/runs/8729959185

yuliadub commented 4 months ago

/preview

github-actions[bot] commented 4 months ago

expo_preview: succeeded ✅ https://github.com/NWACus/avy/actions/runs/8977907151