Skip to content

Don't show border if useSectionContainer is true#7848

Open
mud wants to merge 2 commits intodevelopfrom
mud/fix-7841
Open

Don't show border if useSectionContainer is true#7848
mud wants to merge 2 commits intodevelopfrom
mud/fix-7841

Conversation

@mud
Copy link
Contributor

@mud mud commented Mar 9, 2026

Fixes #7841

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

Remove unnecessary border for CardList playground when contained in a Section.

Reviewers should focus on:

I initially opened the issue as a UI bug, but realized CardList does have a bordered prop to control bordering. Personally, I think embedding a CardList as a direct child should remove the border, but I can also understand that these are consumer responsibility.

Screenshot

Screenshot 2026-03-09 at 12 43 07

@changelog-app
Copy link

changelog-app bot commented Mar 9, 2026

Generate changelog in packages/docs-app/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

Don't show border if useSectionContainer is true

Check the box to generate changelog(s)

  • Generate changelog entry

@mud mud requested a review from ggdouglas March 9, 2026 19:49
@changelog-app
Copy link

changelog-app bot commented Mar 9, 2026

Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.


📋Changelog Preview

🐛 Fixes

  • Don't show border if useSectionContainer is true (#7848)

@svc-palantir-github
Copy link

Don't show border if useSectionContainer is true

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

@svc-palantir-github
Copy link

Merge branch 'develop' into mud/fix-7841

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

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

3 participants