Closed Bug 2072965 Opened 12 days ago Closed 11 days ago

Sidebar header text for extensions is not shown when `Open tools from sidebar` is unchecked

Categories

(Firefox :: Sidebar, defect, P1)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
158 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr140 --- unaffected
firefox-esr153 --- unaffected
firefox156 --- wontfix
firefox157 --- wontfix
firefox158 --- fixed

People

(Reporter: Fanolian+BMO, Assigned: kcochrane)

References

(Regression)

Details

(Keywords: nightly-community, regression, reproducible, Whiteboard: [fidefe-sidebar])

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:158.0) Gecko/20100101 Firefox/158.0
Build ID: 20260916202159

Extension name is not shown in the sidebar header when Open tools from sidebar, introduced in bug 2047653, is unchecked.

Steps to reproduce

  1. In a new profile, install an extension that utilise sidebar, e.g.
    https://addons.mozilla.org/en-GB/firefox/addon/tree-style-tab/
    https://addons.mozilla.org/en-GB/firefox/addon/contextsearch-web-ext/
    https://addons.mozilla.org/en-GB/firefox/addon/firemonkey/
  2. Enable sidebar.
  3. Uncheck Open tools from sidebar. (Or set sidebar.visibility to hide-launcher in about:config.)
  4. Click the sidebar header and select an installed extension.
  5. Observe the header text.

Actual result

The extension name is not shown. It is available in the panel switcher (dropdown menu) though.
Please refer to the attached screenshot.

Expected result

The header shows the name of extension.

Flags: needinfo?(kcochrane)
Has STR: --- → yes
Regressed by: 2047653

In hide-launcher mode sidebar-panel-header.css hides .sidebar-panel-title, so the visible header text is no longer the heading but sidebar-panel-switcher's own label, which sidebar-panel-switcher.mjs derives by matching its view against SidebarController.getRevampSwitcherItems(). Extension panels instantiate the header from webext-panels.xhtml with no view attribute and webext-panels.js sets only heading, so the lookup matches nothing and the label falls back to ""; the same mismatch also leaves the current extension unchecked in the dropdown.

Fix: Have the switcher fall back to SidebarController.currentID when its view is empty, rather than plumbing view in from webext-panels.js, because sidebar-panel-header.css deliberately keys extension-panel padding on the absence of view (bug 2062881). Then extend browser_hide_sidebar.js, whose switcher-label coverage is built-in panels only, with an extension panel case.

Suggested severity: S3. Every extension panel header renders empty in this mode, but nothing is functionally blocked and the panel name is still visible in the switcher dropdown, and reaching it requires unchecking the non-default Open tools from sidebar setting with horizontal tabs.


If you'd like to provide feedback on this comment, please use the 👍 or 👎 reaction.
If you want to categorize your feedback you can add one of the following tags: ai-triage-wrong-file, ai-triage-wrong-cause, ai-triage-hallucination, ai-triage-out-of-scope, ai-triage-wrong-fix, ai-triage-shallow-fix.

Needinfo hackbot@mozilla.tld to have a patch generated for this bug, include any questions or directions as needed in your comment.

Assignee: nobody → kcochrane
Severity: -- → S3
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(kcochrane)
Priority: -- → P1
Whiteboard: [fidefe-sidebar]

This is occurring on all platforms

Attachment #9644102 - Attachment description: Bug 2072965 - Ensure sidebar extension panel header is shown when 'Open tools from sidebar' is unchecked on Win/Linux → Bug 2072965 - Ensure sidebar extension panel header is shown when 'Open tools from sidebar' is unchecked
OS: Unspecified → All
Hardware: Unspecified → Desktop
Pushed by kcochrane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/88b81b9e07b7 https://hg.mozilla.org/integration/autoland/rev/c68d29eb2646 Ensure sidebar extension panel header is shown when 'Open tools from sidebar' is unchecked r=sidebar-reviewers,sfoster
Status: ASSIGNED → RESOLVED
Closed: 11 days ago
Resolution: --- → FIXED
Target Milestone: --- → 158 Branch

The patch landed in nightly and beta is affected.

:kcochrane, please make an uplift decision for beta:

  • Nominate the patch for beta uplift approval if the fix should be included in this release, or
  • Set status-firefox157 to wontfix if the fix can wait for the next release.

See Requesting an Uplift for documentation on how to request an uplift.

For more information, please visit BugBot documentation.

Flags: needinfo?(kcochrane)
Flags: needinfo?(kcochrane)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: