google / ground-android

Ground mobile data collection app for Android
http://groundplatform.org
Apache License 2.0
244 stars 119 forks source link

[Survey list] Show hamburger menu on survey list #2660

Open gino-m opened 3 months ago

gino-m commented 3 months ago

After first sign in the "<-" button has no effect. The proposed solution is:

Image

gino-m commented 3 months ago

Or should we just implement the new design? In that case, what does the hamburger icon do right after sign-in?

obsolete image deleted

rawbzz commented 3 months ago

Shouldn't the hamburger menu open up the side panel?

gino-m commented 3 months ago

Shouldn't the hamburger menu open up the side panel?

That would make sense, a few follow-up questions:

rawbzz commented 3 months ago

I feel like it'd be okay to remove from the survey list, esp if the main place a profile will live is in the side panel?

and also for no survey selected, I think we can do 2 things. One either prompt a user to choose a survey or hide that section completely?

Screenshot 2024-08-19 at 4 13 15 PM
gino-m commented 3 months ago

I think the last option makes sense, as it makes it clear that no survey is selected. Offline map imagery isn't per-survey, so it can stay active, but "Data sync status" (ex "History and sync status") should be disabled

rawbzz commented 3 months ago

ok! disabled the data sync status

Screenshot 2024-08-21 at 12 48 37 PM
gino-m commented 3 months ago

LGTM!

gino-m commented 3 months ago

@amegantz for UXW - "you current have no survey.." sounds redundant.

amegantz commented 2 months ago

How about this? image No survey selected yet [Select a survey]

gino-m commented 2 months ago

@amegantz Do we need the work "yet"? If feels a little extra, no?

amegantz commented 2 months ago

YOU feel a little extra.

Just kidding, we can do without the "yet." It's a casual tone thing I added since we had the space

gino-m commented 2 months ago

Haha, ok, sgtm!

gino-m commented 1 week ago

There's already a fixed issue to disable the up button #2692, so recasting this as a feature request.