Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link:

Just merged main into this branch to grab some CI fixes (to have a better idea of what the remaining failures are).

Thanks ! The remaining failures are due to the snapshots mismatches caused by the removal of the minimap button in the toolbar. If we decide to go in the same path then I'll trigger the updates.

Current place of the Show Minimap option

the removal of the minimap button in the toolbar.

This sounds like a good idea 👍

The notebook toolbar is already a bit crowded with many other buttons, so it looks fine to remove it. And enabling the minimap feels a bit more like a global setting than a per-notebook setting, so having it in the settings and also as a menu item makes sense.

bot please update snapshots

Documentation snapshots updated.

Galata snapshots updated.

Edit: Done in c636cd0.

The remaining two failing tests:

    [jupyterlab] › test/jupyterlab/notebook-toolbar.test.ts:205:7 › Reactive toolbar › Reducing toolbar width should display opener item 
    [jupyterlab] › test/jupyterlab/notebook-toolbar.test.ts:223:7 › Reactive toolbar › Items in popup toolbar should have the same order 

require manual updates:

Error: expect(locator).toHaveCount(expected) failed

Locator:  locator('.jp-NotebookPanel-toolbar').locator('.jp-Toolbar-item:visible')
Expected: 15
Received: 14
Timeout:  5000ms

Call log:
  - Expect "toHaveCount" with timeout 5000ms
  - waiting for locator('.jp-NotebookPanel-toolbar').locator('.jp-Toolbar-item:visible')
    9 × locator resolved to 14 elements
      - unexpected value "14"


  207 |   }) => {
  208 |     const toolbar = page.locator('.jp-NotebookPanel-toolbar');
> 209 |     await expect(toolbar.locator('.jp-Toolbar-item:visible')).toHaveCount(15);
      |                                                               ^