Skip to content

Commit 27db42d

Browse files
Merge pull request #20841 from Snuffleupagus/misc-improvements
A couple of small improvements
2 parents 51cde82 + e6f32f2 commit 27db42d

3 files changed

Lines changed: 11 additions & 10 deletions

File tree

eslint.config.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -480,7 +480,7 @@ export default [
480480
Other
481481
\* ======================================================================== */
482482
{
483-
files: ["gulpfile.mjs", "check_l10n.mjs"],
483+
files: ["gulpfile.mjs"],
484484
languageOptions: { globals: globals.node },
485485
},
486486
];

src/display/api.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1530,10 +1530,9 @@ class PDFPageProxy {
15301530
this._pumpOperatorList(intentArgs);
15311531
}
15321532

1533-
const recordForDebugger = Boolean(
1533+
const recordForDebugger = !!(
15341534
this._pdfBug && globalThis.StepperManager?.enabled
15351535
);
1536-
15371536
const shouldRecordOperations =
15381537
!this.recordedBBoxes && (recordOperations || recordForDebugger);
15391538

@@ -1543,12 +1542,11 @@ class PDFPageProxy {
15431542
if (shouldRecordOperations) {
15441543
const recordedBBoxes = internalRenderTask.gfx?.dependencyTracker.take();
15451544
if (recordedBBoxes) {
1546-
if (internalRenderTask.stepper) {
1547-
internalRenderTask.stepper.setOperatorBBoxes(
1548-
recordedBBoxes,
1549-
internalRenderTask.gfx.dependencyTracker.takeDebugMetadata()
1550-
);
1551-
}
1545+
internalRenderTask.stepper?.setOperatorBBoxes(
1546+
recordedBBoxes,
1547+
internalRenderTask.gfx.dependencyTracker.takeDebugMetadata()
1548+
);
1549+
15521550
if (recordOperations) {
15531551
this.recordedBBoxes = recordedBBoxes;
15541552
}

web/sidebar.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,10 @@ class Sidebar {
7979
toggleButton.addEventListener("click", this.toggle.bind(this));
8080
this._isOpen = false;
8181
sidebar.hidden = true;
82-
globalAbortSignal?.addEventListener("abort", this.destroy.bind(this));
82+
globalAbortSignal?.addEventListener("abort", this.destroy.bind(this), {
83+
once: true,
84+
});
85+
8386
this.#resizeObserver = new ResizeObserver(
8487
([
8588
{

0 commit comments

Comments
 (0)