Skip to content

Allow dragging ingameWindows with middle mouse button#1893

Open
Farmer-Markus wants to merge 2 commits intoReturn-To-The-Roots:masterfrom
Farmer-Markus:master
Open

Allow dragging ingameWindows with middle mouse button#1893
Farmer-Markus wants to merge 2 commits intoReturn-To-The-Roots:masterfrom
Farmer-Markus:master

Conversation

@Farmer-Markus
Copy link
Contributor

Just a small idea to let users drag ingame windows with the middle mouse button.
Very useful when playing on a laptop

Flow86
Flow86 previously approved these changes Feb 16, 2026
@Flow86
Copy link
Member

Flow86 commented Feb 16, 2026

Useful extension, especially since you do not need to find the (relatively) small title bar to move the window around.

Approved

Copy link
Member

@Flamefire Flamefire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks useful indeed.
Just 2 minor comments to help maintainability. Not fully sure about the return value (3rd comment) but looks odd to unconditionally return false, even though I'm aware we aren't consistent with those.

@Spikeone
Copy link
Member

I still think this should be an option you can disable - at least that way I hope that people who, for whatever reason don't want this, aren't bothered and maybe it's a bit more prominent to find and actually know that it is possible at all.

@Flamefire
Copy link
Member

IMO it is so unintrusive we don't need an option: Zoom is the only other use for the mousewheel we have so I doubt people will unintentionally use this
Not sure if we have documentation for the zoom function, but if we could add this there. If not we should likely add both

Copy link
Member

@Flamefire Flamefire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small bug in WinAPI and small suggestions. Would also merge without those though.

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 this pull request may close these issues.

4 participants