Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: align viewWelcome to VS Code #14391

Merged

Conversation

dankeboy36
Copy link
Contributor

What it does

Aligns the viewWelcome behavior to VS Code.

BREAKING:

  • renamed renderCommandLinkNode to renderLinkNode
  • isEnabledClick gets a raw when context expression instead of a URL string
  • from now on, the command enablement is independent of the Theia command handler; only the VS Code's when clause context is the tiebreaker.

How to test

See how to use it: https://github.com/dankeboy36/eclipse-theia-14309/tree/main?tab=readme-ov-file#eclipse-theia-14309-readme.

VSIX: https://github.com/dankeboy36/eclipse-theia-14309/raw/refs/heads/main/eclipse-theia-14309-0.0.1.vsix

VS Code:

Screen.Recording.2024-11-03.at.13.41.04.mov

Theia:

Screen.Recording.2024-11-03.at.13.37.45.mov

Closes: #14309
Ref: arduino/arduino-ide#2543

Follow-ups

Review checklist

Reminder for reviewers

Copy link
Contributor

@rschnekenbu rschnekenbu left a comment

Choose a reason for hiding this comment

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

Looking good to me, Thanks!

@rschnekenbu rschnekenbu merged commit 8d3c5a1 into eclipse-theia:master Nov 7, 2024
10 of 11 checks passed
@sgraband sgraband added this to the 1.56.0 milestone Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[vscode] viewsWelcome#enablement is not honored by Theia
3 participants