Skip to content

Splitter component#679

Merged
itsJohnnyGrid merged 8 commits into
betafrom
splitter-component
May 13, 2026
Merged

Splitter component#679
itsJohnnyGrid merged 8 commits into
betafrom
splitter-component

Conversation

@KinduD21
Copy link
Copy Markdown
Collaborator

@KinduD21 KinduD21 requested a review from itsJohnnyGrid May 11, 2026 14:56
@KinduD21 KinduD21 self-assigned this May 11, 2026
KinduD21 added 2 commits May 11, 2026 18:01
- Enhanced gutter styles with rounded corners and focus-visible outlines.
- Removed gutterColor prop from Props interface and related usages.
- Cleaned up USplitter component by eliminating unused divider attributes.
- Updated storybook examples to reflect changes.
Comment thread src/ui.container-splitter/config.ts Outdated
},
},
panel: {
base: "",
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.

panel: ""

Comment thread src/ui.container-splitter/config.ts Outdated
disabled: false,
stateKey: null,
stateStorage: "session",
step: 5,
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.

resizeStep: 5,

},
defaults: {
orientation: "horizontal",
gutterSize: 8,
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.

minSizes, maxSizes

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No need to add default values in config, because minSizes and maxSizes are possibly arrays, so they should be added into withDefaults same as modelValue

image

Comment thread src/ui.container-splitter/config.ts Outdated
},
},
defaults: {
orientation: "horizontal",
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.

better to have boolean vertical (false by default, that mean apply classes from horisontal)

- Changed orientation prop to vertical boolean for clearer semantics.
- Updated related styles and computed properties to reflect the new vertical layout.
- Adjusted storybook examples and tests to accommodate the new prop structure.
@itsJohnnyGrid itsJohnnyGrid merged commit f86974a into beta May 13, 2026
10 checks passed
@itsJohnnyGrid itsJohnnyGrid deleted the splitter-component branch May 13, 2026 10:46
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.

2 participants