Bug 236850 - Hide PDF.js sidebar for long documents
Summary: Hide PDF.js sidebar for long documents
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: PDF (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Nguyen (:ntim)
URL:
Keywords: InRadar
Depends on:
Blocks: 235969
  Show dependency tree / graph
 
Reported: 2022-02-18 11:18 PST by Tim Nguyen (:ntim)
Modified: 2022-02-22 11:05 PST (History)
7 users (show)

See Also:

Attachments
[fast-cq] Patch (1.68 KB, patch)
2022-02-22 09:50 PST, Tim Nguyen (:ntim) 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Nguyen (:ntim) 2022-02-18 11:18:39 PST 
PDF.js automatically opens a sidebar for large documents, we don't want it.
Comment 1 Radar WebKit Bug Importer 2022-02-18 11:28:27 PST 
<rdar://problem/89157217>
Comment 2 Tim Nguyen (:ntim) 2022-02-18 13:34:46 PST 
I think we can do this from the content-script: removing the sidebarOpen class from #outerContainer.
Comment 3 Tim Nguyen (:ntim) 2022-02-22 09:50:12 PST 
Created attachment 452879 [details]
[fast-cq] Patch
Comment 4 EWS 2022-02-22 11:05:50 PST 
Committed r290321 (247643@main): <https://commits.webkit.org/247643@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 452879 [details].