Update SettingsObjectAboutSection.tsx changed "Disable" CTA to "Deact… (#4175)

* Update SettingsObjectAboutSection.tsx changed "Disable" CTA to "Deactivate"

* Update SettingsObjects.tsx

Additional changes: Disabled sections to inactive

* Update SettingsObjectAboutSection.tsx

I think you meant changing Disable to Deactivate

* Update and rename SettingsObjectDisabledMenuDropDown.tsx to SettingsObjectInactiveMenuDropDown.tsx

 additional changes to #4153

* Update SettingsObjects.tsx

* Update and rename SettingsObjectDisabledMenuDropDown.stories.tsx to SettingsObjectInactiveMenuDropDown.stories.tsx

* fix typescript errors

* respect issue requirements

---------

Co-authored-by: bosiraphael <raphael.bosi@gmail.com>
This commit is contained in:
selango1
2024-03-04 08:59:31 -05:00
committed by GitHub
parent 3c63584ef8
commit 38a0aae030
8 changed files with 46 additions and 46 deletions

View File

@ -16,7 +16,7 @@ export const useObjectMetadataItemForSettings = () => {
const activeObjectMetadataItems = objectMetadataItems.filter( const activeObjectMetadataItems = objectMetadataItems.filter(
({ isActive, isSystem }) => isActive && !isSystem, ({ isActive, isSystem }) => isActive && !isSystem,
); );
const disabledObjectMetadataItems = objectMetadataItems.filter( const inactiveObjectMetadataItems = objectMetadataItems.filter(
({ isActive, isSystem }) => !isActive && !isSystem, ({ isActive, isSystem }) => !isActive && !isSystem,
); );
@ -88,7 +88,7 @@ export const useObjectMetadataItemForSettings = () => {
activateObjectMetadataItem, activateObjectMetadataItem,
activeObjectMetadataItems, activeObjectMetadataItems,
createObjectMetadataItem, createObjectMetadataItem,
disabledObjectMetadataItems, inactiveObjectMetadataItems,
disableObjectMetadataItem, disableObjectMetadataItem,
editObjectMetadataItem, editObjectMetadataItem,
eraseObjectMetadataItem, eraseObjectMetadataItem,

View File

@ -19,7 +19,7 @@ type SettingsAboutSectionProps = {
iconKey?: string; iconKey?: string;
isCustom: boolean; isCustom: boolean;
name: string; name: string;
onDisable: () => void; onDeactivate: () => void;
onEdit: () => void; onEdit: () => void;
}; };
@ -49,7 +49,7 @@ export const SettingsAboutSection = ({
iconKey = '', iconKey = '',
isCustom, isCustom,
name, name,
onDisable, onDeactivate,
onEdit, onEdit,
}: SettingsAboutSectionProps) => { }: SettingsAboutSectionProps) => {
const theme = useTheme(); const theme = useTheme();
@ -63,8 +63,8 @@ export const SettingsAboutSection = ({
closeDropdown(); closeDropdown();
}; };
const handleDisable = () => { const handleDeactivate = () => {
onDisable(); onDeactivate();
closeDropdown(); closeDropdown();
}; };
@ -92,9 +92,9 @@ export const SettingsAboutSection = ({
onClick={handleEdit} onClick={handleEdit}
/> />
<MenuItem <MenuItem
text="Disable" text="Deactivate"
LeftIcon={IconArchive} LeftIcon={IconArchive}
onClick={handleDisable} onClick={handleDeactivate}
/> />
</DropdownMenuItemsContainer> </DropdownMenuItemsContainer>
</DropdownMenu> </DropdownMenu>

View File

@ -14,7 +14,7 @@ import { MenuItem } from '@/ui/navigation/menu-item/components/MenuItem';
type SettingsObjectFieldActiveActionDropdownProps = { type SettingsObjectFieldActiveActionDropdownProps = {
isCustomField?: boolean; isCustomField?: boolean;
onDisable?: () => void; onDeactivate?: () => void;
onEdit: () => void; onEdit: () => void;
onSetAsLabelIdentifier?: () => void; onSetAsLabelIdentifier?: () => void;
scopeKey: string; scopeKey: string;
@ -22,7 +22,7 @@ type SettingsObjectFieldActiveActionDropdownProps = {
export const SettingsObjectFieldActiveActionDropdown = ({ export const SettingsObjectFieldActiveActionDropdown = ({
isCustomField, isCustomField,
onDisable, onDeactivate,
onEdit, onEdit,
onSetAsLabelIdentifier, onSetAsLabelIdentifier,
scopeKey, scopeKey,
@ -36,8 +36,8 @@ export const SettingsObjectFieldActiveActionDropdown = ({
closeDropdown(); closeDropdown();
}; };
const handleDisable = () => { const handleDeactivate = () => {
onDisable?.(); onDeactivate?.();
closeDropdown(); closeDropdown();
}; };
@ -67,11 +67,11 @@ export const SettingsObjectFieldActiveActionDropdown = ({
onClick={handleSetAsLabelIdentifier} onClick={handleSetAsLabelIdentifier}
/> />
)} )}
{!!onDisable && ( {!!onDeactivate && (
<MenuItem <MenuItem
text="Disable" text="Deactivate"
LeftIcon={IconArchive} LeftIcon={IconArchive}
onClick={handleDisable} onClick={handleDeactivate}
/> />
)} )}
</DropdownMenuItemsContainer> </DropdownMenuItemsContainer>

View File

@ -6,17 +6,17 @@ import { DropdownMenuItemsContainer } from '@/ui/layout/dropdown/components/Drop
import { useDropdown } from '@/ui/layout/dropdown/hooks/useDropdown'; import { useDropdown } from '@/ui/layout/dropdown/hooks/useDropdown';
import { MenuItem } from '@/ui/navigation/menu-item/components/MenuItem'; import { MenuItem } from '@/ui/navigation/menu-item/components/MenuItem';
type SettingsObjectFieldDisabledActionDropdownProps = { type SettingsObjectFieldInactiveActionDropdownProps = {
isCustomField?: boolean; isCustomField?: boolean;
onActivate: () => void; onActivate: () => void;
onErase: () => void; onErase: () => void;
scopeKey: string; scopeKey: string;
}; };
export const SettingsObjectFieldDisabledActionDropdown = ({ export const SettingsObjectFieldInactiveActionDropdown = ({
onActivate, onActivate,
scopeKey, scopeKey,
}: SettingsObjectFieldDisabledActionDropdownProps) => { }: SettingsObjectFieldInactiveActionDropdownProps) => {
const dropdownId = `${scopeKey}-settings-field-disabled-action-dropdown`; const dropdownId = `${scopeKey}-settings-field-disabled-action-dropdown`;
const { closeDropdown } = useDropdown(dropdownId); const { closeDropdown } = useDropdown(dropdownId);

View File

@ -6,20 +6,20 @@ import { DropdownMenuItemsContainer } from '@/ui/layout/dropdown/components/Drop
import { useDropdown } from '@/ui/layout/dropdown/hooks/useDropdown'; import { useDropdown } from '@/ui/layout/dropdown/hooks/useDropdown';
import { MenuItem } from '@/ui/navigation/menu-item/components/MenuItem'; import { MenuItem } from '@/ui/navigation/menu-item/components/MenuItem';
type SettingsObjectDisabledMenuDropDownProps = { type SettingsObjectInactiveMenuDropDownProps = {
isCustomObject: boolean; isCustomObject: boolean;
onActivate: () => void; onActivate: () => void;
onErase: () => void; onErase: () => void;
scopeKey: string; scopeKey: string;
}; };
export const SettingsObjectDisabledMenuDropDown = ({ export const SettingsObjectInactiveMenuDropDown = ({
onActivate, onActivate,
scopeKey, scopeKey,
onErase, onErase,
isCustomObject, isCustomObject,
}: SettingsObjectDisabledMenuDropDownProps) => { }: SettingsObjectInactiveMenuDropDownProps) => {
const dropdownId = `${scopeKey}-settings-object-disabled-menu-dropdown`; const dropdownId = `${scopeKey}-settings-object-inactive-menu-dropdown`;
const { closeDropdown } = useDropdown(dropdownId); const { closeDropdown } = useDropdown(dropdownId);

View File

@ -3,7 +3,7 @@ import { expect, fn, userEvent, within } from '@storybook/test';
import { ComponentDecorator } from '~/testing/decorators/ComponentDecorator'; import { ComponentDecorator } from '~/testing/decorators/ComponentDecorator';
import { SettingsObjectDisabledMenuDropDown } from '../SettingsObjectDisabledMenuDropDown'; import { SettingsObjectInactiveMenuDropDown } from '../SettingsObjectInactiveMenuDropDown';
const handleActivateMockFunction = fn(); const handleActivateMockFunction = fn();
const handleEraseMockFunction = fn(); const handleEraseMockFunction = fn();
@ -16,11 +16,11 @@ const ClearMocksDecorator: Decorator = (Story, context) => {
return <Story />; return <Story />;
}; };
const meta: Meta<typeof SettingsObjectDisabledMenuDropDown> = { const meta: Meta<typeof SettingsObjectInactiveMenuDropDown> = {
title: 'Modules/Settings/DataModel/SettingsObjectDisabledMenuDropDown', title: 'Modules/Settings/DataModel/SettingsObjectInactiveMenuDropDown',
component: SettingsObjectDisabledMenuDropDown, component: SettingsObjectInactiveMenuDropDown,
args: { args: {
scopeKey: 'settings-object-disabled-menu-dropdown', scopeKey: 'settings-object-inactive-menu-dropdown',
onActivate: handleActivateMockFunction, onActivate: handleActivateMockFunction,
onErase: handleEraseMockFunction, onErase: handleEraseMockFunction,
}, },
@ -31,7 +31,7 @@ const meta: Meta<typeof SettingsObjectDisabledMenuDropDown> = {
}; };
export default meta; export default meta;
type Story = StoryObj<typeof SettingsObjectDisabledMenuDropDown>; type Story = StoryObj<typeof SettingsObjectInactiveMenuDropDown>;
export const Default: Story = {}; export const Default: Story = {};

View File

@ -10,7 +10,7 @@ import { isLabelIdentifierField } from '@/object-metadata/utils/isLabelIdentifie
import { SettingsPageContainer } from '@/settings/components/SettingsPageContainer'; import { SettingsPageContainer } from '@/settings/components/SettingsPageContainer';
import { SettingsAboutSection } from '@/settings/data-model/object-details/components/SettingsObjectAboutSection'; import { SettingsAboutSection } from '@/settings/data-model/object-details/components/SettingsObjectAboutSection';
import { SettingsObjectFieldActiveActionDropdown } from '@/settings/data-model/object-details/components/SettingsObjectFieldActiveActionDropdown'; import { SettingsObjectFieldActiveActionDropdown } from '@/settings/data-model/object-details/components/SettingsObjectFieldActiveActionDropdown';
import { SettingsObjectFieldDisabledActionDropdown } from '@/settings/data-model/object-details/components/SettingsObjectFieldDisabledActionDropdown'; import { SettingsObjectFieldInactiveActionDropdown } from '@/settings/data-model/object-details/components/SettingsObjectFieldDisabledActionDropdown';
import { import {
SettingsObjectFieldItemTableRow, SettingsObjectFieldItemTableRow,
StyledObjectFieldTableRow, StyledObjectFieldTableRow,
@ -94,7 +94,7 @@ export const SettingsObjectDetail = () => {
iconKey={activeObjectMetadataItem.icon ?? undefined} iconKey={activeObjectMetadataItem.icon ?? undefined}
name={activeObjectMetadataItem.labelPlural || ''} name={activeObjectMetadataItem.labelPlural || ''}
isCustom={activeObjectMetadataItem.isCustom} isCustom={activeObjectMetadataItem.isCustom}
onDisable={handleDisableObject} onDeactivate={handleDisableObject}
onEdit={() => navigate('./edit')} onEdit={() => navigate('./edit')}
/> />
<Section> <Section>
@ -155,7 +155,7 @@ export const SettingsObjectDetail = () => {
) )
: undefined : undefined
} }
onDisable={ onDeactivate={
isLabelIdentifier isLabelIdentifier
? undefined ? undefined
: () => handleDisableField(activeMetadataField) : () => handleDisableField(activeMetadataField)
@ -168,7 +168,7 @@ export const SettingsObjectDetail = () => {
</TableSection> </TableSection>
)} )}
{!!disabledMetadataFields.length && ( {!!disabledMetadataFields.length && (
<TableSection isInitiallyExpanded={false} title="Disabled"> <TableSection isInitiallyExpanded={false} title="Inactive">
{disabledMetadataFields.map((disabledMetadataField) => ( {disabledMetadataFields.map((disabledMetadataField) => (
<SettingsObjectFieldItemTableRow <SettingsObjectFieldItemTableRow
key={disabledMetadataField.id} key={disabledMetadataField.id}
@ -179,7 +179,7 @@ export const SettingsObjectDetail = () => {
} }
fieldMetadataItem={disabledMetadataField} fieldMetadataItem={disabledMetadataField}
ActionIcon={ ActionIcon={
<SettingsObjectFieldDisabledActionDropdown <SettingsObjectFieldInactiveActionDropdown
isCustomField={!!disabledMetadataField.isCustom} isCustomField={!!disabledMetadataField.isCustom}
scopeKey={disabledMetadataField.id} scopeKey={disabledMetadataField.id}
onActivate={() => onActivate={() =>

View File

@ -11,7 +11,7 @@ import {
StyledObjectTableRow, StyledObjectTableRow,
} from '@/settings/data-model/object-details/components/SettingsObjectItemTableRow'; } from '@/settings/data-model/object-details/components/SettingsObjectItemTableRow';
import { SettingsObjectCoverImage } from '@/settings/data-model/objects/SettingsObjectCoverImage'; import { SettingsObjectCoverImage } from '@/settings/data-model/objects/SettingsObjectCoverImage';
import { SettingsObjectDisabledMenuDropDown } from '@/settings/data-model/objects/SettingsObjectDisabledMenuDropDown'; import { SettingsObjectInactiveMenuDropDown } from '@/settings/data-model/objects/SettingsObjectInactiveMenuDropDown';
import { IconChevronRight, IconPlus, IconSettings } from '@/ui/display/icon'; import { IconChevronRight, IconPlus, IconSettings } from '@/ui/display/icon';
import { H1Title } from '@/ui/display/typography/components/H1Title'; import { H1Title } from '@/ui/display/typography/components/H1Title';
import { H2Title } from '@/ui/display/typography/components/H2Title'; import { H2Title } from '@/ui/display/typography/components/H2Title';
@ -37,7 +37,7 @@ export const SettingsObjects = () => {
const { const {
activateObjectMetadataItem, activateObjectMetadataItem,
activeObjectMetadataItems, activeObjectMetadataItems,
disabledObjectMetadataItems, inactiveObjectMetadataItems,
eraseObjectMetadataItem, eraseObjectMetadataItem,
} = useObjectMetadataItemForSettings(); } = useObjectMetadataItemForSettings();
@ -89,25 +89,25 @@ export const SettingsObjects = () => {
))} ))}
</TableSection> </TableSection>
)} )}
{!!disabledObjectMetadataItems.length && ( {!!inactiveObjectMetadataItems.length && (
<TableSection title="Disabled"> <TableSection title="Inactive">
{disabledObjectMetadataItems.map( {inactiveObjectMetadataItems.map(
(disabledObjectMetadataItem) => ( (inactiveObjectMetadataItem) => (
<SettingsObjectItemTableRow <SettingsObjectItemTableRow
key={disabledObjectMetadataItem.namePlural} key={inactiveObjectMetadataItem.namePlural}
objectItem={disabledObjectMetadataItem} objectItem={inactiveObjectMetadataItem}
action={ action={
<SettingsObjectDisabledMenuDropDown <SettingsObjectInactiveMenuDropDown
isCustomObject={disabledObjectMetadataItem.isCustom} isCustomObject={inactiveObjectMetadataItem.isCustom}
scopeKey={disabledObjectMetadataItem.namePlural} scopeKey={inactiveObjectMetadataItem.namePlural}
onActivate={() => onActivate={() =>
activateObjectMetadataItem( activateObjectMetadataItem(
disabledObjectMetadataItem, inactiveObjectMetadataItem,
) )
} }
onErase={() => onErase={() =>
eraseObjectMetadataItem( eraseObjectMetadataItem(
disabledObjectMetadataItem, inactiveObjectMetadataItem,
) )
} }
/> />