Bug 236850

Summary: Hide PDF.js sidebar for long documents
Product: WebKit Reporter: Tim Nguyen (:ntim) <ntim>
Component: PDFAssignee: Tim Nguyen (:ntim) <ntim>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, changseok, esprehn+autocc, ews-watchlist, gyuyoung.kim, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 235969    
Attachments:
Description Flags
[fast-cq] Patch none

Tim Nguyen (:ntim)
Reported 2022-02-18 11:18:39 PST
PDF.js automatically opens a sidebar for large documents, we don't want it.
Attachments
[fast-cq] Patch (1.68 KB, patch)
2022-02-22 09:50 PST, Tim Nguyen (:ntim) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-02-18 11:28:27 PST
<rdar://problem/89157217>
Tim Nguyen (:ntim)
Comment 2 2022-02-18 13:34:46 PST
I think we can do this from the content-script: removing the sidebarOpen class from #outerContainer.
Tim Nguyen (:ntim)
Comment 3 2022-02-22 09:50:12 PST
Created attachment 452879 [details] [fast-cq] Patch
EWS
Comment 4 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].
Note You need to log in before you can comment on or make changes to this bug.