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

DataViews: Rename the header property of fields to label #63843

Merged
merged 1 commit into from
Jul 23, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/dataviews/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

- `onSelectionChange` prop has been renamed to `onChangeSelection` and its argument has been updated to be a list of ids.
- `setSelection` prop has been removed. Please use `onChangeSelection` instead.
- `header` field property has been renamed to `label`.

## 3.0.0 (2024-07-10)

Expand Down
4 changes: 2 additions & 2 deletions packages/dataviews/src/components/dataform/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function DataFormTextControl< Item >( {
field,
onChange,
}: DataFormControlProps< Item > ) {
const { id, header, placeholder } = field;
const { id, label, placeholder } = field;
const value = field.getValue( { item: data } );

const onChangeControl = useCallback(
Expand All @@ -47,7 +47,7 @@ function DataFormTextControl< Item >( {

return (
<TextControl
label={ header }
label={ label }
placeholder={ placeholder }
value={ value }
onChange={ onChangeControl }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function Filters() {
const isPrimary = !! field.filterBy?.isPrimary;
filters.push( {
field: field.id,
name: field.header,
name: field.label,
elements: field.elements,
singleSelection: operators.some( ( op ) =>
[ OPERATOR_IS, OPERATOR_IS_NOT ].includes( op )
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ function FieldsVisibilityMenu< Item >( {
} }
>
<DropdownMenuItemLabel>
{ field.header }
{ field.label }
</DropdownMenuItemLabel>
</DropdownMenuCheckboxItem>
);
Expand Down Expand Up @@ -212,7 +212,7 @@ function SortMenu< Item >( {
<DropdownMenuItem
suffix={
<span aria-hidden="true">
{ currentSortedField?.header }
{ currentSortedField?.label }
</span>
}
>
Expand All @@ -230,7 +230,7 @@ function SortMenu< Item >( {
trigger={
<DropdownMenuItem>
<DropdownMenuItemLabel>
{ field.header }
{ field.label }
</DropdownMenuItemLabel>
</DropdownMenuItem>
}
Expand Down
2 changes: 1 addition & 1 deletion packages/dataviews/src/layouts/grid/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ function GridItem< Item >( {
>
<>
<FlexItem className="dataviews-view-grid__field-name">
{ field.header }
{ field.label }
</FlexItem>
<FlexItem
className="dataviews-view-grid__field-value"
Expand Down
2 changes: 1 addition & 1 deletion packages/dataviews/src/layouts/list/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ function ListItem< Item >( {
as="span"
className="dataviews-view-list__field-label"
>
{ field.header }
{ field.label }
</VisuallyHidden>
<span className="dataviews-view-list__field-value">
<field.render item={ item } />
Expand Down
6 changes: 3 additions & 3 deletions packages/dataviews/src/layouts/table/column-header-menu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ const _HeaderMenu = forwardRef( function HeaderMenu< Item >(
);
const index = view.fields?.indexOf( fieldId ) as number;
if ( !! combinedField ) {
return combinedField.header;
return combinedField.label;
}
const field = fields.find( ( f ) => f.id === fieldId );
if ( ! field ) {
Expand All @@ -92,7 +92,7 @@ const _HeaderMenu = forwardRef( function HeaderMenu< Item >(
!! operators.length &&
! field.filterBy?.isPrimary;
if ( ! isSortable && ! isHidable && ! canAddFilter ) {
return field.header;
return field.label;
}
return (
<DropdownMenu
Expand All @@ -104,7 +104,7 @@ const _HeaderMenu = forwardRef( function HeaderMenu< Item >(
ref={ ref }
variant="tertiary"
>
{ field.header }
{ field.label }
{ view.sort && isSorted && (
<span aria-hidden="true">
{ sortArrows[ view.sort.direction ] }
Expand Down
2 changes: 1 addition & 1 deletion packages/dataviews/src/normalize-fields.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export function normalizeFields< Item >(

return {
...field,
header: field.header || field.id,
label: field.label || field.id,
getValue,
render: field.render || getValue,
};
Expand Down
12 changes: 6 additions & 6 deletions packages/dataviews/src/stories/fixtures.js
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ export const actions = [

export const fields = [
{
header: 'Image',
label: 'Image',
id: 'image',
render: ( { item } ) => {
return (
Expand All @@ -170,13 +170,13 @@ export const fields = [
enableSorting: false,
},
{
header: 'Title',
label: 'Title',
id: 'title',
enableHiding: false,
enableGlobalSearch: true,
},
{
header: 'Type',
label: 'Type',
id: 'type',
enableHiding: false,
elements: [
Expand All @@ -187,18 +187,18 @@ export const fields = [
],
},
{
header: 'Satellites',
label: 'Satellites',
id: 'satellites',
enableSorting: true,
},
{
header: 'Description',
label: 'Description',
id: 'description',
enableSorting: false,
enableGlobalSearch: true,
},
{
header: 'Categories',
label: 'Categories',
id: 'categories',
elements: [
{ value: 'Space', label: 'Space' },
Expand Down
6 changes: 3 additions & 3 deletions packages/dataviews/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ export type Field< Item > = {
/**
* The label of the field. Defaults to the id.
*/
header?: string;
label?: string;

/**
* Placeholder for the field.
Expand Down Expand Up @@ -116,7 +116,7 @@ export type Field< Item > = {
} );

export type NormalizedField< Item > = Field< Item > & {
header: string;
label: string;
getValue: ( args: { item: Item } ) => any;
render: ComponentType< { item: Item } >;
};
Expand Down Expand Up @@ -242,7 +242,7 @@ interface ViewBase {
export interface CombinedField {
id: string;

header: string;
label: string;

/**
* The fields to use as columns.
Expand Down
8 changes: 4 additions & 4 deletions packages/edit-site/src/components/page-patterns/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -284,15 +284,15 @@ export default function DataviewsPatterns() {
const fields = useMemo( () => {
const _fields = [
{
header: __( 'Preview' ),
label: __( 'Preview' ),
id: 'preview',
render: ( { item } ) => (
<Preview item={ item } viewType={ view.type } />
),
enableSorting: false,
},
{
header: __( 'Title' ),
label: __( 'Title' ),
id: 'title',
getValue: ( { item } ) => item.title?.raw || item.title,
render: ( { item } ) => <Title item={ item } />,
Expand All @@ -302,7 +302,7 @@ export default function DataviewsPatterns() {

if ( type === PATTERN_TYPES.user ) {
_fields.push( {
header: __( 'Sync status' ),
label: __( 'Sync status' ),
id: 'sync-status',
render: ( { item } ) => {
const syncStatus =
Expand Down Expand Up @@ -333,7 +333,7 @@ export default function DataviewsPatterns() {
} );
} else if ( type === TEMPLATE_PART_POST_TYPE ) {
_fields.push( {
header: __( 'Author' ),
label: __( 'Author' ),
id: 'author',
getValue: ( { item } ) => item.author_text,
render: ( { item } ) => {
Expand Down
10 changes: 5 additions & 5 deletions packages/edit-site/src/components/page-templates/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ const defaultLayouts = {
combinedFields: [
{
id: 'template',
header: __( 'Template' ),
label: __( 'Template' ),
children: [ 'title', 'description' ],
direction: 'vertical',
},
Expand Down Expand Up @@ -286,15 +286,15 @@ export default function PageTemplates() {
const fields = useMemo(
() => [
{
header: __( 'Preview' ),
label: __( 'Preview' ),
id: 'preview',
render: ( { item } ) => {
return <Preview item={ item } viewType={ view.type } />;
},
enableSorting: false,
},
{
header: __( 'Template' ),
label: __( 'Template' ),
id: 'title',
getValue: ( { item } ) => item.title?.rendered,
render: ( { item } ) => (
Expand All @@ -304,7 +304,7 @@ export default function PageTemplates() {
enableGlobalSearch: true,
},
{
header: __( 'Description' ),
label: __( 'Description' ),
id: 'description',
render: ( { item } ) => {
return (
Expand All @@ -319,7 +319,7 @@ export default function PageTemplates() {
enableGlobalSearch: true,
},
{
header: __( 'Author' ),
label: __( 'Author' ),
id: 'author',
getValue: ( { item } ) => item.author_text,
render: ( { item } ) => {
Expand Down
10 changes: 5 additions & 5 deletions packages/edit-site/src/components/post-fields/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -167,15 +167,15 @@ function usePostFields( viewType ) {
() => [
{
id: 'featured-image',
header: __( 'Featured Image' ),
label: __( 'Featured Image' ),
getValue: ( { item } ) => item.featured_media,
render: ( { item } ) => (
<FeaturedImage item={ item } viewType={ viewType } />
),
enableSorting: false,
},
{
header: __( 'Title' ),
label: __( 'Title' ),
id: 'title',
type: 'text',
getValue: ( { item } ) =>
Expand Down Expand Up @@ -233,7 +233,7 @@ function usePostFields( viewType ) {
enableHiding: false,
},
{
header: __( 'Author' ),
label: __( 'Author' ),
id: 'author',
getValue: ( { item } ) => item._embedded?.author[ 0 ]?.name,
elements:
Expand All @@ -244,7 +244,7 @@ function usePostFields( viewType ) {
render: PostAuthorField,
},
{
header: __( 'Status' ),
label: __( 'Status' ),
id: 'status',
getValue: ( { item } ) =>
STATUSES.find( ( { value } ) => value === item.status )
Expand All @@ -257,7 +257,7 @@ function usePostFields( viewType ) {
},
},
{
header: __( 'Date' ),
label: __( 'Date' ),
id: 'date',
render: ( { item } ) => {
const isDraftOrPrivate = [ 'draft', 'private' ].includes(
Expand Down
Loading