Skip to content

[Regression] Fix @Links layout rendering for grid styles#1021

Open
mportiz08 wants to merge 1 commit into
swiftlang:mainfrom
mportiz08:fix-default-align-items-for-column
Open

[Regression] Fix @Links layout rendering for grid styles#1021
mportiz08 wants to merge 1 commit into
swiftlang:mainfrom
mportiz08:fix-default-align-items-for-column

Conversation

@mportiz08

Copy link
Copy Markdown
Contributor

Bug/issue #, if applicable: 179413082

Summary

#1018 introduces a regression for the way that columns are rendered within the @Links DocC directive when the visual style is either compactGrid or detailedGrid.

This happens because the new align-items property has a default of flex-start, which has unintended consequences for the way that the TopicsLinkCardGrid component composes the Column component without setting an explicit alignment.

By changing the default value from flex-start to normal, we can fix the unexpected rendering for the @Links directive without impacting the new enhancement to support the alignment property for @Column.

Testing

Steps:

  1. Create or test some DocC content that utilizes @PageImage and @Links(visualStyle: detailedGrid)
  2. Verify that the grid items rendered for the links continue to look the way they should, with images appearing in a fixed grid pattern.
  3. Verify that other content using @Column(alignment: leading|center|trailing) still renders with the expected alignment for its items as implemented in Add support for optional alignment property on row columns #1018

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests (CSS-only change)
  • Ran npm test, and it succeeded
  • Updated documentation if necessary

@mportiz08 mportiz08 requested a review from marinaaisa as a code owner June 11, 2026 23:12
@mportiz08 mportiz08 changed the title [Regression] Fix @Topics layout rendering for grid styles [Regression] Fix @Links layout rendering for grid styles Jun 11, 2026
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