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

QColor spectrum view is broken in RTL #17269

Open
tamirvs opened this issue Jun 9, 2024 · 0 comments
Open

QColor spectrum view is broken in RTL #17269

tamirvs opened this issue Jun 9, 2024 · 0 comments
Labels
area/components bug/2-confirmed We have reproduce the problem and confirmed that this is a bug. flavour/quasar-cli-vite kind/bug 🐞 Qv2 🔝 Quasar v2 issues

Comments

@tamirvs
Copy link

tamirvs commented Jun 9, 2024

What happened?

Bug

When using QColor with a $q.lang which is RTL, the color spectrum stays the same, but the calculation of the color is transformed.

Suggestion

As an RTL user, I don't think the spectrum should be reversed for RTL (in Word it's the same for LTR/RTL).
To avoid BC, maybe add an option to disable RTL for this component, through dir="ltr" or a specific prop.

What did you expect to happen?

When selecting a color on the spectrum, show the correct color.

Reproduction URL

https://codepen.io/tamirvs/pen/LYojXbj

How to reproduce?

  1. Select a color
  2. The selected color is the opposite on the X-axis

Flavour

Quasar CLI with Vite (@quasar/cli | @quasar/app-vite)

Areas

Components (quasar)

Platforms/Browsers

No response

Quasar info output

No response

Relevant log output

No response

Additional context

No response

@tamirvs tamirvs added kind/bug 🐞 Qv2 🔝 Quasar v2 issues labels Jun 9, 2024
@github-actions github-actions bot added area/components bug/1-repro-available A reproduction is available and needs to be confirmed. flavour/quasar-cli-vite labels Jun 9, 2024
@yusufkandemir yusufkandemir changed the title QColor RTL mode is messed up QColor spectrum view is broken in RTL Jun 26, 2024
@yusufkandemir yusufkandemir added bug/2-confirmed We have reproduce the problem and confirmed that this is a bug. and removed bug/1-repro-available A reproduction is available and needs to be confirmed. labels Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/components bug/2-confirmed We have reproduce the problem and confirmed that this is a bug. flavour/quasar-cli-vite kind/bug 🐞 Qv2 🔝 Quasar v2 issues
Projects
None yet
Development

No branches or pull requests

2 participants