Merge pull request 'Move language switch button to the footer' (#100) from feature/language-switch-in-footer into main

Reviewed-on: #100
Reviewed-by: Johannes D. Vos

src/features/footer/components/Footer.vue

@@ -15,6 +15,8 @@ limitations under the License.
 -->
 
 <script setup>
+import LanguageSwitchButton from '@/features/i18n/components/LanguageSwitchButton.vue';
+
 import { useI18n } from 'vue-i18n';
 
 const { t } = useI18n();
@@ -36,12 +38,11 @@ const copyrightPeriod =
             <RouterLink to="settings" class="link">
                 {{ t('preferences.settings') }}
             </RouterLink>
+            <LanguageSwitchButton />
         </div>
         <div class="footer-segment">
-            <div class="copyright-note">
             &copy; {{ copyrightPeriod }} Seekra
         </div>
-        </div>
     </footer>
 </template>
 
@@ -51,6 +52,9 @@ const copyrightPeriod =
 }
 
 .footer-segment {
+    display: flex;
+    justify-content: center;
+    gap: 32px;
     padding: var(--padding-y);
     background-color: var(--light-bg);
     border-top: 1px solid var(--border);
@@ -59,9 +63,4 @@ const copyrightPeriod =
 .global-footer a {
     color: var(--dark);
 }
-
-.copyright-note {
-    display: flex;
-    justify-content: center;
-}
 </style>

src/features/i18n/components/LanguageSwitchButton.vue

@@ -71,7 +71,15 @@ const open = function () {
             </span>
         </button>
 
-        <ul v-if="isOpen" ref="languageDropdown" class="language-dropdown" role="listbox">
+        <ul
+            v-if="isOpen"
+            ref="languageDropdown"
+            class="language-dropdown"
+            role="listbox"
+            :style="{
+                bottom: 'calc(var(--offset) + 2 * var(--trigger-padding-y) + 1em)' // easier to add auto adaption to the available space
+            }"
+        >
             <li
                 v-for="lang in SUPPORTED_LANGUAGES"
                 :key="lang"
@@ -88,6 +96,7 @@ const open = function () {
 
 <style scoped>
 .language-switch {
+    --trigger-padding-y: 4px;
     position: relative;
 }
 
@@ -98,7 +107,7 @@ const open = function () {
     background: none;
     border: 1px solid var(--border);
     border-radius: 6px;
-    padding: 4px 10px;
+    padding: var(--trigger-padding-y) 10px;
     cursor: pointer;
     color: var(--dark);
 }
@@ -118,9 +127,9 @@ const open = function () {
 }
 
 .language-dropdown {
+    --offset: 6px;
     position: absolute;
     right: 0;
-    top: calc(100% + 6px);
     background-color: var(--light-bg);
     border: 1px solid var(--border);
     border-radius: 8px;

src/features/nav/components/Navbar.vue

@@ -16,7 +16,6 @@ limitations under the License.
 
 <script setup>
 import ColorSchemeButton from '@/features/colorScheme/components/ColorSchemeButton.vue';
-import LanguageSwitchButton from '@/features/i18n/components/LanguageSwitchButton.vue';
 import logo from '@/assets/images/logo.svg';
 import NavbarSearchBarWrapper from './NavbarSearchBarWrapper.vue';
 </script>
@@ -28,9 +27,6 @@ import NavbarSearchBarWrapper from './NavbarSearchBarWrapper.vue';
         </RouterLink>
         <NavbarSearchBarWrapper class="navbar-search-bar-wrapper" />
         <ul class="right-links">
-            <li>
-                <LanguageSwitchButton />
-            </li>
             <li>
                 <ColorSchemeButton />
             </li>