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

Button to close side panel is missing #27799

Closed
kongo09 opened this issue Jul 22, 2024 · 13 comments · Fixed by element-hq/matrix-react-sdk#99
Closed

Button to close side panel is missing #27799

kongo09 opened this issue Jul 22, 2024 · 13 comments · Fixed by element-hq/matrix-react-sdk#99
Assignees
Labels
A-Right-Panel O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect

Comments

@kongo09
Copy link

kongo09 commented Jul 22, 2024

On receiving feedback on the new room header and right panel, it's clear that folks are missing the [x] button.

This issue represents the work needed to add the close button to the right panel. Please see the designs for more details (designs are internal only, screenshots of expected outcome are in the comments below)

Thanks!


_Original Issue_

Steps to reproduce

  1. Click on (i) in header to open side panel on the right

Outcome

What did you expect?

A button to close the side panel, typically an X in the top right

What happened instead?

No obvious way to close the side panel is visible

image

Side panel in Edge:

image

Side panel in Slack:

image

Side panel toggle button in MS Teams

image

It seems a X is typical, but even without the X there are alternatives in the top right to close the panel

Operating system

Windows 11

Application version

Element version: 1.11.71 Crypto version: Rust SDK 0.7.1 (80a151e), Vodozemac 0.6.0

How did you install the app?

element.io

Homeserver

element.io

Will you send logs?

No

@dosubot dosubot bot added A-Right-Panel O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Jul 22, 2024
@kongo09
Copy link
Author

kongo09 commented Jul 23, 2024

I noticed another weird thing. If I hit the Threads icon because, hey, it has a notification bubble, the right panel closes and does not show Threads, instead of switching over to the Threads section:

image

@t3chguy
Copy link
Member

t3chguy commented Jul 24, 2024

That is as design asked for it

@daniellekirkwood
Copy link
Contributor

we need to figure out the close button. I also 'miss' it and get confused by how it's there sometimes and now there other times. I've connected this to the Epic to work on after first launch and will discuss it with @americanrefugee when back

@dbkr dbkr added the S-Tolerable Low/no impact on users label Jul 29, 2024
@tibroc
Copy link

tibroc commented Jul 31, 2024

To me what @kongo09 mentioned in his comment seems like a very obvious bug and was immediatley confusing to people I talked to. @t3chguy Should this be treated as a separate issue?

@t3chguy
Copy link
Member

t3chguy commented Jul 31, 2024

You can file a separate issue, it won't change how Design/Product investigate/think about it, but might be cleaner.

@daniellekirkwood
Copy link
Contributor

Our suggested solution is to drop the tabs and introduce a close button [x] to the far right of the right panel - we're just gathering feedback on the designs before committing to this solution... What would y'all think to that?

What that might look like 👇
Screenshot 2024-08-13 at 12 26 20

Screenshot 2024-08-13 at 12 26 33

@tibroc as for the buttons opening the right place in the right panel instead of closing the right panel, we'll address that too. We don't need an issue to track it (unless you already made one?)

@opusforlife2
Copy link

Why not keep the tabs and just add an X button? That seems like the most straightforward solution involving minimum effort.

@tibroc
Copy link

tibroc commented Aug 13, 2024

@daniellekirkwood The screenshots you provide seem like a good idea to me. It also removes the duplicated functionality between the buttons and the tabs, so it is already less confusing.

I did not yet open a separte issue, so I'm fine with tracking progress here.

@daniellekirkwood
Copy link
Contributor

We drew up what keeping the tabs and introducing the [x] would look like and it gets really 'busy' as you'd also need a scroll bar to see all the tabs.

Screenshot 2024-08-13 at 13 44 37

To tibroc's point, removing the tabs removes the duplicated functionality

@opusforlife2
Copy link

Oh, right. I didn't notice that the pre-existing buttons still remain along with the tabs. Then yeah, removing the tabs makes sense. They don't serve any purpose.

@Krishcode264
Copy link

Screenshot 2024-09-18 082355
do we still need to add [X] icon to close the right Side panel ?

@daniellekirkwood
Copy link
Contributor

Yes, we will be reintroducing the close button. Thanks for catching that I had not yet updated the content of this issue!

@daniellekirkwood
Copy link
Contributor

Screenshot 2024-09-18 at 08 57 33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Right-Panel O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants