Make a distinction between `active` and `selected` in the issue author dropdown (#30207)
Signed-off-by: Yarden Shoham <git@yardenshoham.com> (cherry picked from commit f8fbaaf26fa7798fde690f4400910069fbccd40e)
This commit is contained in:
parent
c5dc579017
commit
6c3dbc2eee
|
@ -149,7 +149,9 @@ function initRepoIssueListAuthorDropdown() {
|
||||||
$searchDropdown.dropdown('refresh');
|
$searchDropdown.dropdown('refresh');
|
||||||
// defer our selection to the next tick, because dropdown will set the selection item after this `menu` function
|
// defer our selection to the next tick, because dropdown will set the selection item after this `menu` function
|
||||||
setTimeout(() => {
|
setTimeout(() => {
|
||||||
menu.querySelector('.item.active, .item.selected')?.classList.remove('active', 'selected');
|
for (const el of menu.querySelectorAll('.item.active, .item.selected')) {
|
||||||
|
el.classList.remove('active', 'selected');
|
||||||
|
}
|
||||||
menu.querySelector(`.item[data-value="${selectedUserId}"]`)?.classList.add('selected');
|
menu.querySelector(`.item[data-value="${selectedUserId}"]`)?.classList.add('selected');
|
||||||
}, 0);
|
}, 0);
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in New Issue