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

Move app to another monitor unexpected behaviour #538

Closed
T04435 opened this issue Sep 7, 2020 · 3 comments · Fixed by #481
Closed

Move app to another monitor unexpected behaviour #538

T04435 opened this issue Sep 7, 2020 · 3 comments · Fixed by #481
Assignees

Comments

@T04435
Copy link

T04435 commented Sep 7, 2020

Distribution (run cat /etc/os-release):

VERSION="20.04 LTS"
ID=pop
ID_LIKE="ubuntu debian"
PRETTY_NAME="Pop!_OS 20.04 LTS"
VERSION_ID="20.04"
HOME_URL="https://pop.system76.com"
SUPPORT_URL="https://support.system76.com"
BUG_REPORT_URL="https://github.com/pop-os/pop/issues"
PRIVACY_POLICY_URL="https://system76.com/privacy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal
LOGO=distributor-logo-pop-os

Related Application and/or Package Version (run apt policy $PACKAGE NAME):

Issue/Bug Description:
Vertical screens are ignored when moving app from monitor to monitor. I think this could be realed to different resolutions on the monitors a the app keeps the previous monitor size when IMHO it should adjust to the new monitor resolution/scale

Steps to reproduce (if you know):

  1. Setup 3 monitors
  2. Set monitor layout as per attachement 1
  3. Use shortcuts(Disable auto-tiling: I can't find the related Issue where moving app from monitor to monitor does not work with auto-tiling ON):
    image
  4. See attachments for issue:
    1
    original_layout
    0_original_layout
    after_shortcut: Shift + Ctrl + Super + Down
    1_after_shortcut
    after_enabling_tiling
    2_after_enabling_tiling

Expected behavior:
Move app to the monitor on direction to work on all monitor laytouts
3_expected

Other Notes:
I manage to move app to monitor after multiple iterations of Super + Ctrl + Shift + H | J | K | L but is not consistent

@jacobgkau jacobgkau transferred this issue from pop-os/pop Sep 8, 2020
@jacobgkau
Copy link
Member

jacobgkau commented Sep 8, 2020

Vertical screens are ignored when moving app from monitor to monitor.

I can confirm windows won't move between displays vertically with the arrow keys shortcuts in adjustment mode when tiling is enabled, only horizontally. After further testing, windows do move up/down vertical displays in adjustment mode when tiling is enabled. The only vertical-movement issue I'm able to reproduce is that windows do not move between displays vertically using the arrow keys in adjustment mode when tiling is disabled. However, Ctrl-Shift-Super-Up/Down does work in this case.

I think this could be realed to different resolutions on the monitors a the app keeps the previous monitor size when IMHO it should adjust to the new monitor resolution/scale

If you have tiling disabled (like your example shows), then there would not be any mechanism to resize the window automatically, so I'm not sure that's an issue. It looks like the window is hanging in between two displays after you move it to a smaller display using the GNOME shortcut, and then when you enable tiling, Pop Shell tiles it onto a different display than you were wanting because more of the window was on that display already.

  1. Use shortcuts(Disable auto-tiling: I can't find the related Issue where moving app from monitor to monitor does not work with auto-tiling ON):

That sounds like #522.

@hovo1990
Copy link

hovo1990 commented Sep 12, 2020

Hi,

I also have this weird issue. Mine is dual monitor setup, and when I use the hotkey, the window does not move to the left monitor.
Here is a video demonstration.

https://youtu.be/0u75o0Y6vpQ

@bjonnh
Copy link

bjonnh commented Sep 13, 2020

@hovo1990 I have the exact same issue as yours, please check #522 or create another issue if you think it is not related.

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

Successfully merging a pull request may close this issue.

5 participants