qt-ui, qt-core: Merge qt-ui into qt-core#517
Open
benchoq wants to merge 1 commit into
Open
Conversation
e353a9c to
6cc086a
Compare
907dd6c to
62d1461
Compare
Collaborator
Collaborator
Author
8295976 to
6cfb5c8
Compare
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)
6cfb5c8 to
db334fa
Compare
Collaborator
Author
|
@OrkunTokdemir Just finished reflecting your sugguestions. Please have it reviewed when you're available. :-) |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.


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
openWidgetDesignercommand 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 theadditionalQtPathssettings, 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
.tsfiles. When a.uifile 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