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

[vscode] Add WindowState active in plugin API #13718

Merged
merged 1 commit into from
May 28, 2024

Conversation

rschnekenbu
Copy link
Contributor

What it does

Implements the new VS Code API WindowState Active

Fixes #13692

Contributed on behalf of ST Microelectronics

How to test

This can be tested with the following extension:

This extension will display a notification has soon as window state is changed, focus or active. To test, you can get or lose focus with any combination of idle time to check activity state. User is considered as inactive after 20sec of doing nothing, and this timer will be reset has soon as a mouse button is clicked / key stroke or touch gesture.

Follow-ups

None

Review checklist

Reminder for reviewers

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Works generally pretty well, but I some suggestions on the logic & code, see below.

@rschnekenbu
Copy link
Contributor Author

Thanks for your review, @msujew !
Good catch for the logic of the reset. I changed the inactivity check interval to 1sec and extended the number of checks to 30.

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

LGTM 👍

fixes eclipse-theia#13692

contributed on behalf of STMicroelectronics

Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
@msujew msujew merged commit 5844c0d into eclipse-theia:master May 28, 2024
14 checks passed
@msujew msujew added the vscode issues related to VSCode compatibility label May 28, 2024
@rschnekenbu rschnekenbu deleted the issues/13692 branch May 28, 2024 12:11
@jfaltermeier jfaltermeier added this to the 1.50.0 milestone Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vscode issues related to VSCode compatibility
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[vscode] Support WindowState active boolean introduced in VS Code 1.89
4 participants