Skip to content

feat: [SC-79340] Add card view to GridTableLayout#1172

Open
roberthdg wants to merge 8 commits intomainfrom
sc-79340/add-card-view
Open

feat: [SC-79340] Add card view to GridTableLayout#1172
roberthdg wants to merge 8 commits intomainfrom
sc-79340/add-card-view

Conversation

@roberthdg
Copy link
Contributor

@roberthdg roberthdg commented Jan 2, 2026

Demo:

Screen.Recording.2026-01-02.at.6.56.01.PM.mov

)}
<div css={Css.df.gap1.$}>
{cardView && (
<ButtonGroup
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The original design has a dropdown button that changes the icon based on the selected option.

Image

I’m not sure we have an out-of-the-box component with that functionality, and it also requires an extra click for users to toggle between views... So I went with this approach. We could add it as a follow-up if needed.

Copy link

Choose a reason for hiding this comment

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

@roberthdg Yea I don't believe we do have a component like this so it might need to be added as a variant to Select. The reason I went with a Select dropdown for this instead of using the Button Group component is to be consistent on other pages where we need more 'View' options and a toggle wouldn't work, like on Schedules here (List, Cal, Gantt, Milestone, Lookahead, etc):
Screenshot 2026-01-08 at 10 48 28 AM. Although I'm not sure when we'll be getting to updating the Schedule page, so maybe this is ok for now. I'll throw this into the Beam slack channel as a heads up for other folks using this component.

Copy link

Choose a reason for hiding this comment

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

Posted in Beam. Let's stick with this toggle you have for now since Schedules is a one-off anyway and it the only one.

@roberthdg roberthdg marked this pull request as ready for review January 2, 2026 22:54
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