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

Widget fullscreen UX feels like a toggle, but isn't (resulting in **suprise**) #21683

Closed
lampholder opened this issue Apr 5, 2022 · 2 comments
Labels
A-Widgets O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Low/no impact on users T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter Z-Maximised-Widgets

Comments

@lampholder
Copy link
Member

Your use case

What would you like to do?

I would like to:

  • click the two diagonal arrows to open the widget full screen
  • click it again to go back to pinned

Instead I click it again and it disappears! And I am surprised.

Have you considered any alternatives?

No response

Additional context

This feedback has been echoed by a customer, too.

@jryans
Copy link
Collaborator

jryans commented Apr 5, 2022

Yes, this is effectively the same as #20506. We tried to improve this somewhat in matrix-org/matrix-react-sdk#7734 by explicitly showing both maximise and pin actions, but perhaps it's not enough.

To summarise the current state, at the moment, "pinned" and "maximised" are really locations a widget moves to (as opposed to independent flags), so when you maximise, the widget has no knowledge that it used to be pinned.

@lampholder, did you notice the separate pin button which would have pinned it? It might be best to transform this report into an additional feedback comment on #20506.

@dbkr dbkr added S-Tolerable Low/no impact on users O-Occasional Affects or can be seen by some users regularly or most users rarely labels Apr 7, 2022
@jryans jryans added T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter and removed T-Enhancement labels Apr 13, 2022
@andybalaam
Copy link
Member

I think this is a duplicate of #20506. @lampholder if you disagree, please re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Widgets O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Low/no impact on users T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter Z-Maximised-Widgets
Projects
None yet
Development

No branches or pull requests

4 participants