Merge pull request #12616 from Snuffleupagus/PDFThumbnailViewer-cleanup
Improve the cleanup functionality for thumbnails
This commit is contained in:
commit
1f65896093
@ -502,10 +502,6 @@ class PDFThumbnailView {
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
static cleanup() {
|
|
||||||
TempImageFactory.destroyCanvas();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
export { PDFThumbnailView };
|
export { PDFThumbnailView, TempImageFactory };
|
||||||
|
@ -20,7 +20,8 @@ import {
|
|||||||
scrollIntoView,
|
scrollIntoView,
|
||||||
watchScroll,
|
watchScroll,
|
||||||
} from "./ui_utils.js";
|
} from "./ui_utils.js";
|
||||||
import { PDFThumbnailView } from "./pdf_thumbnail_view.js";
|
import { PDFThumbnailView, TempImageFactory } from "./pdf_thumbnail_view.js";
|
||||||
|
import { RenderingStates } from "./pdf_rendering_queue.js";
|
||||||
|
|
||||||
const THUMBNAIL_SCROLL_MARGIN = -19;
|
const THUMBNAIL_SCROLL_MARGIN = -19;
|
||||||
const THUMBNAIL_SELECTED_CLASS = "selected";
|
const THUMBNAIL_SELECTED_CLASS = "selected";
|
||||||
@ -156,7 +157,15 @@ class PDFThumbnailViewer {
|
|||||||
}
|
}
|
||||||
|
|
||||||
cleanup() {
|
cleanup() {
|
||||||
PDFThumbnailView.cleanup();
|
for (let i = 0, ii = this._thumbnails.length; i < ii; i++) {
|
||||||
|
if (
|
||||||
|
this._thumbnails[i] &&
|
||||||
|
this._thumbnails[i].renderingState !== RenderingStates.FINISHED
|
||||||
|
) {
|
||||||
|
this._thumbnails[i].reset();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
TempImageFactory.destroyCanvas();
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
Loading…
Reference in New Issue
Block a user