Skip to content

[*] improve SQL popup readability, closes #1241#1280

Merged
pashagolub merged 2 commits intocybertec-postgresql:masterfrom
mohamadyasser118:fix/sql-popup-readability-1241
Mar 6, 2026
Merged

[*] improve SQL popup readability, closes #1241#1280
pashagolub merged 2 commits intocybertec-postgresql:masterfrom
mohamadyasser118:fix/sql-popup-readability-1241

Conversation

@mohamadyasser118
Copy link
Copy Markdown
Contributor

Description

This solves the problem that The SQLs popup renders raw SQL strings in plain DataGrid cells with no
formatting, no monospace font, no line breaks, no indentation. Any multi-line query becomes a single unreadable wall of text.

Fixes #1241

Before:

Before

After:

After (2)

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.

AI/automation tools used (leave blank if none):

Drafted with the assistance of Claude Sonnet 4.6.

Checklist

  • Code compiles and existing tests pass locally.
  • New or updated tests are included where applicable.
  • Documentation is updated where applicable.

@0xgouda
Copy link
Copy Markdown
Collaborator

0xgouda commented Mar 6, 2026

Could you include a video instead of a screenshot? I can't see where the version number appears.

@mohamadyasser118
Copy link
Copy Markdown
Contributor Author

Could you include a video instead of a screenshot? I can't see where the version number appears.

Screen.Recording.2026-03-06.065424.mp4

@pashagolub pashagolub changed the title fix(webui): improve SQL popup readability, closes #1241 [*] improve SQL popup readability, closes #1241 Mar 6, 2026
@pashagolub pashagolub self-assigned this Mar 6, 2026
@pashagolub pashagolub added the webui Admin UI related issues label Mar 6, 2026
Copy link
Copy Markdown
Collaborator

@pashagolub pashagolub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look great! Thanks!

@pashagolub pashagolub merged commit 7b45189 into cybertec-postgresql:master Mar 6, 2026
5 checks passed
@mohamadyasser118
Copy link
Copy Markdown
Contributor Author

Hello @Mohamed-Elfardy,
I saw your PR, and it has very useful features, so as this PR has been merged, I suggest you make the other features, like react-syntax-highlighter or any other things, in a new PR so the SQL strings can be more readable.

@Mohamed-Elfardy
Copy link
Copy Markdown
Contributor

thanks for your suggestion,
if it's acceptable i will can do it @0xgouda

@mohamadyasser118 mohamadyasser118 deleted the fix/sql-popup-readability-1241 branch March 10, 2026 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

webui Admin UI related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The metric sqls in webui are not readable

4 participants