-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 is rising an error when it's inside a QMenu and you double click on it #10028
Comments
@kevingermain I am not seeing any issues here. Can you try in incognito mode? |
@hawkeye64 I tried with all disabled extensions, with brave (without the shield) and another chrome from another computer, still the issue (the issue is not showing with firefox). Try to double click a lot on the same spot. |
After 10 minutes od relentles clicking, I've managed to reproduce the issue:
Repro video link. |
I am still unable to reproduce it. But, I have an idea what is happening. The mouse event queue is getting backed up and by the time you click outside, the panel has been destroyed, but mouse events are still coming in and being processed. function changeSpectrum (left, top, change) {
const panel = spectrumRef.value
if (panel === void 0) { return }
const
width = panel.clientWidth, // < -- here
height = panel.clientHeight,
rect = panel.getBoundingClientRect() |
Fix will be available in Quasar v2.0.3 |
The error disappear in the console but the problem still here. Can someone confirm please. |
If QColor is inside a QMenu and the user double click (by mistake for example) QColor rise an error
Uncaught TypeError: Cannot read property 'clientWidth' of null
and it's closing the menu even it's not the desired behavior.Can be simply tested here: https://quasar.dev/vue-components/color-picker#with-qinput
Open the console first, try to double click, and see the error and the QMenu closing unexpectedly.
(it works if the screen is small enough (mobile mode for example))
The text was updated successfully, but these errors were encountered: