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

Pile of space member avatars is no longer clickable #26164

Closed
robintown opened this issue Sep 13, 2023 · 4 comments · Fixed by matrix-org/matrix-react-sdk#11657
Closed

Pile of space member avatars is no longer clickable #26164

robintown opened this issue Sep 13, 2023 · 4 comments · Fixed by matrix-org/matrix-react-sdk#11657
Assignees
Labels
A-Avatar A-Spaces Spaces, groups, communities O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression

Comments

@robintown
Copy link
Member

robintown commented Sep 13, 2023

Steps to reproduce

  1. Go to the home page of a space
  2. Click the pile of space member avatars (the ones at the bottom of this screenshot: )
    Screenshot 2023-09-13 at 14-33-44 Element 3 Matrix Community

Outcome

What did you expect?

The member list for the space should open (this is what it did until recently)

What happened instead?

Nothing happens. I thought that I might be able to work around this by using the Ctrl + . shortcut to open the right panel and show the member list, but that doesn't work either. What does still work is clicking the "3277 members" text, which is apparently (surprisingly!) a button

Operating system

NixOS unstable

Browser information

Firefox 117.0

URL for webapp

develop.element.io

Application version

Element version: 361941e-react-da1daa1b18ff-js-6d118008b6b3 Olm version: 3.2.14

Homeserver

Not relevant

Will you send logs?

No

@robintown robintown added T-Defect X-Regression S-Major Severely degrades major functionality or product features, with no satisfactory workaround A-Spaces Spaces, groups, communities A-Avatar O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Sep 13, 2023
@robintown robintown changed the title Can no longer open the space member list Pile of space member avatars is no longer clickable Sep 13, 2023
@germain-gg germain-gg self-assigned this Sep 21, 2023
@manancodes
Copy link

manancodes commented Sep 24, 2023

@germain-gg I have opened Pull Requests for both the solutions to this problem. Please have a look and let me know if any enhancements are needed. Thanks.

These took time to find the root cause, but it was a fun-to-solve problem.

@germain-gg germain-gg assigned manancodes and unassigned germain-gg Sep 25, 2023
@manancodes
Copy link

manancodes commented Sep 25, 2023

I'm not sure I understand why there's two PRs? matrix-org/matrix-react-sdk#11656 seems to already fix the problem?

Moreover the solution here with onClick on a div element is not accessible thus requiring more work. as well as some tests to avoid regressions on this in the future.

@germain-gg The output of these 2 PRs is different.

  1. Fix: Clicking on members pile does nothing (Alternate fix) matrix-org/matrix-react-sdk#11656 - If I click on a user icon, the user profile opens up. If I click the more(...) button, the members list opens.(Explained in detail with recordings in the PR itself)

This solution makes more sense to me, but the problem is, hovering over this pile shows a tooltip which says "See all 3357 Members" which makes it confusing and is a bad user experience because only the more(...) button will open the list.
Screenshot 2023-09-25 at 2 42 49 PM

  1. Fix: Clicking on members pile does nothing. matrix-org/matrix-react-sdk#11657 - Clicking anywhere on the pile will open the members list. Which is more aligned with the tooltip and the previous implementation before the regression. But again, it is not a complete solution.

Which one should I go ahead with?

Meanwhile I'll research about best workarounds for onClick on non-accessible elements, any assistance or ideas will surely help.

@manancodes
Copy link

@robintown @germain-gg Please assist what should I go ahead with?

@germain-gg
Copy link
Contributor

@manancodes matrix-org/matrix-react-sdk#11657 is the desired output, as the tooltip label suggests that you will see all members.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Avatar A-Spaces Spaces, groups, communities O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression
Projects
None yet
3 participants