Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add optional mobile guide toast #5586

Merged
merged 1 commit into from
Jan 29, 2021
Merged

Add optional mobile guide toast #5586

merged 1 commit into from
Jan 29, 2021

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Jan 28, 2021

This adds an optional toast which Element deployments can enable to nudge users towards the mobile apps once they are inside the app. It will only appear on iOS and Android web browsers.

image

Part of element-hq/element-web#16283
Documented in element-hq/element-web#16301

This adds an optional toast which Element deployments can enable to nudge users
towards the mobile apps once they are inside the app.

Part of element-hq/element-web#16283
@jryans jryans requested a review from a team January 28, 2021 17:24
@jryans jryans requested a review from a team January 28, 2021 17:31
Copy link
Member

@benparsons benparsons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(discussed in room separately)
lgtm

@jryans
Copy link
Collaborator Author

jryans commented Jan 29, 2021

Okay, since this is only optional, I'm going to go ahead and merge it so it's part of today's RC.

@jryans jryans merged commit b8e3908 into develop Jan 29, 2021
@niquewoodhouse
Copy link
Contributor

👋 @jryans just a couple of (super small) concerns from design, but they're nothing major at all/anything worth blocking shipping. I'm probably just missing some context around why a toast etc, so just commenting in case helpful, but feel to free to not respond if busy.

- When does this appear? Is it noticeable enough?

If people's experience of Element will be broken, they'll be more likely to decide not to return/think of the product as useful. So, if we have a modal component, I'd suggest it might be better to block people briefly, but at least be sure they know we have stable options somewhere?

Basic example
Screenshot 2021-01-29 at 12 16 24

- Copy
Some copy changes could improve clarity.
As a user, I wouldn't be sure what learn more does, does it go to a page to learn more/does it open the app store? And the heading "mobile experience" doesn't say much. Something like the following might help encourage interaction:

  • Use apps for a better mobile experience Mobile experience
  • Element web is experimental on mobile. For a better experience and the latest features, use our free app Element web is....
  • Download app Learn more

@jryans
Copy link
Collaborator Author

jryans commented Jan 29, 2021

@niquewoodhouse Thanks for the feedback! To clarify slightly, this toast is an off-by-default thing we're trialling with some deployments, so it's at least not going out to the entire user base at once, which I think buys us a bit more time for iterating and improving.

About it being a full-screen overlay instead of toast... I used a toast partly out of expediency and partly because we want to still allow a 3PID room invite to be completed on screen. It's certainly a messy UX for sure, so we should aim for something better / more holistic where there's more time available. I'll copy these notes to a separate issue for the future.

About the copy changes, those look good to me, so I'll add those now. 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants