Skip to content
New issue

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

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

Already on GitHub? # to your account

Closing a paused terminal with the spacebar triggers QuickLook #23

Closed
cbuchart opened this issue May 31, 2017 · 6 comments
Closed

Closing a paused terminal with the spacebar triggers QuickLook #23

cbuchart opened this issue May 31, 2017 · 6 comments
Labels

Comments

@cbuchart
Copy link

  1. User calls a batch file which internally calls pause before finishing (Press any key to continue . . .).
  2. User presses the Spacebar to close the window.
  3. QuickLook unexpectedly previews the batch file.

Attached a very simple test case (it is zipped because cannot upload .bat directly)

test.zip

@xupefei
Copy link
Member

xupefei commented May 31, 2017

Hi, I have tested this .bat file but the QuickLook didn't popped up. I think you are pressing the SPACE key when focusing the file in explorer window, not the cmd window.
QuickLook will popup preview window when (i) the current foreground window is the desktop or an explorer window & (ii) there is a file selected. Therefore the window should not pop up in your case since (i) is not satisfied.

@cbuchart
Copy link
Author

cbuchart commented Jun 6, 2017

No, I've double-checked that the focus is on the terminal, otherwise the terminal windows wouldn't close. More information: Windows 10 (1703, 15063.296). If I can get more information about it I'll let you know.

@xupefei
Copy link
Member

xupefei commented Jun 6, 2017 via email

@cbuchart
Copy link
Author

cbuchart commented Jun 7, 2017

latest so far: 0.1.13

@xupefei
Copy link
Member

xupefei commented Jun 7, 2017

Finally I figured out how to reproduce it issue. A .bat on Desktop would not work, but one in a explorer window.
I think it is because the KeyUp/KeyDown sequence. The cmd window closes in KeyDown, but QuickLook is handling KeyUp. I will fix this asap.

@xupefei xupefei added the bug label Jun 7, 2017
@xupefei xupefei closed this as completed in 00a6100 Jun 8, 2017
@cbuchart
Copy link
Author

cbuchart commented Jun 9, 2017

0.1.14 works flawlessly, nice bug fixing, thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants