Skip to content

Make SidePanel width adjustable by dragging its inner edge - #73

Merged
methodofaction merged 2 commits into
mainfrom
side-panel-resizable
Sep 1, 2026
Merged

Make SidePanel width adjustable by dragging its inner edge#73
methodofaction merged 2 commits into
mainfrom
side-panel-resizable

Conversation

@migueltorresvalls

Copy link
Copy Markdown
Contributor

Detail panels often need more room than their fixed width allows — the SidePanel's inner edge is now a drag handle that resizes it, with an accent-green line on hover and while dragging.

Changes:

  • Resize handle on the panel's inner edge; hover:bg-border-selected-bold highlight, kept green while dragging
  • sidePanel Alpine controller gains width/resizing state and startResize, clamped to [320px, viewport − 320px]
  • Width prop is now the initial width; the adjusted width survives open/close cycles
  • Docs API tables and prop comments updated

Generated with Claude Code — https://claude.ai/code/1abc88c1-6a00-4dc1-abc9-b2e7a2b175c6

The panel's inner edge is now a drag handle that shows an accent-green
line on hover and while dragging. Width is clamped to [320, viewport-320]
and the adjusted width survives open/close cycles.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@netlify

netlify Bot commented Aug 26, 2026

Copy link
Copy Markdown

Deploy Preview for popui-go ready!

Name Link
🔨 Latest commit f80ec0a
🔍 Latest deploy log https://app.netlify.com/projects/popui-go/deploys/6a96fdc83aebfc0008012514
😎 Deploy Preview https://deploy-preview-73--popui-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@methodofaction methodofaction left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This has more implications than is apparent: we probably want to save the width but only for certain contexts (suppose you widen the Peppol launcher sidebar, but then move to France, do you expect the sidebar to change on both launchers). No response needed, this is for consideration.

@mariocnovoa
mariocnovoa self-requested a review September 1, 2026 16:22
@methodofaction

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	assets/popui.css

Co-authored-by: methodofaction <51445+methodofaction@users.noreply.github.com>

Copilot AI commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved — merged origin/main into this branch (commit f80ec0a). The only conflict was in the generated assets/popui.css; main's version already included all the classes added by this PR (bg-border-selected-bold, cursor-col-resize), so I accepted it.

Copilot AI requested a review from methodofaction September 1, 2026 16:31
@methodofaction
methodofaction merged commit 1e31c3c into main Sep 1, 2026
4 checks passed
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