fx-compat: Fix sync button tooltip

XULTooltipElement now prepends a <description> node, so the nodes we were
getting were off by one. Just use class names for all the children.
This commit is contained in:
Abe Jellinek 2022-09-07 11:15:36 -04:00
parent 9f9dde511e
commit df9d8d593c
2 changed files with 5 additions and 5 deletions

View file

@ -1601,12 +1601,12 @@ Zotero.Sync.Runner_Module = function (options = {}) {
*
* If no label passed, unregister current label
*
* @param {Tooltip} [tooltip]
* @param {π} [tooltip]
*/
this.registerSyncStatus = function (tooltip) {
if (tooltip) {
_currentSyncStatusLabel = tooltip.firstChild.nextSibling;
_currentLastSyncLabel = tooltip.firstChild.nextSibling.nextSibling;
_currentSyncStatusLabel = tooltip.querySelector('.sync-button-tooltip-status');
_currentLastSyncLabel = tooltip.querySelector('.sync-button-tooltip-last-sync');
_currentTooltipMessages = tooltip.querySelector('.sync-button-tooltip-messages');
}
else {

View file

@ -848,8 +848,8 @@
onpopuphiding="Zotero.Sync.Runner.registerSyncStatus()"
noautohide="true">
<label id="zotero-tb-sync-label"/>
<label id="zotero-tb-sync-status" hidden="true"/>
<label id="zotero-tb-sync-last-sync"/>
<label id="zotero-tb-sync-status" class="sync-button-tooltip-status" hidden="true"/>
<label id="zotero-tb-sync-last-sync" class="sync-button-tooltip-last-sync"/>
<div xmlns="http://www.w3.org/1999/xhtml" class="sync-button-tooltip-messages"/>
</tooltip>
</toolbarbutton>