Skip to content
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

Create room: Improve Userflow when adding users to a room #2205

Closed
1 of 2 tasks
VolkerJunginger opened this issue Nov 20, 2023 · 9 comments
Closed
1 of 2 tasks

Create room: Improve Userflow when adding users to a room #2205

VolkerJunginger opened this issue Nov 20, 2023 · 9 comments

Comments

@VolkerJunginger
Copy link
Contributor

VolkerJunginger commented Nov 20, 2023

Story

When creating a room the userflow is not very good right now (see attached video).

Instead of forcing the user to cancel the search before they can continue to create the room the user should be able to continue. The change to a search focusses screen seem unnecessary.

RPReplay_Final1700468777.MP4

Dependencies

  • TBD

Sign-offs

Android

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

iOS

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

Scope

Android

iOS

Out of scope

  • TBD
@callumu
Copy link

callumu commented Nov 28, 2023

On my iPhone I see the required behaviour which is this:

RPReplay_Final1701184772.MP4

It seems like this is a bug on your end

@Velin92
Copy link
Member

Velin92 commented Nov 28, 2023

There is definitely a bug, I was currently investigating this, since it was an easy fix, and realised that sometimes the top bar is hidden and sometimes not, even when it should not be.

@Velin92
Copy link
Member

Velin92 commented Nov 28, 2023

I found a fix here:
element-hq/element-x-ios#2182

@Velin92 Velin92 closed this as completed Nov 28, 2023
@callumu
Copy link

callumu commented Nov 29, 2023

I found a fix here: vector-im/element-x-ios#2182

Nice! Is there a way to add the 'Cancel' button in like the video I shared above? It's not a huge issue this is mainly for consistency

@Velin92 Velin92 reopened this Nov 29, 2023
@Velin92
Copy link
Member

Velin92 commented Nov 29, 2023

Sadly I have to reopen the issue, on iOS 17.1.1 there is a bug where this fix makes the screen scrambled and look like this. Seems to only happen on 17.1.1
IMG_0881

Looking into it

@Velin92
Copy link
Member

Velin92 commented Nov 29, 2023

At this point I am reverting my fix, in the hope that when a new update of iOS is going to get released this issue won't happen anymore.

@Velin92
Copy link
Member

Velin92 commented Dec 1, 2023

We fixed the scrambled screen issue however on iOS 17.1.1 is not possible to NOT hide the navigation bar in any way on 17.1.1

I tested the 17.2 Beta, where is technically possible to do a fix that allows the navigation bar to not hide, but the fix may break on previous versions, which is fine we could handle that so that it doesn't, but since Apple is making this system component less customisable on SwiftUI to allow for such a change, maybe we should consider changing the designs for the search bar

@manuroe
Copy link
Member

manuroe commented Dec 1, 2023

@VolkerJunginger do you want to do something on android too?

@manuroe manuroe added the T-Epic Issue is at Epic level label Dec 6, 2023
@pixlwave
Copy link
Member

pixlwave commented Dec 6, 2023

I've fixed this on iOS here: element-hq/element-x-ios#2209

Note that the only down side is that the search field pops in after the invites screen has been pushed, but this seems like a reasonable trade-off to fix the flow.

Simulator.Screen.Recording.-.iPhone.15.-.2023-12-06.at.15.49.10.mp4

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

No branches or pull requests

5 participants