Merge pull request #18742 from Snuffleupagus/pageColorsBackground-unl…

…oaded-pages Use the "pageColorsBackground" option as background-color for non-loaded pages (issue 18680)
mozilla · Sep 12, 2024 · c72fb9b · c72fb9b
2 parents b915a9c + d86f94a
commit c72fb9b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/web/pdf_page_view.js b/web/pdf_page_view.js
@@ -222,6 +222,13 @@ class PDFPageView {
  this.scale * PixelsPerInch.PDF_TO_CSS_UNITS
  );
 
+ if (this.pageColors?.background) {
+ container?.style.setProperty(
+ "--page-bg-color",
+ this.pageColors.background
+ );
+ }
+
  const { optionalContentConfigPromise } = options;
  if (optionalContentConfigPromise) {
  // Ensure that the thumbnails always display the *initial* document

diff --git a/web/pdf_viewer.css b/web/pdf_viewer.css
@@ -61,6 +61,7 @@
  /* Define this variable here and not in :root to avoid to reflow all the UI
  when scaling (see #15929). */
  --scale-factor: 1;
+ --page-bg-color: unset;
 
  padding-bottom: var(--pdfViewer-padding-bottom);
 
@@ -114,7 +115,7 @@
  overflow: visible;
  border: var(--page-border);
  background-clip: content-box;
- background-color: rgb(255 255 255);
+ background-color: var(--page-bg-color, rgb(255 255 255));
 }
 
 .pdfViewer .dummyPage {

diff --git a/web/pdf_viewer.js b/web/pdf_viewer.js
@@ -927,6 +927,10 @@ class PDFViewer {
  // Ensure that the various layers always get the correct initial size,
  // see issue 15795.
  viewer.style.setProperty("--scale-factor", viewport.scale);
+
+ if (pageColors?.background) {
+ viewer.style.setProperty("--page-bg-color", pageColors.background);
+ }
  if (
  pageColors?.foreground === "CanvasText" ||
  pageColors?.background === "Canvas"