Skip to content

"Close" button for user sidebar actually brings up user list #11965

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

Closed
jdm opened this issue Jan 21, 2020 · 3 comments
Closed

"Close" button for user sidebar actually brings up user list #11965

jdm opened this issue Jan 21, 2020 · 3 comments
Assignees

Comments

@jdm
Copy link

jdm commented Jan 21, 2020

Description

When I click on a user's avatar in a room, it brings up the user sidebar. When I press the button labeled close in order to close this sidebar, it gets replaced with the room member list. I then have to press the "Members" button to hide this sidebar.

Steps to reproduce

  • click on an avatar in a room
  • click on the sidebar's close button

My expectation is that pressing the button labeled "Close" will cause the room to return to its state from before I clicked on the avatar.

Logs being sent: no

Base room state, about to click on stas' avatar:
Screen Shot 2020-01-21 at 11 32 30 AM

Before pressing the "<" button labelled "Close":
Screen Shot 2020-01-21 at 11 32 39 AM

After pressing the "<" button labelled "Close":
Screen Shot 2020-01-21 at 11 32 44 AM

Version information

  • Platform: web (in-browser)

For the web app:

  • Browser: Firefox 74 (nightly)
  • OS: macOS 10.14
  • URL: chat.mozilla.org
@michal-weglarz
Copy link

At this point I find it very confusing what that button is intended to do. It tells user neither to collapse sidepanel (I'd expect the arrow to be turned the other way- to the right: ">"), nor to display list of all users in a room (its label says "Close"). It kind of contradicts itself and there should be made some decision what's the real intention and deliver it to the end user more clearly. Anyway, It looks like it's more of a displaying all room's members than closing sidebar entirely so I'd opt for changing the label.

@t3chguy
Copy link
Member

t3chguy commented Jan 22, 2020

It's intended to close the User Info panel, and reveal the Member List panel below it, but in your case you're right the Member List should not be waiting there below it.

@t3chguy
Copy link
Member

t3chguy commented Sep 9, 2020

Fixed by matrix-org/matrix-react-sdk#5167

@t3chguy t3chguy closed this as completed Sep 9, 2020
@jryans jryans removed the Z-UI/UX label Mar 8, 2021
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