Amethyst icon indicating copy to clipboard operation
Amethyst copied to clipboard

Float small windows not working with GoLand popups?

Open ivanpk opened this issue 5 years ago • 6 comments

System

  • Amethyst version: 0.14.2 (76)
  • macOS version: 10.14.5

What's the problem?

Unable to float small popups in GoLand, they become tiled immediately

How can it be reproduced?

Use GoLand and call a small window popup (ctrl-G for example, to go to a line number opens a small window) - Amethyst then tiles this instead of leaving it floating. The float small windows setting is enabled as well.

If you add Goland to the automatically float applications listed setting, then the entire app floats, when I'd like only the popups not to float.

What applications are involved?

Jetbrain's GoLand

Has anything fixed it, even temporarily?

No

ivanpk avatar Oct 08 '19 17:10 ivanpk

also all "CMD + ," settings popups is not floating

simonelnahas avatar Oct 17 '19 03:10 simonelnahas

What counts as a "small" window appears to be hard-coded: https://github.com/ianyh/Amethyst/blob/master/Amethyst/Model/Window.swift#L249 -- perhaps if this were configurable it could be used to exclude some of the GoLand pop-ups as well? I measured the GoLand pop-ups and they are a bit wider than 500px for me (779x484 px), so they don't qualify as small

justenwalker avatar Apr 13 '20 21:04 justenwalker

noticed the same issue in Intellij for almost all its popups. Maybe it would be better to make all child / modal windows floating by default

luxmeter avatar Nov 02 '23 13:11 luxmeter

It's unfortunately not that simple. Some applications just use normal windows for their "modals," so you can't reliably differentiate them.

ianyh avatar Nov 02 '23 13:11 ianyh

I see, but they would have a parent, wound't they? I have no clue, I just imagine that there is some kind of hierarchy. Might be beneficial to add an user option to make all child windows floating by default.

luxmeter avatar Nov 02 '23 15:11 luxmeter

Or may I suggest making the max height/width user configurable, as it is still hard coded to 500px

justenwalker avatar Nov 02 '23 16:11 justenwalker