Skip to content

Commit

Permalink
Remove content wrapper class styles to not interfere with text-overflow
Browse files Browse the repository at this point in the history
  • Loading branch information
oandregal committed Jan 15, 2024
1 parent 16e85b0 commit 6b3b92c
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 33 deletions.
10 changes: 0 additions & 10 deletions packages/dataviews/src/style.scss
Expand Up @@ -168,16 +168,6 @@
padding-left: $grid-unit-05;
}
}
tbody {
td {
vertical-align: top;
}
.dataviews-view-table__cell-content-wrapper {
min-height: $grid-unit-40;
display: flex;
align-items: center;
}
}
.dataviews-view-table-header-button {
padding: $grid-unit-05 $grid-unit-10;
font-size: 11px;
Expand Down
39 changes: 16 additions & 23 deletions packages/dataviews/src/view-table.js
Expand Up @@ -536,21 +536,17 @@ function ViewTable( {
minWidth: 20,
} }
>
<span className="dataviews-view-table__cell-content-wrapper">
<SingleSelectionCheckbox
id={
getItemId( item ) || index
}
item={ item }
selection={ selection }
onSelectionChange={
onSelectionChange
}
getItemId={ getItemId }
data={ data }
primaryField={ primaryField }
/>
</span>
<SingleSelectionCheckbox
id={ getItemId( item ) || index }
item={ item }
selection={ selection }
onSelectionChange={
onSelectionChange
}
getItemId={ getItemId }
data={ data }
primaryField={ primaryField }
/>
</td>
) }
{ visibleFields.map( ( field ) => (
Expand All @@ -565,14 +561,11 @@ function ViewTable( {
} }
>
<span
className={ classnames(
'dataviews-view-table__cell-content-wrapper',
{
'dataviews-view-table__primary-field':
primaryField?.id ===
field.id,
}
) }
className={ classnames( {
'dataviews-view-table__primary-field':
primaryField?.id ===
field.id,
} ) }
>
{ field.render( {
item,
Expand Down

0 comments on commit 6b3b92c

Please sign in to comment.