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

Binary settings choice with inconsistent interaction pattern #27193

Open
kongo09 opened this issue Mar 14, 2024 · 6 comments
Open

Binary settings choice with inconsistent interaction pattern #27193

kongo09 opened this issue Mar 14, 2024 · 6 comments
Labels
A-User-Settings 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-Needs-Design

Comments

@kongo09
Copy link

kongo09 commented Mar 14, 2024

Steps to reproduce

  1. Go to settings
  2. Observe a binary setting choice

Outcome

What did you expect?

Always the same pattern, i.e. either a toggle or a checkbox

What happened instead?

Sometimes toggles, sometimes checkboxes

image

image

Operating system

Windows 11

Application version

Element version: 1.11.61 Crypto version: Olm 3.2.15

How did you install the app?

No response

Homeserver

element.io

Will you send logs?

No

@dbkr dbkr added X-Needs-Design S-Minor Impairs non-critical functionality or suitable workarounds exist A-User-Settings O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Mar 18, 2024
@robintown
Copy link
Member

robintown commented Mar 19, 2024

This is another issue that would most likely be addressed by the redesign of settings described in #27180 (comment)

@daniellekirkwood
Copy link
Contributor

Agree with @robintown I've added this item to the list WRT the Settings update project.

@daniellekirkwood
Copy link
Contributor

@kongo09 I suggest that we close this issue as it will be handled with the Settings redesign.

If you disagree, feel free to reopen this task.

@kongo09
Copy link
Author

kongo09 commented Apr 29, 2024

Settings redesign currently still has two patterns for the same binary choice, e.g. in notifications.

@kongo09 kongo09 reopened this Apr 29, 2024
@gaelledel
Copy link

Indeed this should be redesigned as part of the settings work we're doing. There might be a reasoning to explain why we use different components. @americanrefugee I'll leave you to review this if needed.

@americanrefugee
Copy link

There are only a few places we're using checkboxes:

Otherwise all binary options are indeed a toggle. The only actual exception being the checkbox for "Show all rooms", which was so that primary option and the modifier felt more connected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-User-Settings 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-Needs-Design
Projects
None yet
Development

No branches or pull requests

6 participants