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

Room Notification settings: Clicking on "settings" link also selects the option #23391

Closed
jaller94 opened this issue Oct 2, 2022 · 4 comments
Labels
A-Notifications A-Room-Settings good first issue Good for newcomers Hacktoberfest Issues which are suitable for Hacktoberfest PRs: https://hacktoberfest.digitalocean.com/ O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-GetYourUpdates

Comments

@jaller94
Copy link
Contributor

jaller94 commented Oct 2, 2022

Steps to reproduce

  1. Open any room's settings.
  2. Go to the tab "Notifications"
  3. Click on a "settings" link of an option that is not currently selected.

Screenshot 2022-10-02 at 18-19-13 Element P2P Matrix

Outcome

What did you expect?

  1. The global notification settings open.

What happened instead?

  1. The option is selected.
  2. Additionally, the global notification settings open.

Operating system

No response

Browser information

Firefox 105.0b4

URL for webapp

app.element.io

Application version

Element version: 1.11.8 Olm version: 3.2.12

Homeserver

matrix.org

Will you send logs?

No

@SimonBrandner SimonBrandner added S-Minor Impairs non-critical functionality or suitable workarounds exist A-Room-Settings A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely labels Oct 3, 2022
@justjanne justjanne added good first issue Good for newcomers Hacktoberfest Issues which are suitable for Hacktoberfest PRs: https://hacktoberfest.digitalocean.com/ labels Oct 4, 2022
@SoldierCorp
Copy link

Hi @jaller94 !

I believe it's working as expected. The option is being selected and the global settings gets open. Although the selected option it's not visible until you re-open the room settings.

Screen.Recording.2022-10-04.at.19.29.29.mov

@jaller94
Copy link
Contributor Author

jaller94 commented Oct 5, 2022

As a user I don't expect a link to trigger a second action.
The link is a component inside the label. Other applications like Firefox, Chromium and VS Code don't toggle checkboxes when you click on links like "Learn more" inside a label.

@SoldierCorp
Copy link

@jaller94 You know what, I read the expected behaviours in the opposite way. Now I know this is wrong and it's supposed to only open Global Notifications without selecting the option.

I would like to work on this one, could you please assign it to me?

@BhuvaneshPatil
Copy link

@jaller94 , can we close this issue as it's resolved in the mentioned PR

@t3chguy t3chguy closed this as completed Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications A-Room-Settings good first issue Good for newcomers Hacktoberfest Issues which are suitable for Hacktoberfest PRs: https://hacktoberfest.digitalocean.com/ O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-GetYourUpdates
Projects
None yet
Development

No branches or pull requests

7 participants