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

Dragging window while maximized in Windows 10 doesn't reset maximized status #2203

Closed
2 tasks done
axelitus opened this issue Sep 13, 2017 · 5 comments
Closed
2 tasks done
Labels
🖼 Platform: Windows Issue pertains to Windows 🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper

Comments

@axelitus
Copy link

axelitus commented Sep 13, 2017

  • I am on the latest Hyper.app version

  • I have searched the issues of this repo and believe that this is not a duplicate

  • OS version and name: Windows 10 version 1703

  • Hyper.app version: 1.4.4

  • Link of a Gist with the contents of your .hyper.js: https://gist.github.com/axelitus/742ad5ce3359b9c22bba547ca2b1f8ed

  • Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A

  • The issue is reproducible in vanilla Hyper.app:

Issue

When Hyper is maximized and one drags the window to move it, the normal behavior is that the window gets "restored" (looses the maximized status). This does not happen for Hyper.

The Hyper window gets moved to the new coordinates, but the window is still "maximized".
2017-09-13_16h35_39

So dragging the window again and trying to "attach" to the Windows sweet spots does not work (because it is already maximized). The "restore" button must be clicked and then the maximize button again.

What should happen is that when Hyper get's dragged, the "restored" window status should be set.

@chabou chabou added 🖼 Platform: Windows Issue pertains to Windows 🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper labels Sep 13, 2017
@wilsontayar
Copy link
Contributor

Hmm... For me everything is working as expected.
By looking at your .hyper.js file it seems that you are using some plugins.
Do you mind testing this without them?

I guess this code takes care of that:
https://github.com/zeit/hyper/blob/master/app/ui/window.js#L154

For reference, here's my vanilla hyper:
maximized_hyper

@chabou
Copy link
Collaborator

chabou commented Sep 16, 2017

cc @axelitus

@axelitus
Copy link
Author

I'll reset the hyper.js file, but I need to do this on monday to do this on the same computer.

In my laptop that I'm currently on it works as expected, exactly as @wilsontayar image (I'm using latest version 1.4.6 here though).

@axelitus
Copy link
Author

Indeed I can confirm this is happening due to a plugin installed. I removed my hyper.js config file and started with a clean one and it's working flawlessly. I'm going to try and pinpoint which one of the plugins caused this issue, but for now, I'll close this as it's obviously not Hyper's issue. Thanks, @wilsontayar and @chabou!

@axelitus
Copy link
Author

axelitus commented Sep 19, 2017

It seems the plugin 'hyper-tabs-enhanced' was causing all of my issues (#2094, #2095 and this one). I have removed that plugin and now it's working fine. They also have an issue placed in their github repository: hyper-tabs-enhanced issue 24

This is my new hyper.js file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖼 Platform: Windows Issue pertains to Windows 🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper
Projects
None yet
Development

No branches or pull requests

3 participants