Skip to content

feat(clickgui): auto-align panels on background double-click#8627

Open
M4h45amu7x wants to merge 2 commits into
CCBlueX:nextgenfrom
M4h45amu7x:feat/clickgui-auto-align
Open

feat(clickgui): auto-align panels on background double-click#8627
M4h45amu7x wants to merge 2 commits into
CCBlueX:nextgenfrom
M4h45amu7x:feat/clickgui-auto-align

Conversation

@M4h45amu7x

@M4h45amu7x M4h45amu7x commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Double-click empty space in the ClickGUI to auto-arrange all panels toward that spot:

  • left / right = vertical column on that side
  • center = grouped near the top under the search bar

Panels glide into place and dragging still works like before.

@liquidbounce-support-bot

Copy link
Copy Markdown
Contributor

Thanks for the PR, @M4h45amu7x — nice quality-of-life idea, and the drag-to-still-work handling looks thought through.

A couple of things for the reviewers to weigh in on (not blockers, just flags):

  • Coordinate space / scaling: arrangePanels mixes catcherElement.offsetWidth/offsetHeight and each panel's getSize() (real layout px) with setPosition, which writes into panelConfig.left/top — a space the drag code scales by 2 / $scaleFactor. If the ClickGUI is rendered at a non-default scaleFactor, the measured area size and the stored positions may not be in the same units, so panels could land off-screen or overlap. Worth testing at a couple of GUI scales.
  • Full-screen catcher: .align-catcher is position: fixed; inset: 0. Please double-check it doesn't swallow double-clicks intended for anything sitting at the base z-index, and that it behaves under the ClickGUI's own transform.
  • Discoverability: a double-click gesture with no visible hint is easy to miss — might be worth a mention in the ClickGUI docs page if this lands.

I couldn't find an existing issue tracking this, so it looks new rather than a duplicate. Leaving the actual review to the maintainers.


🤖 Automated support reply — a human maintainer will review if this doesn't help.

@M4h45amu7x
M4h45amu7x force-pushed the feat/clickgui-auto-align branch from 296b482 to c430de0 Compare July 9, 2026 15:47
@M4h45amu7x
M4h45amu7x force-pushed the feat/clickgui-auto-align branch from c430de0 to 9283f7b Compare July 9, 2026 15:54
@M4h45amu7x
M4h45amu7x marked this pull request as ready for review July 9, 2026 15:56
…-align

# Conflicts:
#	src-theme/src/routes/clickgui/ClickGui.svelte
#	src-theme/src/routes/clickgui/Panel.svelte
#	src-theme/src/routes/clickgui/clickgui_store.ts
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.

1 participant