Skip to content

Commit 77e6ab6

Browse files
authored
fix(ui5-multi-combobox): fix initial focus on mobile (#1508)
1 parent 71a25ad commit 77e6ab6

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

packages/main/src/MultiComboBox.js

+9
Original file line numberDiff line numberDiff line change
@@ -485,6 +485,15 @@ class MultiComboBox extends UI5Element {
485485
}
486486
}
487487

488+
_setInitialFocusInResponsivePopover() {
489+
this._innerInput.focus();
490+
}
491+
492+
_onAllItemsPopoverAfterOpen() {
493+
this._setInitialFocusInResponsivePopover();
494+
this._toggleIcon();
495+
}
496+
488497
_getSelectedItems() {
489498
// Angular 2 way data binding
490499
this.selectedValues = this.items.filter(item => item.selected);

packages/main/src/MultiComboBoxPopover.hbs

+2-1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
_disable-initial-focus
88
content-only-on-desktop
99
@ui5-afterClose={{_afterClosePopover}}
10+
@ui5-afterOpen={{_setInitialFocusInResponsivePopover}}
1011
>
1112
<div slot="header" class="ui5-responsive-popover-header">
1213
<div class="row">
@@ -75,7 +76,7 @@
7576
content-only-on-desktop
7677
@ui5-selectionChange={{_listSelectionChange}}
7778
@ui5-afterClose={{_toggleIcon}}
78-
@ui5-afterOpen={{_toggleIcon}}
79+
@ui5-afterOpen={{_onAllItemsPopoverAfterOpen}}
7980
>
8081
<div slot="header" class="ui5-responsive-popover-header">
8182
<div class="row">

0 commit comments

Comments
 (0)