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

Add recapture button to inspector #343

Merged
merged 1 commit into from
Mar 10, 2024
Merged

Conversation

MinusGix
Copy link
Member

@MinusGix MinusGix commented Mar 2, 2024

This PR simply adds a 'Recapture' button to the inspector view:
image

(Q: should it be 'Refresh'? Should the button be smaller? Should it just be a refresh icon, but then we have to package a small svg?)

This is useful in scenarios where you want to recapture without interacting with the primary window itself.
An example is a focus issue in Lapce that I'm debugging, where I expect the focus is ending up somewhere it shouldn't, but if I want to trigger the inspector then I need the focus elsewhere. So I can't find out where the focus ends up! Recapture lets me interact with the primary window, recapture, and see where the focus is at.

Why this is a draft pr is that there's a bug with this code where sometimes when pressing recapture it will produce a blank screen, for reasons I don't know.

@Zoxc
Copy link
Contributor

Zoxc commented Mar 5, 2024

Why this is a draft pr is that there's a bug with this code where sometimes when pressing recapture it will produce a blank screen, for reasons I don't know.

Do you mean the Captured Window panel will be blank? If so, that's an existing issue.

@MinusGix MinusGix marked this pull request as ready for review March 6, 2024 02:08
@MinusGix
Copy link
Member Author

MinusGix commented Mar 6, 2024

Do you mean the Captured Window panel will be blank? If so, that's an existing issue.

That's what I meant, thanks for clarifying!

@dzhou121 dzhou121 merged commit 0ae15f9 into lapce:main Mar 10, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants