-
Notifications
You must be signed in to change notification settings - Fork 82
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
refactor: use new mixins, simplify keyboard handling logic #2976
Conversation
e64a6bd
to
562fd7d
Compare
@@ -214,7 +214,6 @@ describe('edit column editor type', () => { | |||
grid.enterNextRow = true; | |||
const item = editor._overlayElement.querySelector('vaadin-item'); | |||
enter(item); | |||
item.click(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't necessary because click
is called on Enter anyways.
506f883
to
df4c012
Compare
b1c4ed6
to
48f2066
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This ticket/PR has been released with platform 22.0.0.beta2 and is also targeting the upcoming stable 22.0.0 version. |
Description
ActiveMixin
andFocusMixin
so that we handle these consistently for all components.focus-ring
when focusing an item inside a list-boxfocus-ring
was always set unless there was amousedown
event beforefocusin
,focus-ring
is set only if there was akeydown
(same as in button, input fields etc),focus-ring
set byvaadin-select
,vaadin-list-box
,vaadin-menu-bar
.Type of change