sublimehq / sublime_text

Issue tracker for Sublime Text
https://www.sublimetext.com
800 stars 35 forks source link

Dragging tabs between screens of different dpi glitches #6308

Open titoBouzout opened 4 months ago

titoBouzout commented 4 months ago

Description of the bug

Primary screen on the laptop: 2k 150% 2560x1440 Secondary vertical screen: 100% 1080x1920

Dragging the tab from the secondary screen to the primary screen glitches Dragging the tab from the primary screen to the secondary screen glitches

This makes it very difficult to drag a tab to an existing window.

Steps to reproduce

  1. have two screens with different dpi
  2. have two sublime text open on both screens
  3. drag one tab to the other screen

Expected behavior

-

Actual behavior

-

Sublime Text build number

4172

Operating system & version

Windows 10

(Linux) Desktop environment and/or window manager

No response

Additional information

from 0:00 I'm trying to drag to my primary screen from 0:24 I'm trying to drag to my secondary screen https://github.com/sublimehq/sublime_text/assets/64156/de23a63f-b258-4198-8bb4-cb4a62c8df58

OpenGL context information

No response

BenjaminSchaaf commented 4 months ago

Fixed in build 4173.

titoBouzout commented 4 months ago

Thanks Benjamin, the tab follows the cursor and draws correctly, but the drop target is messed up.

Dragging Tab from "screen 2 - 100% 1080x1920" To "screen 1 - 2k 150% 2560x1440"

Dragging Tab from "screen 1 - 2k 150% 2560x1440" high dpi To "screen 2 - 100% 1080x1920"

Dragging Tab to very same Screen the Tab is dragged from:

https://github.com/sublimehq/sublime_text/assets/64156/01fdb20a-65da-4675-a5a0-bb444daa15fd

titoBouzout commented 4 months ago

@BenjaminSchaaf do you want me to enter this issue as a new one? I still consider it the same one, but I could enter a new one specific for "drop target"

BenjaminSchaaf commented 4 months ago

No need to open a new one.

titoBouzout commented 3 months ago

@BenjaminSchaaf maybe the buffer could be added as drop target? that would make it easier to move tabs between panes, and will probably make it work moving tabs between screens.

BenjaminSchaaf commented 2 months ago

Should be fixed in build 4174.

titoBouzout commented 2 months ago

Thanks! Did some quick tests and everything works great between editors. There's 1 remaining issue, and two other comments that I am not sure about it.

Issue:

Feature:

Fancy Zones:

The first issue is related to the recent improvements to tab drag and dropping. The other two idk, I can add them as separated issues. Let me know, thanks!

titoBouzout commented 2 months ago

@BenjaminSchaaf just ccing you in case it has been missed

BenjaminSchaaf commented 2 months ago

Reopening for that issue. The rest I'd ask you to report separately, though extending the drop target to the whole buffer is unlikely to ever happen for various reasons.

titoBouzout commented 2 months ago

ok, thanks, reopening for :

titoBouzout commented 1 month ago

@BenjaminSchaaf entered https://github.com/sublimehq/sublime_text/issues/6387 for fancy zones, and reopened this one for the tab dragging to a screen that doesn't have a Sublime Text instance running.