hyperterm-summon icon indicating copy to clipboard operation
hyperterm-summon copied to clipboard

`hideDock` does not work on Windows 10

Open DRSDavidSoft opened this issue 5 years ago • 6 comments

  • [x] I am on the latest hyperterm-summon version
  • [x] I have searched the issues of this repo and believe that this is not a duplicate
  • OS version and name: Windows 10 v1803 – build 17134
  • Hyper.exe version: 2.0.0.1008
  • hyperterm-summon version: 2.0.9
  • Link of a hastepin with the contents of my .hyper.js file: .hyper.js
  • Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A

Issue

The config.summon.hideDock does not work as expected on Windows platform.

Steps to Reproduce

  1. Set the said option to true in the configuration file.
  2. Press the hotkey to summon hyper (e.g. Ctrl+;)
  3. Press again the hotkey to “hide” the hyper window.
  4. The hideDock has no effect on the window.

Expected Result

The hyper icon should be hidden from the taskbar (equivalent to dock in macOS).

Actual Result

The window is minimized instead of the window getting hidden.

Demonstration

hyper is minimized instead of being hidden

Shouldn't hyper hide the icon in taskbar when this option is enabled on Windows?

DRSDavidSoft avatar Oct 31 '18 20:10 DRSDavidSoft

Unfortunately, I believe hiding the dock is an ability Electron can only perform on macOS. I don’t think what you desire is possible.

If you research and find otherwise, let me know! I’d welcome a PR adding the functionality.

dcalhoun avatar Oct 31 '18 21:10 dcalhoun

@dcalhoun Thanks for the quick reply.

To overcome this limitation on Windows, Terminus puts the window into the traybar which effectively removes the icon from the taskbar.

This method (sample code) requires browser-window dependency.

Does this make sense, and can I do that in the PR? It'll be cool to have a Hyper window in the background and summon it using the hotkey :)

DRSDavidSoft avatar Nov 01 '18 03:11 DRSDavidSoft

Oh, great! It does look like Electron provides an API for managing the Tray as well as a BrowserWindow API with similar hooks to your sample code.

I'm not too familiar with the Windows OS or the Tray (nor do I have access to a Windows machine for testing), so it'd be challenging for me to implement this feature. However, if you'd like to explore implementing this feature in a new PR, I support that.

Rather than adding the browser-window dependency, I'd prefer this feature be built with the API that Electron provides (i.e. the aforementioned Tray, BrowserWindow).

dcalhoun avatar Nov 01 '18 12:11 dcalhoun

+💯 on this :)

Also out of curiosity but is it intended functionality that this hotkey cannot summon a hyper term unless it's already been manually opened and minimized?

jcklpe avatar Nov 12 '18 03:11 jcklpe

@jcklpe unless I'm mistaken, what you are describing is unrelated to this issue. Please open a new issue with detailed steps to reproduce and your expected outcome. Thanks.

dcalhoun avatar Nov 12 '18 03:11 dcalhoun

@dcalhoun Will do. I assumed from reading this stuff that it is impossible to access an electron app with a global hotkey though. Will log though.

jcklpe avatar Nov 12 '18 07:11 jcklpe