-
Notifications
You must be signed in to change notification settings - Fork 0
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
Review: 'Invisible Mode Overlay' #4829
Comments
We can update the text and icon to something like this? |
Option 2 looks nice but I vote for Option 1 as it's a much simpler text change. |
Whoops, I forgot to follow up on this. Hmmm ... can we get rid of the overlay? Does it still serve a purpose? Yesterday, I was debugging some stuff related to invisible mode and the overlay just got in my way. Yes, it prevented me from using the header widget but I could still access the user-in-the-header-widget-I-was-looking-for by going to my list of Friends. So the header widget didn't actually prevent me from doing anything. I was able to do what I wanted ... just that it was more troublesome to do it. |
I think the overlay is still useful for telling the user why they're not seeing others' locations if they're in ghost mode. Unless we're talking about options 2 above which would take them to another popup when they tap on a user portrait. |
It used to be the case that, if you were in invisible/hidden/ghost mode, you wouldn't see the locations of people who were sharing to you. It's not the case anymore. |
I think the overlay is helpful in letting the user know they are in invisible mode, but I think it needs to be changed. The reasoning is, now that we have gotten rid of the "current location" cta, we need the top header to allow the user to return to current location (by tapping on their avatar). Because of this, I think option 2 (of the two options discussed) is the best way forward. That said, we could come up with an option 3-4, but we need to keep in mind that we should be able to tap on the users avatar to return to "current location". @irfirl any comments on this? |
@thescurry Ah yes, I think in that case option 2 would be better. |
The 'Invisible Mode Overlay' (see https://zpl.io/29RKLmW) is probably out of date. The text probably needs changing, or maybe ... Do we even still want it?
cc @thescurry @irfirl @mstidham
The text was updated successfully, but these errors were encountered: