Skip to content

qt-ui, qt-core: Merge qt-ui into qt-core#517

Open
benchoq wants to merge 1 commit into
qt-labs:devfrom
benchoq:ben_0621_merge_ui
Open

qt-ui, qt-core: Merge qt-ui into qt-core#517
benchoq wants to merge 1 commit into
qt-labs:devfrom
benchoq:ben_0621_merge_ui

Conversation

@benchoq

@benchoq benchoq commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

This commit merges the code from qt-ui into qt-core. The custom editor has been reimplemented using Svelte to remove the dependency on the deprecated package @vscode/webview-ui-toolkit.

The openWidgetDesigner command has been removed because it is outdated and its use case is unclear at the moment. For example, the current implementation does not look up available designers from the additionalQtPaths settings, nor does it consider the PySide virtual environment (venv or uv).

A new command has been added to open the current file in the designer, matching the existing behavior for .ts files. When a .ui file is opened in the plain text editor, a button is shown in the editor tab to allow opening it in the designer.

Log output has also been improved to make failures when opening the designer easier to understand.

Task-number: VSCODEEXT-263

@benchoq benchoq force-pushed the ben_0621_merge_ui branch 5 times, most recently from e353a9c to 6cc086a Compare June 30, 2026 11:50
@benchoq benchoq marked this pull request as ready for review June 30, 2026 11:50
@benchoq benchoq force-pushed the ben_0621_merge_ui branch 3 times, most recently from 907dd6c to 62d1461 Compare June 30, 2026 14:03
@OrkunTokdemir

Copy link
Copy Markdown
Collaborator

Generally, it works well.
There is one thing I noticed. It uses the Qt icon instead of the ui icon. Is it possible to use the ui icon there?
image

NIT: It would be quite good to click Open with Qt Widgets Designer by just pressing Enter, or select the button with the arrow keys, then press Enter like in the old one. It is not curial but it helps to keep hands on the keyboard 🙂

@benchoq

benchoq commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator Author

Generally, it works well. There is one thing I noticed. It uses the Qt icon instead of the ui icon. Is it possible to use the ui icon there? image

NIT: It would be quite good to click Open with Qt Widgets Designer by just pressing Enter, or select the button with the arrow keys, then press Enter like in the old one. It is not curial but it helps to keep hands on the keyboard 🙂

Both sugguestions look good. Will do it. thanks!

@benchoq benchoq force-pushed the ben_0621_merge_ui branch 2 times, most recently from 8295976 to 6cfb5c8 Compare July 7, 2026 13:43
This commit merges the code from qt-ui into qt-core. The custom editor
has been reimplemented using Svelte to remove the dependency on the
deprecated package `@vscode/webview-ui-toolkit`.

The `openWidgetDesigner` command has been removed because it is outdated
and its use case is unclear at the moment. For example, the current
implementation does not look up available designers from the
`additionalQtPaths` settings, nor does it consider the PySide virtual
environment (venv or uv).

A new command has been added to open the current file in the designer,
matching the existing behavior for `.ts` files. When a `.ui` file is
opened in the plain text editor, a button is shown in the editor tab to
allow opening it in the designer.

Log output has also been improved to make failures when opening the
designer easier to understand.

Task-number: [VSCODEEXT-263](https://qt-project.atlassian.net/browse/VSCODEEXT-263)
@benchoq benchoq force-pushed the ben_0621_merge_ui branch from 6cfb5c8 to db334fa Compare July 7, 2026 14:03
@benchoq

benchoq commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator Author

@OrkunTokdemir Just finished reflecting your sugguestions. Please have it reviewed when you're available. :-)

@benchoq benchoq requested a review from OrkunTokdemir July 7, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants