diff --git a/chrome/content/zotero/elements/itemPaneSidenav.js b/chrome/content/zotero/elements/itemPaneSidenav.js index 582185e3cd..3175e50f34 100644 --- a/chrome/content/zotero/elements/itemPaneSidenav.js +++ b/chrome/content/zotero/elements/itemPaneSidenav.js @@ -28,48 +28,52 @@ { class ItemPaneSidenav extends XULElementBase { content = MozXULElement.parseXULToFragment(` - + - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + - + + @@ -134,6 +138,7 @@ get _contextNotesPaneVisible() { return this._contextNotesPane + && !this._collapsed && this._contextNotesPane.parentElement.selectedPanel == this._contextNotesPane; } @@ -339,19 +344,19 @@ } render() { + let contextNotesPaneVisible = this._contextNotesPaneVisible; let pinnedPane = this.pinnedPane; for (let toolbarbutton of this.querySelectorAll('toolbarbutton')) { let pane = toolbarbutton.dataset.pane; if (pane == 'context-notes') { let hidden = !this._contextNotesPane; - let selected = this._contextNotesPaneVisible; + let selected = contextNotesPaneVisible; toolbarbutton.parentElement.hidden = hidden; toolbarbutton.parentElement.previousElementSibling.hidden = hidden; // Divider toolbarbutton.setAttribute('aria-selected', selected); - toolbarbutton.classList.toggle('active', selected); continue; } @@ -367,13 +372,16 @@ continue; } - toolbarbutton.setAttribute('aria-selected', - !this._contextNotesPaneVisible && pane == pinnedPane); + toolbarbutton.setAttribute('aria-selected', !contextNotesPaneVisible && pane == pinnedPane); toolbarbutton.parentElement.hidden = !this.getPane(pane); // Set .pinned on the container, for pin styling toolbarbutton.parentElement.classList.toggle('pinned', pane == pinnedPane); } + + this.querySelector('.highlight-notes-active').classList.toggle('highlight', contextNotesPaneVisible); + this.querySelector('.highlight-notes-inactive').classList.toggle('highlight', + this._contextNotesPane && !contextNotesPaneVisible); } } customElements.define("item-pane-sidenav", ItemPaneSidenav); diff --git a/scss/elements/_itemPaneSidenav.scss b/scss/elements/_itemPaneSidenav.scss index cea2c339ad..5e271c7be1 100644 --- a/scss/elements/_itemPaneSidenav.scss +++ b/scss/elements/_itemPaneSidenav.scss @@ -21,7 +21,21 @@ item-pane-sidenav { padding-block: 4px; } - .toolbarbutton-container { + .inherit-flex { + display: flex; + flex-direction: inherit; + flex-wrap: inherit; + align-items: inherit; + justify-content: inherit; + gap: inherit; + } + + .highlight { + border-radius: 5px; + background-color: var(--fill-quinary); + } + + .pin-wrapper { position: relative; // Disable pointer events here, re-enable on the button, so that :hover styles are only applied // when the button itself is hovered @@ -64,7 +78,7 @@ item-pane-sidenav { background-color: var(--fill-quinary); } - &:active, &.active { + &:active { background-color: var(--fill-quarternary); }