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-81877 Image preview should not zoom on first click if unfocused #81935

Closed
wants to merge 1 commit into from

Conversation

skprabhanjan
Copy link
Contributor

@skprabhanjan skprabhanjan commented Oct 4, 2019

@mjbvz , Here is the fix for #81877 .
I am not sure if this is the solution you wanted but just wanted to know if this is towards the right direction. ( Because you have mentioned about focus , I am not quite sure how to get that info here )
Thanks :)

@caiges
Copy link

caiges commented Oct 4, 2019

I opened a PR yesterday for this but forgot to link the issue, #81933 to my PR. I took a similar approach. It doesn't appear to be possible to query for whether the current view is active -- vscode.window is undefined and even if it were available, by the time the 'click' handler executes, the window is already active.

@skprabhanjan
Copy link
Contributor Author

@mjbvz , Any updates on this ?

@mjbvz
Copy link
Collaborator

mjbvz commented Oct 14, 2019

Hey sorry for the delay. A few other people also jumped on the issue before I could unmark it as help wanted and those PRs also handled a few important additional cases, such as when the preview is unfocused an refocused after the initial load

I'm close this PR in favor of #82074 but the feature will be making it into the next release

@mjbvz mjbvz closed this Oct 14, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants