Skip to content

[Bug]: The clear button plugin uses different unicode characters than the remove button #901

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

Open
FlorianBoe opened this issue Mar 21, 2025 · 1 comment · May be fixed by #902
Open

[Bug]: The clear button plugin uses different unicode characters than the remove button #901

FlorianBoe opened this issue Mar 21, 2025 · 1 comment · May be fixed by #902
Labels
bug Something isn't working

Comments

@FlorianBoe
Copy link

Bug description

I know it's probably not something that many people will notice, but in relevant cases it leads to unsightly visual results.

The clear button plugin uses the Unicode character U+2A2F (VECTOR OR CROSS PRODUCT) as the close icon, while U+00D7 (MULTIPLICATION SIGN) is used to deselected items.

Here are the two unicode characters:
⨯ ×

The difference between these two characters is the vertical alignment and that U+2A2F is rarely part of a font file. In these cases, a fallback font is used (in my case Apple Symbols), which then also differs from the original font in terms of stroke width.

I would suggest replacing ⨯ with × in this line.

Expected behavior

⨯ sould be ×

Steps to reproduce

Take a look at this line

Additional context

No context needed

@FlorianBoe FlorianBoe added the bug Something isn't working label Mar 21, 2025
@FlorianBoe FlorianBoe linked a pull request Mar 21, 2025 that will close this issue
@bazineta
Copy link

The fix in the PR solves the issue for us. On Macs in particular, the cross-product symbol used is indeed quite tiny, and the standard times is a better size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants