Backport PR jupyterlab#14865: In defer mode render enough cells to fi…

…ll the viewport
meeseeksmachine · Aug 2, 2023 · eda8b6c · eda8b6c
1 parent 4461759
commit eda8b6c
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/packages/notebook/src/widget.ts b/packages/notebook/src/widget.ts
@@ -887,8 +887,19 @@ export class StaticNotebook extends WindowedList {
       cellIdx++;
     }
 
+    // If the notebook is not fully rendered
     if (cellIdx < this.cellsArray.length) {
-      this._scheduleCellRenderOnIdle();
+      // If we are defering the cell rendering and the rendered cells do
+      // not fill the viewport yet
+      if (
+        this.notebookConfig.windowingMode === 'defer' &&
+        this.viewportNode.clientHeight < this.node.clientHeight
+      ) {
+        // Spend more time rendering cells to fill the viewport
+        await this._runOnIdleTime();
+      } else {
+        this._scheduleCellRenderOnIdle();
+      }
     } else {
       if (this._idleCallBack) {
         window.cancelIdleCallback(this._idleCallBack);