diff --git a/packages/dataviews/src/dataviews-layouts/table/column-header-menu.tsx b/packages/dataviews/src/dataviews-layouts/table/column-header-menu.tsx index 763cf83b5c2f93..55cd8d867eff40 100644 --- a/packages/dataviews/src/dataviews-layouts/table/column-header-menu.tsx +++ b/packages/dataviews/src/dataviews-layouts/table/column-header-menu.tsx @@ -93,168 +93,171 @@ const _HeaderMenu = forwardRef( function HeaderMenu< Item >( ! field.filterBy?.isPrimary; return ( - - { header } - { view.sort && isSorted && ( - - ) } - - } - style={ { minWidth: '240px' } } - > - - { isSortable && ( - - { SORTING_DIRECTIONS.map( - ( direction: SortDirection ) => { - const isChecked = - view.sort && - isSorted && - view.sort.direction === direction; + + + } + > + { header } + { view.sort && isSorted && ( + + ) } + + + + { isSortable && ( + + { SORTING_DIRECTIONS.map( + ( direction: SortDirection ) => { + const isChecked = + view.sort && + isSorted && + view.sort.direction === direction; - const value = `${ fieldId }-${ direction }`; + const value = `${ fieldId }-${ direction }`; - return ( - { - onChangeView( { - ...view, - sort: { - field: fieldId, - direction, - }, - } ); - } } - > - - { sortLabels[ direction ] } - - - ); - } - ) } - - ) } - { canAddFilter && ( - - } - onClick={ () => { - setOpenedFilter( fieldId ); - onChangeView( { - ...view, - page: 1, - filters: [ - ...( view.filters || [] ), - { - field: fieldId, - value: undefined, - operator: operators[ 0 ], - }, - ], - } ); - } } - > - - { __( 'Add filter' ) } - - - - ) } - { ( canMove || isHidable ) && field && ( - - { canMove && ( + return ( + { + onChangeView( { + ...view, + sort: { + field: fieldId, + direction, + }, + } ); + } } + > + + { sortLabels[ direction ] } + + + ); + } + ) } + + ) } + { canAddFilter && ( + } - disabled={ index < 1 } + prefix={ } onClick={ () => { + setOpenedFilter( fieldId ); onChangeView( { ...view, - fields: [ - ...( visibleFieldIds.slice( - 0, - index - 1 - ) ?? [] ), - fieldId, - visibleFieldIds[ index - 1 ], - ...visibleFieldIds.slice( - index + 1 - ), + page: 1, + filters: [ + ...( view.filters || [] ), + { + field: fieldId, + value: undefined, + operator: operators[ 0 ], + }, ], } ); } } > - { __( 'Move left' ) } + { __( 'Add filter' ) } - ) } - { canMove && ( - } - disabled={ index >= visibleFieldIds.length - 1 } - onClick={ () => { - onChangeView( { - ...view, - fields: [ - ...( visibleFieldIds.slice( - 0, - index - ) ?? [] ), - visibleFieldIds[ index + 1 ], - fieldId, - ...visibleFieldIds.slice( - index + 2 + + ) } + { ( canMove || isHidable ) && field && ( + + { canMove && ( + } + disabled={ index < 1 } + onClick={ () => { + onChangeView( { + ...view, + fields: [ + ...( visibleFieldIds.slice( + 0, + index - 1 + ) ?? [] ), + fieldId, + visibleFieldIds[ index - 1 ], + ...visibleFieldIds.slice( + index + 1 + ), + ], + } ); + } } + > + + { __( 'Move left' ) } + + + ) } + { canMove && ( + } + disabled={ + index >= visibleFieldIds.length - 1 + } + onClick={ () => { + onChangeView( { + ...view, + fields: [ + ...( visibleFieldIds.slice( + 0, + index + ) ?? [] ), + visibleFieldIds[ index + 1 ], + fieldId, + ...visibleFieldIds.slice( + index + 2 + ), + ], + } ); + } } + > + + { __( 'Move right' ) } + + + ) } + { isHidable && field && ( + } + onClick={ () => { + onHide( field ); + onChangeView( { + ...view, + fields: visibleFieldIds.filter( + ( id ) => id !== fieldId ), - ], - } ); - } } - > - - { __( 'Move right' ) } - - - ) } - { isHidable && field && ( - } - onClick={ () => { - onHide( field ); - onChangeView( { - ...view, - fields: visibleFieldIds.filter( - ( id ) => id !== fieldId - ), - } ); - } } - > - - { __( 'Hide column' ) } - - - ) } - - ) } - + } ); + } } + > + + { __( 'Hide column' ) } + + + ) } + + ) } + + ); } );