-
Notifications
You must be signed in to change notification settings - Fork 80
Tooltip ESC key press propagation #6292
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
Comments
Installed and assigned for verification. |
Verified with https://codepen.io/anastasiia-zrh/pen/bGjrZYX in Does not include the https://codepen.io/anastasiia-zrh/pen/MWBvZdB sample. @jcfranco, should we open a new issue to close a contextual element (with the Esc key), following visual order? |
Thank you for the update! Recently, I've checked the link:
And realized that the behavior has changed (scrolls appear when tooltip is visible), please see the recordings below: Safari Version 16.0 (17614.1.25.9.10, 17614) Ther similar behavior is in v1.0.3 https://codepen.io/anastasiia-zrh/pen/RwBzVeJ?editors=100: Please, let me know if any additional info is needed. |
@Anastasiia-Boleiko Ah, yes, forgot to mention I added the The attribute is needed to display when used within parent components, to ensure proper placement. For instance: <calcite-tooltip label="Example label" overlay-positioning="fixed" reference-element="tooltip-button3" close-on-click> |
@geospatialem Thank you very much! Now the tooltip has a proper placement👍 Could you please suggest if you can reproduce the same behavior:
I've reproduced it in v1.0.3: https://codepen.io/anastasiia-zrh/pen/JjBQrER |
Yes, it would be present in 1.0.3 - the fix was mitigated in last night's patch, |
Oh, got it! I do apologize for the misunderstanding. |
No worries! Was waiting to close this out to address the nice to have - just added a new issue and CC'ed you on it to follow along the progress. Thanks for the great feedback to continuously improve CC! |
Summary
Calcite tooltip is dismissable that complies with Success Criterion 1.4.13 Content on Hover or Focus (Level AA)
But when pressing the 'Esc' key in an opened calcite-tooltip, that is located in a calcite-modal, the whole modal is closed.
The expected behavior is to only close the tooltip and close the Dialog on the next 'ESC' key press.
Generally speaking, it would be nice to close a context element at the moment (with the Esc key).
Actual Behavior
After pressing 'Esc' key to close a tooltip, the whole dialog gets closed.
Expected Behavior
After pressing 'Esc' key - close the tooltip, and close the Dialog on the next 'ESC' key press.
Reproduction Sample
https://codepen.io/anastasiia-zrh/pen/bGjrZYX
Reproduction Steps
One more example, where it would be nice to close a contextual element (with the Esc key), following visual order:
https://codepen.io/anastasiia-zrh/pen/MWBvZdB
Reproduction Version
1.0.0-beta.97
Working W3C Example/Tutorial
No response
Relevant Info
A similar issue was found in our product by LevelAccess (so we tried calcite components first and found a similar issue):
Ensure that content that appears on hover or focus may be dismissed by the user
Violation:
Ensure that content that appears on hover or focus may be dismissed by the user.
Description:
Examples include:
Tooltips
In this case, pressing the Esc key does close the tooltip, but it also closes the entire dialog.
User Impact
Users may press the Esc key to close the tooltip and may unwittingly and unexpectedly close the entire dialog.
Regression?
No response
Esri team
ArcGIS Urban
The text was updated successfully, but these errors were encountered: