Closed Bug 2019550 Opened 6 months ago Closed 5 months ago

Cannot collapse/expand the full document outline of a PDF anymore

Categories

(Firefox :: PDF Viewer, defect, P1)

defect

Tracking

()

RESOLVED FIXED
150 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr140 --- unaffected
firefox148 --- wontfix
firefox149 --- wontfix
firefox150 --- fixed

People

(Reporter: robwu, Assigned: calixte)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Until Firefox 147, it used to be possible to expand or collapse all sections of a PDF document outline by double-clicking on the "Show Document Outline" button (with tooltip Show Document Outline (double-click to expand/collapse all items)).

I originally added the functionality and explanation in https://github.com/mozilla/pdf.js/pull/6242.

STR:

  1. Open https://opensource.adobe.com/dc-acrobat-sdk-docs/pdfstandards/PDF32000_2008.pdf
  2. Look for the "Document outline" button
  3. Try to double-click it.

Expected:

  • Am able to double-click to collapse/expand all outlines.

Actual:

  • Not possible from Firefox 148.

This appears to be a regression from https://github.com/mozilla/pdf.js/pull/20495
The underlying logic is still in the code, but it appears that the functionality is unavailable because the "Document outline" button has now moved into a submenu (opened via the "Views" button).

If I force the menu to be shown (and not hide on click), with document.getElementById("viewsManagerSelectorOptions").style.visibility = "visible"
then I can double-click on the "Document outline" button.

"Regressed by" cannot link to a Github URL, so I am adding the regressor as See also: https://github.com/mozilla/pdf.js/pull/20495

Calixte, as the author of the regression, could you take a look? Perhaps we should add a toggle all outlines button if a PDF has multiple outlines?

Flags: needinfo?(cdenizet)

Note that the original flow was documented at https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#outline-sidebar :

  • To expand/collapse all items under the selected item, press Shift while clicking on the triangle.
  • Double-click on the "Show document outline" button (Show document outline) to expand/collapse all outline items.

Right now, when I Shift-click in the outline, the whole outline collapses, not just the section underneath. That is a bug... It should only collapse the outlines under the section.

And to deal with the regression reported here, an option could be to introduce an artificial top-level triangle, extra buttons, or something else.

I wasn't aware of this feature so we didn't take into account when designing the new sidebar.
:Ryan, what do you think we can do ?

Flags: needinfo?(cdenizet) → needinfo?(rcasey)

So for now we don't plan to fix the issue but we keep it in mind.

Severity: -- → S3
Priority: -- → P3

UX has taken note and will keep in mind for future updates.

Status: NEW → RESOLVED
Closed: 6 months ago
Flags: needinfo?(rcasey)
Resolution: --- → WONTFIX

For now, let's "fix" this bug in adding a new hidden feature: double-clicking on the header part of the sidebar will collapse/expand all the outlines or hide/show all the layers.
And we'll figure out later a better way to expose this feature or not to everybody and not keep it confidential.

Assignee: nobody → cdenizet
Status: RESOLVED → REOPENED
Priority: P3 → P1
Resolution: WONTFIX → ---
Depends on: 2024405
Status: REOPENED → RESOLVED
Closed: 6 months ago5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 150 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: