refactor(dev-only): hide non-applicable options in display options menu

This commit is contained in:
Aman Harwara
2022-12-24 21:55:24 +05:30
parent fc6d28d166
commit b3fe769a60

View File

@@ -5,6 +5,7 @@ import {
isSmartView,
isSystemView,
PrefKey,
SystemViewId,
TagMutator,
TagPreferences,
VectorIconNameOrEmoji,
@@ -23,6 +24,7 @@ import NoSubscriptionBanner from '@/Components/NoSubscriptionBanner/NoSubscripti
import MenuRadioButtonItem from '@/Components/Menu/MenuRadioButtonItem'
import MenuSwitchButtonItem from '@/Components/Menu/MenuSwitchButtonItem'
import { Pill } from '@/Components/Preferences/PreferencesComponents/Content'
import { featureTrunkEnabled, FeatureTrunkName } from '@/FeatureTrunk'
const DailyEntryModeEnabled = true
@@ -201,6 +203,9 @@ const DisplayOptionsMenu: FunctionComponent<DisplayOptionsMenuProps> = ({
)
}
const isFilesTableViewEnabled = featureTrunkEnabled(FeatureTrunkName.FilesTableView)
const shouldHideNonApplicableOptions = isFilesTableViewEnabled && selectedTag?.uuid === SystemViewId.Files
return (
<Menu className="text-sm" a11yLabel="Notes list options menu" isOpen={isOpen}>
<div className="my-1 px-3 text-base font-semibold uppercase text-text lg:text-xs">Preferences for</div>
@@ -283,6 +288,8 @@ const DisplayOptionsMenu: FunctionComponent<DisplayOptionsMenuProps> = ({
) : null}
</div>
</MenuRadioButtonItem>
{!shouldHideNonApplicableOptions && (
<>
<MenuItemSeparator />
<div className="px-3 py-1 text-base font-semibold uppercase text-text lg:text-xs">View</div>
{!isFilesSmartView && (
@@ -353,6 +360,8 @@ const DisplayOptionsMenu: FunctionComponent<DisplayOptionsMenuProps> = ({
>
Show trashed
</MenuSwitchButtonItem>
</>
)}
{currentMode === 'tag' && DailyEntryModeEnabled && (
<>
@@ -376,8 +385,9 @@ const DisplayOptionsMenu: FunctionComponent<DisplayOptionsMenuProps> = ({
</>
)}
{!shouldHideNonApplicableOptions && (
<>
<MenuItemSeparator />
<NewNotePreferences
disabled={controlsDisabled}
application={application}
@@ -385,6 +395,8 @@ const DisplayOptionsMenu: FunctionComponent<DisplayOptionsMenuProps> = ({
mode={currentMode}
changePreferencesCallback={changePreferences}
/>
</>
)}
</Menu>
)
}