corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1801297 - Better align urlbar result menu button styling with the UX spec. r=jteow

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D167072
This commit is contained in:
Dão Gottwald
2023-01-19 19:10:05 +00:00
parent 8f5d8c9d06
commit 0af8b03623
3 changed files with 35 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
browser/components/urlbar/UrlbarView.sys.mjs
View File

@@ -1962,15 +1962,18 @@ export class UrlbarView {
if (this.#selectedElement) {
this.#selectedElement.toggleAttribute("selected", false);
this.#selectedElement.removeAttribute("aria-selected");
this.#getSelectedRow()?.toggleAttribute("selected", false);
}
let row = this.#getRowFromElement(element);
if (element) {
element.toggleAttribute("selected", true);
element.setAttribute("aria-selected", "true");
row?.toggleAttribute("selected", true);
}
this.#setAccessibleFocus(setAccessibleFocus && element);
this.#selectedElement = element;
let result = this.#getRowFromElement(element)?.result;
let result = row?.result;
if (updateInput) {
let urlOverride = null;
if (element?.classList?.contains("urlbarView-button-help")) {