You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I find it quite difficult to read the InitSQL popup
The improvement is minimal, the type === "sql" branch currently renders a plain <pre> block. I just replace that with SyntaxHighlighter, exactly like we did in SqlPopUp.consts.tsx. The Box wrapper and dialog structure stay completely untouched.
Before:
After:
AI & Automation Policy
I am the human author and take full personal responsibility for every change in this PR.
No AI or automated generative tool was used in any part of this PR OR I have disclosed all tool(s) below.
Drafted with the assistance of Claude.
Checklist
Code compiles and existing tests pass locally.
New or updated tests are included where applicable.
pashagolub
changed the title
feat(webui): add SQL syntax highlighting to InitSQL popup
[+] add SQL syntax highlighting to InitSQL popup in webui
Mar 20, 2026
pashagolub
changed the title
[+] add SQL syntax highlighting to InitSQL popup in webui
[+] add SQL syntax highlighting to webui InitSQL popup
Mar 20, 2026
Hello @mohamadyasser118 , I just have a small question. is there a reason we use the full build here and the light build in the other sql highlighter view? for me it would make sense to make them consistent and use the light build if we don't need the full version to avoid unnecessary bundle size increase. (also since we need to support one language)
Hello @mohamadyasser118 , I just have a small question. is there a reason we use the full build here and the light build in the other sql highlighter view? for me it would make sense to make them consistent and use the light build if we don't need the full version to avoid unnecessary bundle size increase. (also since we need to support one language)
Hello Mazen, Actually the bundle impact is minimal in practice, so there is no big difference ..
but this can be a good enhancement if you want to make,
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
I find it quite difficult to read the
InitSQLpopupThe improvement is minimal,
the type === "sql"branch currently renders a plain<pre>block. I just replace that withSyntaxHighlighter, exactly like we did inSqlPopUp.consts.tsx. TheBoxwrapper and dialog structure stay completely untouched.Before:
After:
AI & Automation Policy
Drafted with the assistance of Claude.
Checklist