src/App.scss

@@ -408,11 +408,32 @@ nav {
   }
 }
 
+.menu-item,
 .list-item {
+  display: block;
+  box-sizing: border-box;
+  border: none;
+  outline: none;
+  text-align: initial;
+  color: inherit;
+  clear: both;
+  position: relative;
+  white-space: nowrap;
   border-color: var(--border);
   border-style: solid;
   border-width: 0;
   border-top-width: 1px;
+  width: 100%;
+  padding: var(--__vertical-gap) var(--__horizontal-gap);
+  background: transparent;
+
+  --__line-height: 1.5em;
+  --__horizontal-gap: 0.75em;
+  --__vertical-gap: 0.5em;
+
+  &.-non-interactive {
+    cursor: auto;
+  }
 
   &.-active,
   &:hover {
@@ -434,42 +455,6 @@ nav {
     border-bottom-width: 1px;
   }
 
-  &:first-child {
-    border-top-right-radius: var(--roundness);
-    border-top-left-radius: var(--roundness);
-    border-top-width: 0;
-  }
-
-  &:last-child {
-    border-bottom-right-radius: var(--roundness);
-    border-bottom-left-radius: var(--roundness);
-    border-bottom-width: 0;
-  }
-}
-
-.menu-item,
-.list-item {
-  display: block;
-  box-sizing: border-box;
-  border: none;
-  outline: none;
-  text-align: initial;
-  color: inherit;
-  clear: both;
-  position: relative;
-  white-space: nowrap;
-  width: 100%;
-  padding: var(--__vertical-gap) var(--__horizontal-gap);
-  background: transparent;
-
-  --__line-height: 1.5em;
-  --__horizontal-gap: 0.75em;
-  --__vertical-gap: 0.5em;
-
-  &.-non-interactive {
-    cursor: auto;
-  }
-
   a,
   button:not(.button-default) {
     text-align: initial;
@@ -481,6 +466,18 @@ nav {
     font-family: inherit;
     line-height: unset;
   }
+
+  &:first-child {
+    border-top-right-radius: var(--roundness);
+    border-top-left-radius: var(--roundness);
+    border-top-width: 0;
+  }
+
+  &:last-child {
+    border-bottom-right-radius: var(--roundness);
+    border-bottom-left-radius: var(--roundness);
+    border-bottom-width: 0;
+  }
 }
 
 .button-unstyled {

src/components/conversation/conversation.js

@@ -350,7 +350,6 @@ const conversation = {
     },
     ...mapGetters(['mergedConfig']),
     ...mapState({
-      mobileLayout: state => state.interface.layoutType === 'mobile',
       mastoUserSocketStatus: state => state.api.mastoUserSocketStatus
     })
   },

src/components/conversation/conversation.vue

@@ -20,7 +20,7 @@
         {{ $t('timeline.collapse') }}
       </button>
       <QuickFilterSettings
-        v-if="!collapsable && mobileLayout"
+        v-if="!collapsable"
         :conversation="true"
         class="rightside-button"
       />

src/components/input.style.js

@@ -18,7 +18,7 @@ export default {
     {
       component: 'Root',
       directives: {
-        '--defaultInputBevel': 'shadow | $borderSide(#FFFFFF bottom 0.2), $borderSide(#000000 top 0.2), inset 0 0 2 #000000 / 0.15, 1 0 1 1 --text / 0.15, -1 0 1 1 --text / 0.15',
+        '--defaultInputBevel': 'shadow | $borderSide(#FFFFFF bottom 0.2), $borderSide(#000000 top 0.2), inset 0 0 2 #000000 / 0.15',
         '--defaultInputHoverGlow': 'shadow | 0 0 4 --text / 0.5',
         '--defaultInputFocusGlow': 'shadow | 0 0 4 4 --link / 0.5'
       }

src/components/quick_filter_settings/quick_filter_settings.js

@@ -1,5 +1,5 @@
 import Popover from '../popover/popover.vue'
-import { mapGetters, mapState } from 'vuex'
+import { mapGetters } from 'vuex'
 import { library } from '@fortawesome/fontawesome-svg-core'
 import { faFilter, faFont, faWrench } from '@fortawesome/free-solid-svg-icons'
 
@@ -11,8 +11,7 @@ library.add(
 
 const QuickFilterSettings = {
   props: {
-    conversation: Boolean,
-    nested: Boolean
+    conversation: Boolean
   },
   components: {
     Popover
@@ -28,25 +27,6 @@ const QuickFilterSettings = {
   },
   computed: {
     ...mapGetters(['mergedConfig']),
-    ...mapState({
-      mobileLayout: state => state.interface.layoutType === 'mobile'
-    }),
-    triggerAttrs () {
-      if (this.mobileLayout) {
-        return {}
-      } else {
-        return {
-          title: this.$t('timeline.quick_filter_settings')
-        }
-      }
-    },
-    mainClass () {
-      if (this.mobileLayout) {
-        return 'main-button'
-      } else {
-        return 'dropdown-item'
-      }
-    },
     loggedIn () {
       return !!this.$store.state.users.currentUser
     },

src/components/quick_filter_settings/quick_filter_settings.vue

@@ -1,10 +1,9 @@
 <template>
   <Popover
-    :trigger="nested ? 'hover' : 'click'"
+    trigger="click"
     class="QuickFilterSettings"
     :bound-to="{ x: 'container' }"
-    :position="nested ? 'right' : 'top'"
-    :trigger-attrs="triggerAttrs"
+    :trigger-attrs="{ title: $t('timeline.quick_filter_settings') }"
   >
     <template #content>
       <div
@@ -138,19 +137,7 @@
       </div>
     </template>
     <template #trigger>
-      <div :class="mobileLayout ? 'main-button' : ''">
-        <FAIcon icon="filter" :fixed-width="nested"/>
-        <template v-if="nested">
-          {{$t('timeline.filter_settings')}}
-        </template>
-        <FAIcon
-          v-if="nested"
-          class="chevron-icon"
-          size="lg"
-          icon="chevron-right"
-          fixed-width
-        />
-      </div>
+      <FAIcon icon="filter" />
     </template>
   </Popover>
 </template>

src/components/quick_view_settings/quick_view_settings.js

@@ -1,6 +1,5 @@
-import Popover from 'src/components/popover/popover.vue'
-import QuickFilterSettings from 'src/components/quick_filter_settings/quick_filter_settings.vue'
-import { mapGetters, mapState } from 'vuex'
+import Popover from '../popover/popover.vue'
+import { mapGetters } from 'vuex'
 import { library } from '@fortawesome/fontawesome-svg-core'
 import { faList, faFolderTree, faBars, faWrench } from '@fortawesome/free-solid-svg-icons'
 
@@ -16,8 +15,7 @@ const QuickViewSettings = {
     conversation: Boolean
   },
   components: {
-    Popover,
-    QuickFilterSettings
+    Popover
   },
   methods: {
     setConversationDisplay (visibility) {
@@ -29,9 +27,6 @@ const QuickViewSettings = {
   },
   computed: {
     ...mapGetters(['mergedConfig']),
-    ...mapState({
-      mobileLayout: state => state.interface.layoutType === 'mobile'
-    }),
     loggedIn () {
       return !!this.$store.state.users.currentUser
     },

src/components/quick_view_settings/quick_view_settings.vue

@@ -3,24 +3,13 @@
     trigger="click"
     class="QuickViewSettings"
     :bound-to="{ x: 'container' }"
-    :trigger-attrs="triggerAttrs"
+    :trigger-attrs="{ title: $t('timeline.quick_view_settings') }"
   >
     <template #content>
       <div
         class="dropdown-menu"
         role="menu"
       >
-        <div
-          v-if="mobileLayout"
-          class="menu-item dropdown-item -icon"
-        >
-          <QuickFilterSettings :nested="true" />
-        </div>
-        <div
-          v-if="mobileLayout"
-          role="separator"
-          class="dropdown-divider"
-        />
         <div role="group">
           <div class="menu-item dropdown-item -icon-double">
             <button
@@ -71,7 +60,7 @@
             @click="showUserAvatars = !showUserAvatars"
           >
             <span
-              class="input menu-checkbox"
+              class="main-button"
               :class="{ 'menu-checkbox-checked': showUserAvatars }"
               :aria-hidden="true"
             />{{ $t('settings.mention_link_show_avatar_quick') }}

src/components/timeline/timeline.vue

@@ -77,7 +77,6 @@
       </template>
       <QuickFilterSettings
         class="rightside-button"
-        v-if="!mobileLayout"
       />
       <QuickViewSettings
         class="rightside-button"

src/i18n/en.json

@@ -1214,8 +1214,7 @@
     "socket_reconnected": "Realtime connection established",
     "socket_broke": "Realtime connection lost: CloseEvent code {0}",
     "quick_view_settings": "Quick view settings",
-    "quick_filter_settings": "Quick filter settings",
-    "filter_settings": "Filter"
+    "quick_filter_settings": "Quick filter settings"
   },
   "status": {
     "favorites": "Favorites",