fix(ui): adjust dropdown icon right-gap in ComboBox and Select#1653
fix(ui): adjust dropdown icon right-gap in ComboBox and Select#1653MartinS-git wants to merge 8 commits intomainfrom
Conversation
🦋 Changeset detectedLatest commit: ede80cc The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
c64cb05 to
495e2e1
Compare
There was a problem hiding this comment.
Pull request overview
This PR addresses a UI consistency issue in @cloudoperators/juno-ui-components by aligning the dropdown chevron right-gap in ComboBox and Select with the Figma specs, and ensuring the ComboBox portal renders with the correct theme styles in a specific Storybook story.
Changes:
- Updated
Selecttoggle padding (px-4→pl-4 pr-2) to reduce the chevron’s right-gap. - Repositioned the
ComboBoxtoggle button and adjusted validation icon container positioning to match the desired spacing. - Wrapped
DataGridToolbar’sComplexCustomLayoutstory inPortalProviderto ensure portaled dropdown content is styled correctly in Storybook.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/ui-components/src/components/Select/Select.component.tsx | Tweaks toggle padding to match chevron spacing to Figma. |
| packages/ui-components/src/components/DataGridToolbar/DataGridToolbar.stories.tsx | Adds PortalProvider decorator so portaled dropdown renders with correct theme styles in Storybook. |
| packages/ui-components/src/components/ComboBox/ComboBox.component.tsx | Repositions toggle button and validation icon container to adjust right-gap. |
| .changeset/fix-dropdown-icon-gap.md | Adds changeset entry for a patch release of the spacing fix. |
Comments suppressed due to low confidence (1)
packages/ui-components/src/components/ComboBox/ComboBox.component.tsx:117
invalidButtonStyles/validButtonStylesonly setjn:border-theme-*, butbuttonStylesno longer includes anyjn:border*classes. As a result, the border color classes on the toggle won’t have any visible effect. Either remove the invalid/valid toggle border styling entirely, or reintroduce an explicit border (width/sides) on the toggle if you still want the toggle edge to reflect valid/invalid state.
const invalidButtonStyles = `
jn:border-theme-error
`
const validButtonStyles = `
jn:border-theme-success
`
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
packages/ui-components/src/components/ComboBox/ComboBox.component.tsx:117
invalidButtonStyles/validButtonStylesonly set a border color, butbuttonStylesno longer applies any border width/style. This makes those state style constants (and thejuno-combobox-toggle-*(...)class concatenations) effectively no-ops. Consider either reintroducing a border on the toggle button (if it’s meant to visually share the input border) or removing these unused style constants to reduce confusion.
const invalidButtonStyles = `
jn:border-theme-error
`
const validButtonStyles = `
jn:border-theme-success
`
123c604
4a0482a to
f5a184b
Compare
- ComboBox: reposition toggle button (top-[1px], right-2, h-[calc(100%-2px)]), remove border styles, shift iconContainerStyles to right-8 - Select: reduce toggle right padding (px-4 -> pl-4 pr-2) - DataGridToolbar: wrap ComplexCustomLayout in PortalProvider for correct ComboBox dropdown theme styles Signed-off-by: Martin Schulze <martin.schulze04@sap.com> Signed-off-by: MartinS-git <info@eyepic.de>
Signed-off-by: Martin Schulze <martin.schulze04@sap.com> Signed-off-by: MartinS-git <info@eyepic.de>
…n ComboBox Replace static px-4 with dynamic right padding: pr-8 for default state, pr-16 when valid or invalid icons are visible. Signed-off-by: MartinS-git <info@eyepic.de>
Replaces top-[1px]/h-[calc(100%-2px)] offset approach with a full-height button that carries its own border. Fixes defaultButtonStyles not being applied to the toggle. Also reformats DateTimePicker and Select interfaces. Signed-off-by: MartinS-git <info@eyepic.de>
…d Select Signed-off-by: MartinS-git <info@eyepic.de>
Signed-off-by: MartinS-git <info@eyepic.de>
Signed-off-by: MartinS-git <info@eyepic.de>
f5a184b to
ce0ed83
Compare
Signed-off-by: MartinS-git <info@eyepic.de>
Closes #1639
Summary
top-[1px]/h-[calc(100%-2px)]) with a full-height button that carries explicit borders (border-y,border-r,border-l-0), ensuringdefaultButtonStyles/invalidButtonStyles/validButtonStylesare consistently appliedpx-4with dynamic right padding (pr-8default,pr-16when valid/invalid)px-4→pl-4 pr-2) for consistent chevron gapComplexCustomLayoutinPortalProviderso the ComboBox dropdown renders with correct theme styles in StorybookTest plan
ComplexCustomLayoutstory