Skip to content

feat(dock-panel): add internal resizing and simplify component #961 - #995

Open
kristjan-prints-industry62 wants to merge 1 commit into
TEDI-Design-System:rcfrom
kristjan-prints-industry62:feat/961-dock-panel-update
Open

kristjan-prints-industry62 wants to merge 1 commit into
TEDI-Design-System:rcfrom
kristjan-prints-industry62:feat/961-dock-panel-update

Conversation

@kristjan-prints-industry62

Copy link
Copy Markdown
Contributor

Related to #961

Simplifies DockPanel’s implementation and API, and improves application integration.

  • Clean up code and naming; remove custom focus management.
  • Add optional internal resizing using the unchanged Resizer, preserving custom rail, gap and layer settings.
  • Add minimizeCounter, retaining controlled collapse.
  • Place Close above Restore in minimized side panels.
  • Remove external ref forwarding, generic HTML attribute passthrough and custom accessible-name overrides.
  • Update documentation, Storybook examples and tests.

Validation

  • TypeScript, lint/format, library build and Storybook build passed.
  • All 96 DockPanel tests passed after the final fix.
  • Browser checks passed for placements, presets, resizing, state preservation and layout. All four placements were rechecked after the final fix.
  • Full suite before the final fix: 2,406/2,407 passed. One Calendar test failed because it expected Jul 2025 but received juuli 2025.

Mobile integration remains postponed until the new TEDI-Ready Sheet supports external collapse control and preserves content when minimized.

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: TEDI-Design-System/react/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 075901ab-f76c-4d7d-8a86-b5d222438560
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

This branch has not been deployed

No deployments
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