Skip to content

fix(mantine, collection): align horizontal rows on baseline - #4635

Closed
GermainBergeron wants to merge 2 commits into
masterfrom
ADUI-11633-collection-baseline-alignment
Closed

GermainBergeron wants to merge 2 commits into
masterfrom
ADUI-11633-collection-baseline-alignment

Conversation

@GermainBergeron

@GermainBergeron GermainBergeron commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Proposed Changes

In the horizontal Collection layout, a validation error no longer pushes the rest of the row down. Rows align on baseline, and the drag handle and trash button stay centered on the inputs.

New hidden CollectionValidationErrors story for Chromatic.

Potential Breaking Changes

None. Legacy layout handle and trash move 3px down.

Acceptance Criteria

  • The proposed changes are covered by unit tests (CSS only, covered by Chromatic)
  • The potential breaking changes are clearly identified
  • A changeset is added for releasable changes, following CONTRIBUTING.md
  • README.md is adjusted to reflect the proposed changes (if relevant)

@svcsnykcoveo

svcsnykcoveo commented Oct 7, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Release Preview

This PR would bump 1 package version if it were merged and released from master.

Patch

Package Next
@coveord/plasma-mantine 61.3.3

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Live demo of your latest successful build available here:

@GermainBergeron GermainBergeron changed the title fix(mantine, collection): align horizontal rows on baseline (ADUI-11633) fix(mantine, collection): align horizontal rows on baseline Oct 8, 2026
@GermainBergeron
GermainBergeron marked this pull request as ready for review October 8, 2026 00:17
@GermainBergeron
GermainBergeron requested a review from a team as a code owner October 8, 2026 00:17
@GermainBergeron
GermainBergeron requested review from FelixBlaisThon and gdostie and a balanced review from Copilot and removed request for a team October 8, 2026 00:17

Copilot AI 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.

🟢 Approval recommended

The CSS changes are scoped appropriately and the Chromatic story covers the reported alignment regression.

0 open findings

What changed in this PR

Fixes horizontal Collection row alignment when validation errors appear.

Changes:

  • Baseline-aligns horizontal rows and offsets action controls.
  • Adds a hidden Chromatic validation-error story.
  • Adds a patch changeset.
File Description
Collection.stories.tsx Adds validation examples and visual coverage.
VerticalLayout.module.css Preserves vertical control positioning.
HorizontalLayout.module.css Baseline-aligns horizontal rows.
Collection.module.css Centers row action controls.
.changeset/​collection-row-baseline.md Documents the user-facing fix.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

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.

3 participants