diff --git a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectedAccountsListCard.tsx b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectedAccountsListCard.tsx index 510468c422a..66a2bec2722 100644 --- a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectedAccountsListCard.tsx +++ b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectedAccountsListCard.tsx @@ -23,7 +23,6 @@ const StyledAddAccountSection = styled(Section)` display: flex; justify-content: flex-end; padding-top: ${({ theme }) => theme.spacing(2)}; - padding-bottom: ${({ theme }) => theme.spacing(2)}; `; export const SettingsAccountsConnectedAccountsListCard = ({ diff --git a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectionForm.tsx b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectionForm.tsx index 1f79068b184..4fb0d853e4d 100644 --- a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectionForm.tsx +++ b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsConnectionForm.tsx @@ -68,13 +68,9 @@ export const SettingsAccountsConnectionForm = ({ }: SettingsAccountsConnectionFormProps) => { const { t } = useLingui(); - const getTitle = () => { - return isEditing ? t`Edit Email Account` : t`New Email Account`; - }; - const getDescription = () => { if (isEditing) { - return t`Update your email account configuration. Configure any combination of IMAP, SMTP, and CalDAV as needed.`; + return t`Update your account's configuration. Configure any combination of IMAP, SMTP, and CalDAV as needed.`; } return t`You can set up any combination of IMAP (receiving emails), SMTP (sending emails), and CalDAV (calendar sync).`; }; @@ -83,7 +79,7 @@ export const SettingsAccountsConnectionForm = ({ return (
- + { // eslint-disable-next-line react/jsx-props-no-spreading { > } - title={t`Connect Email Account`} + title={t`Connect Account`} /> )} diff --git a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsNewImapSmtpCaldavConnection.tsx b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsNewImapSmtpCaldavConnection.tsx index e37b78d8486..9e00e03497b 100644 --- a/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsNewImapSmtpCaldavConnection.tsx +++ b/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsNewImapSmtpCaldavConnection.tsx @@ -31,17 +31,17 @@ export const SettingsAccountsNewImapSmtpCaldavConnection = () => { // eslint-disable-next-line react/jsx-props-no-spreading { const href = item.path ? getSettingsPath(item.path) : ''; const pathName = useResolvedPath(href).pathname; - const isActive = !!useMatch({ + const matchResult = useMatch({ path: pathName, end: item.matchSubPages === false, }); + const isActive = !!matchResult && !hasActiveSubItem; + if (isDefined(item.isHidden) && item.isHidden) { return null; } diff --git a/packages/twenty-front/src/modules/settings/components/SettingsNavigationDrawerItems.tsx b/packages/twenty-front/src/modules/settings/components/SettingsNavigationDrawerItems.tsx index 581b295266b..68704250efd 100644 --- a/packages/twenty-front/src/modules/settings/components/SettingsNavigationDrawerItems.tsx +++ b/packages/twenty-front/src/modules/settings/components/SettingsNavigationDrawerItems.tsx @@ -55,6 +55,7 @@ export const SettingsNavigationDrawerItems = () => { if (Array.isArray(subItems) && subItems.length > 0) { const selectedSubItemIndex = getSelectedIndexForSubItems(subItems); + const hasActiveSubItem = selectedSubItemIndex !== -1; return ( { > { label: t`Accounts`, path: SettingsPath.Accounts, Icon: IconAt, - matchSubPages: false, subItems: [ { label: t`Emails`,