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

Unchecked checkboxes are circular #25715

Closed
robintown opened this issue Jul 4, 2023 · 11 comments · Fixed by matrix-org/matrix-react-sdk#11224
Closed

Unchecked checkboxes are circular #25715

robintown opened this issue Jul 4, 2023 · 11 comments · Fixed by matrix-org/matrix-react-sdk#11224
Labels
A11y 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-Cannot-Reproduce X-Regression X-Release-Blocker

Comments

@robintown
Copy link
Member

Steps to reproduce

  1. Open one of the sort/appearance menus in the room list

Outcome

What did you expect?

Checkboxes are boxes

What happened instead?

Checkboxes are circles, and therefore look like radio buttons

Screenshot from 2023-07-04 13-43-06

Operating system

NixOS unstable

Browser information

Firefox 114.0.1

URL for webapp

develop.element.io

Application version

Element version: 01a234a-react-2a7780052e60-js-5751df1288b3 Olm version: 3.2.14

Homeserver

Not relevant

Will you send logs?

No

@robintown robintown added T-Defect X-Regression S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Jul 4, 2023
@germain-gg
Copy link
Contributor

Me and @andybalaam are not able to reproduce.
Could you check whether this still happens on your side?

@andybalaam
Copy link
Member

I am unable to reproduce this with the latest code, using Firefox 115 on Ubuntu.

@andybalaam
Copy link
Member

Going ahead with the release candidate even though this is still open, since it looks like it got fixed in the meantime.

@t3chguy
Copy link
Member

t3chguy commented Jul 11, 2023

image

@andybalaam I can reproduce on Firefox

Compared with staging

image

@andybalaam
Copy link
Member

OK, I can now reproduce on Firefox at both small and large font sizes:

image

image

@andybalaam
Copy link
Member

(Especially small?)

@andybalaam
Copy link
Member

I actually think the difference between checked and unchecked is a slight red herring, and the main issue is that they are both too round. Checked and unchecked have identical values of height, width and border-radius, so the slightly different rendering appears to be a Firefox quirk which is particularly noticeable when they are quite round.

@andybalaam
Copy link
Member

andybalaam commented Jul 11, 2023

At both small sizes:

image

and large sizes:

image

0.27rem looks similar to how it looked before, on Firefox.

@andybalaam
Copy link
Member

The same problem is visible on Chromium, and the same fix (0.27rem) looks good:

image

@andybalaam
Copy link
Member

So I am going to attempt a fix that just changes this one number.

@andybalaam
Copy link
Member

A quick search of the source code finds no similar values elsewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11y 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-Cannot-Reproduce X-Regression X-Release-Blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants