Bug 1550423 - When a result's title or URL overflows, expose the full value in a tooltip. r=mak

Differential Revision: https://phabricator.services.mozilla.com/D30470

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Dão Gottwald 2019-05-09 13:06:53 +00:00
Родитель ef01932b50
Коммит 24c3de356b
2 изменённых файлов: 12 добавлений и 0 удалений

Просмотреть файл

@ -60,6 +60,7 @@ class UrlbarInput {
MozXULElement.parseXULToFragment(`
<panel id="urlbar-results"
role="group"
tooltip="aHTMLTooltip"
noautofocus="true"
hidden="true"
flip="none"

Просмотреть файл

@ -571,6 +571,10 @@ class UrlbarView {
let title = item._elements.get("title");
this._addTextContentWithHighlights(
title, result.title, result.titleHighlights);
title._tooltip = result.title;
if (title.hasAttribute("overflow")) {
title.setAttribute("title", title._tooltip);
}
let tagsContainer = item._elements.get("tagsContainer");
tagsContainer.textContent = "";
@ -619,8 +623,13 @@ class UrlbarView {
if (setURL) {
this._addTextContentWithHighlights(url, result.payload.displayUrl,
result.payloadHighlights.displayUrl || []);
url._tooltip = result.payload.displayUrl;
} else {
url.textContent = "";
url._tooltip = "";
}
if (url.hasAttribute("overflow")) {
url.setAttribute("title", url._tooltip);
}
if (isVisitAction) {
@ -820,6 +829,7 @@ class UrlbarView {
(event.target.classList.contains("urlbarView-url") ||
event.target.classList.contains("urlbarView-title"))) {
event.target.toggleAttribute("overflow", true);
event.target.setAttribute("title", event.target._tooltip);
}
}
@ -828,6 +838,7 @@ class UrlbarView {
(event.target.classList.contains("urlbarView-url") ||
event.target.classList.contains("urlbarView-title"))) {
event.target.toggleAttribute("overflow", false);
event.target.removeAttribute("title");
}
}