diff --git a/src/components/nav-menu.jsx b/src/components/nav-menu.jsx index 2e22d532..96b9ce46 100644 --- a/src/components/nav-menu.jsx +++ b/src/components/nav-menu.jsx @@ -35,8 +35,9 @@ function NavMenu(props) { // User may choose pin or not to pin Following // If user doesn't pin Following, we show it in the menu const showFollowing = - (snapStates.settings.shortcutsColumnsMode || - snapStates.settings.shortcutsViewMode === 'multi-column') && + (snapStates.settings.shortcutsViewMode === 'multi-column' || + (!snapStates.settings.shortcutsViewMode && + snapStates.settings.shortcutsColumnsMode)) && !snapStates.shortcuts.find((pin) => pin.type === 'following'); const bindLongPress = useLongPress( diff --git a/src/components/shortcuts-settings.jsx b/src/components/shortcuts-settings.jsx index 1d551c6c..b3ce27f5 100644 --- a/src/components/shortcuts-settings.jsx +++ b/src/components/shortcuts-settings.jsx @@ -300,33 +300,6 @@ function ShortcutsSettings({ onClose }) { ); })} - {/* */} - {/*

-

- - Experimental Multi-column mode - - -
-

*/} {shortcuts.length > 0 ? (
    {shortcuts.filter(Boolean).map((shortcut, i) => { diff --git a/src/components/shortcuts.jsx b/src/components/shortcuts.jsx index b45a2354..08bbcb94 100644 --- a/src/components/shortcuts.jsx +++ b/src/components/shortcuts.jsx @@ -25,8 +25,8 @@ function Shortcuts() { return null; } if ( - settings.shortcutsColumnsMode || - settings.shortcutsViewMode === 'multi-column' + settings.shortcutsViewMode === 'multi-column' || + (!settings.shortcutsViewMode && settings.shortcutsColumnsMode) ) { return null; } diff --git a/src/pages/home.jsx b/src/pages/home.jsx index 35176112..fe4503cd 100644 --- a/src/pages/home.jsx +++ b/src/pages/home.jsx @@ -35,8 +35,9 @@ function Home() { return ( <> - {(snapStates.settings.shortcutsColumnsMode || - snapStates.settings.shortcutsViewMode === 'multi-column') && + {(snapStates.settings.shortcutsViewMode === 'multi-column' || + (!snapStates.settings.shortcutsViewMode && + snapStates.settings.shortcutsColumnsMode)) && !!snapStates.shortcuts?.length ? ( ) : ( diff --git a/src/utils/states.js b/src/utils/states.js index 45023621..2f295715 100644 --- a/src/utils/states.js +++ b/src/utils/states.js @@ -71,8 +71,9 @@ export function initStates() { store.account.get('settings-autoRefresh') ?? false; states.settings.shortcutsViewMode = store.account.get('settings-shortcutsViewMode') ?? null; - states.settings.shortcutsColumnsMode = - store.account.get('settings-shortcutsColumnsMode') ?? false; + if (store.account.get('settings-shortcutsColumnsMode')) { + states.settings.shortcutsColumnsMode = true; + } states.settings.boostsCarousel = store.account.get('settings-boostsCarousel') ?? true; states.settings.contentTranslation = @@ -99,9 +100,6 @@ subscribe(states, (changes) => { if (path.join('.') === 'settings.boostsCarousel') { store.account.set('settings-boostsCarousel', !!value); } - if (path.join('.') === 'settings.shortcutsColumnsMode') { - store.account.set('settings-shortcutsColumnsMode', !!value); - } if (path.join('.') === 'settings.shortcutsViewMode') { store.account.set('settings-shortcutsViewMode', value); }