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.

Sign up for GitHub

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

Jump to bottom

Settings dialog remains after signing out #28111

Open
AndrewFerr opened this issue Sep 27, 2024 · 2 comments
Open

Settings dialog remains after signing out #28111

AndrewFerr opened this issue Sep 27, 2024 · 2 comments
Labels
A-Logout Logout, sign out, etc. 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

Comments

@AndrewFerr
Copy link
Member

Steps to reproduce

  1. Go to Settings -> Account
  2. Click "Sign out"

Outcome

What did you expect?

I should have arrived at the login page.

What happened instead?

I arrived there, but the Settings dialog remained shown.

image

Operating system

Fedora Linux 40 (Workstation Edition)

Browser information

Firefox 130.0.1 (64-bit)

URL for webapp

https://develop.element.io/

Application version

Element version: 1b31e7f-react-81192f6989cb-js-825d85f18dfd Crypto version: Rust SDK 0.7.2 (7a21514), Vodozemac 0.7.0

Homeserver

1.115.0rc2

Will you send logs?

No
@dosubot dosubot bot added A-Logout Logout, sign out, etc. 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 labels Sep 27, 2024
@dbkr
Copy link
Member

dbkr commented Oct 3, 2024

element-hq/matrix-react-sdk@5308c91 is the code that is supposed to be doing this. I tested and it just seems like it doesn't work all the time. I don't know why though.

@dbkr
Copy link
Member

dbkr commented Oct 10, 2024

Actually I don't think this is a regression, it's just probably always been a bit flakey.

@dbkr dbkr removed the X-Regression label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Logout Logout, sign out, etc. 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
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dbkr @AndrewFerr