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

Rooms list doesn't show grandchild rooms as being in a space if you are not a member of their immediate parent subspace #17293

Open
williamkray opened this issue May 13, 2021 · 24 comments
Labels
A-Spaces Spaces, groups, communities A-Subspaces Spaces which are inside other spaces O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Spec-Changes

Comments

@williamkray
Copy link

Screenshot_20210513102222

Description

Joining a new community (#community:matrix.org) which has many rooms I am already a member of, most of those rooms did not render properly in the Spaces panel. Exploring rooms shows that I am indeed in the room by showing me a "View" button instead of "Join". However, clicking "View" takes me back to the Home (All Spaces) space to show me the room.

Logs being sent: yes

Version information

  • Platform: web

For the web app:

  • Browser: Firefox
  • OS: Arch Linux
  • URL: develop.element.io
@SimonBrandner SimonBrandner added A-Spaces Spaces, groups, communities S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels May 13, 2021
@SimonBrandner
Copy link
Contributor

SimonBrandner commented May 13, 2021

Can repro on desktop. It doesn't get fixed after a restart

@SimonBrandner
Copy link
Contributor

Clearing cache doesn't seem to fix this either

@SimonBrandner
Copy link
Contributor

Ok, I don't think I can repro this with other spaces... Can someone please confirm this? Could the space be broken?

@t3chguy
Copy link
Member

t3chguy commented May 14, 2021

So the issue above is because that room is in a subspace and you haven't joined the subspace.
This is a known limitation until we have the /peek API to watch those not-yet-joined subspaces.

@t3chguy t3chguy changed the title Joining space with rooms I'm already a member of doesn't show them in the Spaces panel properly Being in a space but not its subspaces doesn't show grandchild rooms May 14, 2021
@t3chguy t3chguy added X-Spec-Changes and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels May 14, 2021
@williamkray
Copy link
Author

that... makes complete sense. i didn't put that together despite having already experienced that with the testing space.

@toger5
Copy link
Contributor

toger5 commented May 20, 2021

@t3chguy This makes sense.
When peek is available, will that be the default behavior? showing grandchild?
Or is would that be optional?
Imo showing it be default is what I would have expected.

@t3chguy
Copy link
Member

t3chguy commented May 20, 2021

Product/Design hasn't investigated this yet to my knowledge so no idea.

@kittykat kittykat added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Aug 17, 2021
@clokep
Copy link

clokep commented Aug 23, 2021

So the issue above is because that room is in a subspace and you haven't joined the subspace.
This is a known limitation until we have the /peek API to watch those not-yet-joined subspaces.

I think that was a potential solution, but I think the root cause was that the spaces summary previously did not recurse into rooms which had join rules of public (matrix-org/synapse#10298) as well as some other bugs if you had a pending invite to a space, etc.

I think this should be fixed as of Synapse 1.39.0.

Note that the current spaces summary for #community:matrix.org still might not be 100% accurate until pagination is complete. Server-side support for this was done (and is in Synapse 1.41.0), but Element Web needs an update for it still.

@novocaine
Copy link
Contributor

@williamkray can you check if things have improved for you please?

@novocaine novocaine added X-Needs-Info This issue is blocked awaiting information from the reporter and removed X-Spec-Changes labels Aug 23, 2021
@RiotRobot RiotRobot added this to Priority bugs in Web App Team Aug 23, 2021
@williamkray
Copy link
Author

yes, i'm seeing lots more rooms in the subspaces!

Web App Team automation moved this from Priority bugs to In Test Aug 23, 2021
@clokep
Copy link

clokep commented Aug 23, 2021

Looking at this again, is this referring to the main "Explore spaces" view, or did you mean where rooms are shown on the left-sidebar? The Synapse PRs I pointed to are specifically for the explore spaces view, I'm unsure about how Element Web handles the left-sidebar.

@williamkray
Copy link
Author

@clokep you are correct, i lost track of why i originally opened this ticket in the first place. thank you for reminding me.

i have left the "bots" subspace of the matrix community, and the left panel still does not show me those rooms when i have selected the matrix community space. instead, those rooms are only visible when i visit the "home" metaspace.

i would expect that because these rooms are grandchildren of the matrix community space, they would still show up under that space, regardless of whether i am a member of their requisite sub-space.

@clokep clokep reopened this Aug 23, 2021
@clokep clokep changed the title Being in a space but not its subspaces doesn't show grandchild rooms Being in a space but not its subspaces doesn't show grandchild rooms in the rooms list Aug 23, 2021
@clokep
Copy link

clokep commented Aug 23, 2021

Thanks! I've reopened this then (and tried to clarify the title a bit).

@novocaine novocaine removed the X-Needs-Info This issue is blocked awaiting information from the reporter label Sep 1, 2021
@novocaine novocaine changed the title Being in a space but not its subspaces doesn't show grandchild rooms in the rooms list Rooms list doesn't show grandchild rooms as being in a space if you are not a member of their immediate parent subspace Sep 1, 2021
@novocaine
Copy link
Contributor

novocaine commented Sep 1, 2021

This is an annoying limitation, but downgrading severity to S-minor as this doesn't severely degrade major functionality and the workaround is to access the rooms via Home

@novocaine novocaine added 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 1, 2021
@novocaine
Copy link
Contributor

novocaine commented Sep 3, 2021

@kittykat convinced me that this is S-Major as she's mysteriously lost track of rooms without understanding why; so most users wouldn't even know they were missing out

@novocaine novocaine added S-Major Severely degrades major functionality or product features, with no satisfactory workaround and removed S-Minor Impairs non-critical functionality or suitable workarounds exist labels Sep 3, 2021
@t3chguy

This comment has been minimized.

@novocaine

This comment has been minimized.

@t3chguy

This comment has been minimized.

@edrex

This comment has been minimized.

@kittykat

This comment has been minimized.

@novocaine
Copy link
Contributor

Possible workaround - auto-join parent sub-spaces on joining a room.

@afranke
Copy link
Contributor

afranke commented Sep 27, 2021

FWIW I don’t join subspaces on purpose. Spaces replace communities, and so I associate them with the same semantics. Just because I join a room that is part of a (sub)space/community doesn’t mean I feel like I am part of that space/community. Or to put it differently, I wouldn’t necessarily want to seen as a member of that group. The workaround of joining subspaces (automatically or not) doesn’t really solve the issue for me.

@novocaine
Copy link
Contributor

Well .. it does solve the issue, right? You just don't want to be auto-joined to a subspace, which is fair enough.

@novocaine novocaine added O-Occasional Affects or can be seen by some users regularly or most users rarely and removed O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Apr 12, 2022
@dbkr
Copy link
Member

dbkr commented Feb 23, 2023

Another aspect of this is that sometimes the grandchild space just appears in your space panel for no reason I can see. Needless to say, having other spaces just show up in your space bar is somewhat disconcerting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Spaces Spaces, groups, communities A-Subspaces Spaces which are inside other spaces O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Spec-Changes
Projects
None yet
Development

No branches or pull requests