@vaadin/field-base
Advanced tools
Comparing version 23.0.1 to 23.0.2
{ | ||
"name": "@vaadin/field-base", | ||
"version": "23.0.1", | ||
"version": "23.0.2", | ||
"publishConfig": { | ||
@@ -35,3 +35,3 @@ "access": "public" | ||
"@polymer/polymer": "^3.0.0", | ||
"@vaadin/component-base": "^23.0.1", | ||
"@vaadin/component-base": "^23.0.2", | ||
"lit": "^2.0.0" | ||
@@ -44,3 +44,3 @@ }, | ||
}, | ||
"gitHead": "05fcc6daa21325753cf528c9c1072ccd8dd1e52e" | ||
"gitHead": "a50f708dc0fe7d0cc10763f413838d71b1a0788b" | ||
} |
@@ -34,3 +34,3 @@ /** | ||
* Override an event listener from `KeyboardMixin` | ||
* to prevent setting `focused` on Shift Tab. | ||
* to prevent focusing the host element on Shift Tab. | ||
* @param {KeyboardEvent} event | ||
@@ -43,9 +43,6 @@ * @protected | ||
// When focus moves with Shift + Tab, do not mark host as focused. | ||
// The flag set here will be later used in focusin event listener. | ||
// When focus moves with Shift + Tab, skip focusing the host element | ||
// by focusing it before the default browser focus handling runs | ||
if (!event.defaultPrevented && event.keyCode === 9 && event.shiftKey) { | ||
this._isShiftTabbing = true; | ||
HTMLElement.prototype.focus.apply(this); | ||
this._setFocused(false); | ||
setTimeout(() => (this._isShiftTabbing = false), 0); | ||
} | ||
@@ -66,9 +63,9 @@ } | ||
// When focus moves from outside and not with Shift + Tab, delegate it to focusElement. | ||
if (path[0] === this && !this.contains(event.relatedTarget) && !this._isShiftTabbing) { | ||
// When focus moves to the host element itself, then delegate it to the focusElement | ||
// This should only move focus when using keyboard navigation, for clicks we don't want to interfere, | ||
// for example when the user tries to select some text | ||
if (path[0] === this && this._keyboardActive) { | ||
this.focusElement.focus(); | ||
return true; | ||
} | ||
if (path.includes(this.focusElement)) { | ||
if (path[0] === this || path.includes(this.focusElement)) { | ||
return true; | ||
@@ -75,0 +72,0 @@ } |
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
114282
3170