-
Notifications
You must be signed in to change notification settings - Fork 63
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
Table cell overflow [LG-3952] #2175
Table cell overflow [LG-3952] #2175
Conversation
🦋 Changeset detectedLatest commit: 0ba8d2a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Size Change: +419 B (0%) Total Size: 1.23 MB
ℹ️ View Unchanged
|
@@ -198,7 +199,7 @@ export const LiveExample: StoryFn<StoryTableProps> = args => { | |||
<Row key={row.id} row={row}> | |||
{row.getVisibleCells().map(cell => { | |||
return ( | |||
<Cell key={cell.id}> | |||
<Cell key={cell.id} id={cell.id} overflow="truncate"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we add an explicit story to test this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think we need a whole new story. I've set up the makeData
function to make sure it generates at least one row of long content
3030063
into
LG-3439-cell-max-height-2
* change to super high value * change height value * changeset * docs * apply disableAnimations prop down to subcomponents * tests * tests * fix animation jumpiness * remove transition function * rename spec describe * measure content height with ref * apply ref height calculation to cell transition * Update Tooltip.spec.tsx * Table cell overflow [LG-3952] (#2175) * Update makeData.testutils.tsx * Adds `overflow` prop. Styles truncated text * Update InternalCell.tsx * import correct styles * Create wicked-impalas-battle.md * contentClassName prop * fix max-height default --------- Co-authored-by: Adam Thompson <[email protected]> Co-authored-by: Adam Thompson <[email protected]> Co-authored-by: Brooke Scarlett Yalof <[email protected]>
✍️ Proposed changes
Adds
overflow
prop toCell
component. By default there is no change. Whenoverflow === 'truncate'
, the styling of the cell is updated (if necessary) to be aligned to the top, with an ellipsis after 2 lines of text.🎟 Jira ticket:
LG-3952
✅ Checklist
For bug fixes, new features & breaking changes
yarn changeset
and documented my changes