Skip to content

fix(Section): flatten CardList top radius inside SectionCard#8053

Open
Abdeltoto wants to merge 1 commit intopalantir:developfrom
Abdeltoto:fix/section-cardlist-top-border-7841
Open

fix(Section): flatten CardList top radius inside SectionCard#8053
Abdeltoto wants to merge 1 commit intopalantir:developfrom
Abdeltoto:fix/section-cardlist-top-border-7841

Conversation

@Abdeltoto
Copy link
Copy Markdown

@Abdeltoto Abdeltoto commented Apr 14, 2026

Fixes #7841

Checklist

  • Includes tests (N/A: SCSS-only visual fix; verify via Reviewers should focus steps)
  • Update documentation (N/A: internal styling change only)

Changelog

Unreleased entry: packages/core/changelog/@unreleased/pr-7841-section-cardlist.yml (type: fix).

Changes proposed in this pull request

When a bordered CardList is nested in a SectionCard, its outer wrapper is still a Card with the default border-radius, which looked wrong under the section header (rounded top corners and divider clash). This change sets border-top-left-radius and border-top-right-radius to 0 for a bordered CardList that is a direct child of SectionCard.

Reviewers should focus on

Visual check: Docs → CardListInteractive Playground, enable Use Section container (bordered is forced on). Confirm the list lines up cleanly with the section header in light and dark theme.

Bordered CardList keeps the default Card border-radius on its outer wrapper, which looked wrong under the section header (rounded top + divider clash). Zero out the top radii when the list is a direct child of SectionCard.

Fixes palantir#7841.

Made-with: Cursor
@changelog-app
Copy link
Copy Markdown

changelog-app Bot commented Apr 14, 2026

Generate changelog in packages/core/changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

fix(Section): flatten CardList top radius inside SectionCard

Check the box to generate changelog(s)

  • Generate changelog entry

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.

Unnecessary border when CardList is embedded in a Section

1 participant