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

My historical section is missing #6941

Closed
turt2live opened this issue Jun 26, 2018 · 13 comments · Fixed by matrix-org/matrix-react-sdk#2016
Closed

My historical section is missing #6941

turt2live opened this issue Jun 26, 2018 · 13 comments · Fixed by matrix-org/matrix-react-sdk#2016
Labels
P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@turt2live
Copy link
Member

turt2live commented Jun 26, 2018

Description

image

Version information

  • Platform: web (in-browser)
  • Browser: Chrome 67
  • OS: Windows 10
  • URL: riot.im/develop
@lampholder
Copy link
Member

I'm not sure what we want the behaviour to be exactly - I appreciate the clarity of empty headers being invisible, especially when I filter by community, but at the moment we allow users to change a room's dm/room favourite/low-priority status by dragging and that's only possible with a drag target.

@lampholder lampholder added P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Jun 26, 2018
@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

@lampholder those won't vanish because the droptarget means the roomsublist is not empty, but the Historical roomsublist had no text/anything when it was empty, only a Spinner when you clicked on it which yielded either rooms or empty space

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

oh I see what you mean, they can vanish when querying by searchFilter/TagPanel, so its a different break than Historical

@turt2live
Copy link
Member Author

In the screenshot there's no filtering whatsoever going on. The section used to be there, but it disappeared some time yesterday.

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

After discussion, this behaviour is going behind an opt-in flag as well as fixing Historical when it is enabled

@turt2live
Copy link
Member Author

Is this actually a feature? I'm a bit concerned that the actual bug is lost in translation.

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

No sorry probably my bad explanation.
The roomsublist auto-hide behaviour will be opt-in,
Your bug is about the roomsublist-auto-hide behaviour, the PR above fixes the bug
but the behaviour as a whole will still be opt-in

@turt2live
Copy link
Member Author

turt2live commented Jun 26, 2018

There is no PR above. Way above, got it.

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

Faaaaar above

image

@turt2live
Copy link
Member Author

When was this introduced? It seemed like an unintended bug which is why I reported it. I don't actually mind the behaviour.

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

@t3chguy
Copy link
Member

t3chguy commented Jun 26, 2018

it is unintended

@t3chguy t3chguy closed this as completed Jun 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants