Notebook: Scrolling text / markdown editor and preview should ideally in sync

microsoft / azuredatastudio

Azure Data Studio is a data management and development tool with connectivity to popular cloud and on-premises databases. Azure Data Studio supports Windows, macOS, and Linux, with immediate capability to connect to Azure SQL and SQL Server. Browse the extension library for more database support options including MySQL, PostgreSQL, and MongoDB.

https://learn.microsoft.com/sql/azure-data-studio

MIT License

7.57k stars 901 forks source link

Notebook: Scrolling text / markdown editor and preview should ideally in sync #11408

Open MsSQLGirl opened 4 years ago

MsSQLGirl commented 4 years ago

Azure Data Studio Version: Version: 1.21.0-insider (user setup) Commit: b098ce64b5f79b6db2d39f637cea73a2fa9f4550 Date: 2020-07-17T05:40:08.933Z VS Code: 1.48.0 Electron: 7.3.2 Chrome: 78.0.3904.130 Node.js: 12.8.1 V8: 7.8.279.23-electron.0 OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

1.Open a Jupyter Book SQL 2019 BDC and launch TSG100 (long text cell).

Resize your screen such that the text cell preview is a little squished.

Actual: At the start, it looks OK.

When you scroll on the editor, the preview doesn't follow. This makes it a little tricky for the editing flow.

Expected: The scroll on the editor and preview are in-sync.

MsSQLGirl commented 4 years ago

Note: the workaround is to turn the word-wrap on from Settings.

MsSQLGirl commented 4 years ago

Note: this problem is exacerbated when there is a big image rendered on the preview. The editor will easily lose track of where things are at.

MsSQLGirl commented 4 years ago

Removing the Blocking: ADS TSG adoption now that users can simply use WYSIWYG. This would still be handy to have though.

MsSQLGirl commented 3 years ago

Just an idea as well here, I'm wondering if in Split mode, we can show the cursor (or marker) on the preview mode as well as the view mode. This to ease the eyes to catch up on where things are at. This would be useful when in combination with sync'd scrolling.

MsSQLGirl commented 3 years ago

Bumped up the wrong issue. I meant to bump this up instead: #14483