Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] feat: new CardList component #6246

Merged
merged 27 commits into from
Jul 25, 2023
Merged

Conversation

CPerinet
Copy link
Contributor

@CPerinet CPerinet commented Jun 26, 2023

Demo

2023-07-25 15 30 32


Archive

Based on #6245 (edit: that PR has merged ✅)

Original screenshot:

Screenshot 2023-06-26 at 4 21 09 PM

@CPerinet CPerinet changed the title Cp/card list Card list Jun 26, 2023
@CPerinet CPerinet marked this pull request as ready for review July 12, 2023 10:49
}
onChange={this.toggleUseSectionContainer}
/>
<H5 className={classNames({ [Classes.TEXT_MUTED]: !useSectionContainer })}>SectionCard</H5>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this modifier currently has no effect, but I'm fixing that in a separate PR (I was surprised it didn't work): #6304

@adidahiya
Copy link
Contributor

Looks good in latest docs preview :shipit:

@adidahiya adidahiya changed the title Card list [core] feat: new CardList component Jul 25, 2023
@adidahiya adidahiya merged commit 5721ea6 into palantir:develop Jul 25, 2023
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.

2 participants