Skip to content

Feature: allow configuring debug button position - #1009

Merged
srzeszut merged 13 commits into
mainfrom
998-allow-configuring-button-position
Oct 9, 2026
Merged

srzeszut merged 13 commits into
mainfrom
998-allow-configuring-button-position

Conversation

@srzeszut

Copy link
Copy Markdown
Contributor

No description provided.

@srzeszut srzeszut linked an issue Sep 11, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The source map is stale and the resize-reset behavior lacks regression coverage.

1 open finding
What changed in this PR

Adds CSS customization for the debug button’s position through a shadow part.

Changes:

  • Exposes the debug button as ::part(debug-button).
  • Adds a stable shadow-host ID and viewport-reset behavior.
  • Documents and tests custom positioning.
File Description
assets/​client/​client.js Identifies the shadow host.
assets/​client/​components/​debug_button/​debug_button.html Exposes the button part.
assets/​client/​components/​debug_button/​debug_button.js Restores CSS positioning after invalid drag offsets.
docs/​config.md Documents position customization.
e2e/​tests/​debug-button-part.spec.ts Tests shadow-part styling.
priv/​static/​client.js Updates the generated client bundle.
priv/​static/​client.js.map Updates generated source mappings.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread e2e/tests/debug-button-part.spec.ts Resolved
@srzeszut
srzeszut requested a balanced review from Copilot October 9, 2026 10:24
@srzeszut
srzeszut marked this pull request as ready for review October 9, 2026 10:25
@srzeszut
srzeszut requested a review from hhubert6 October 9, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The implementation is focused, documented, and covered by an end-to-end test.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced

@hhubert6 hhubert6 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Neat 🐬

@srzeszut
srzeszut merged commit 4d9253a into main Oct 9, 2026
5 checks passed
srzeszut added a commit that referenced this pull request Oct 9, 2026
* allow configure debug button position

* add doc

* add test

* update mix.lock

* format

* add precommit

* fix precommit

* fix 1.16 formating

* format

* remove format from workflow

* simplify and remove unecessary changes

* cr suggerstions
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.

Allow configuring button position

4 participants