Skip to content

Commit

Permalink
Remove empty descriptions
Browse files Browse the repository at this point in the history
  • Loading branch information
youknowriad committed Jul 8, 2024
1 parent 4287bd8 commit 1680214
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 30 deletions.
26 changes: 16 additions & 10 deletions packages/dataviews/src/view-table.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -360,17 +360,23 @@ function TableColumnField< Item >( {
item,
field,
}: TableColumnFieldProps< Item > ) {
const value = field.render( {
item,
} );
return (
<div
className={ clsx( 'dataviews-view-table__cell-content-wrapper', {
'dataviews-view-table__primary-field':
primaryField?.id === field.id,
} ) }
>
{ field.render( {
item,
} ) }
</div>
!! value && (
<div
className={ clsx(
'dataviews-view-table__cell-content-wrapper',
{
'dataviews-view-table__primary-field':
primaryField?.id === field.id,
}
) }
>
{ value }
</div>
)
);
}

Expand Down
26 changes: 6 additions & 20 deletions packages/edit-site/src/components/page-templates/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,7 @@ import clsx from 'clsx';
/**
* WordPress dependencies
*/
import {
Icon,
__experimentalText as Text,
__experimentalHStack as HStack,
VisuallyHidden,
} from '@wordpress/components';
import { Icon, __experimentalHStack as HStack } from '@wordpress/components';
import { __ } from '@wordpress/i18n';
import { useState, useMemo, useCallback, useEffect } from '@wordpress/element';
import { useEntityRecords } from '@wordpress/core-data';
Expand Down Expand Up @@ -301,20 +296,11 @@ export default function PageTemplates() {
header: __( 'Description' ),
id: 'description',
render: ( { item } ) => {
return item.description ? (
<span className="page-templates-description">
{ decodeEntities( item.description ) }
</span>
) : (
view.type === LAYOUT_TABLE && (
<>
<Text variant="muted" aria-hidden="true">
&#8212;
</Text>
<VisuallyHidden>
{ __( 'No description.' ) }
</VisuallyHidden>
</>
return (
item.description && (
<span className="page-templates-description">
{ decodeEntities( item.description ) }
</span>
)
);
},
Expand Down

0 comments on commit 1680214

Please sign in to comment.