From 858739eaa00b62c8095c0fe5b0d630eb25bd0d47 Mon Sep 17 00:00:00 2001
From: ogerly <fridolin@tutanota.com>
Date: Thu, 22 Sep 2022 18:42:58 +0200
Subject: [PATCH] fix review

---
 backend/src/schema/resolvers/users/location.spec.js |  2 +-
 webapp/assets/_new/styles/tokens.scss               |  2 +-
 webapp/components/FilterMenu/CategoriesMenu.vue     | 10 ++--------
 3 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/backend/src/schema/resolvers/users/location.spec.js b/backend/src/schema/resolvers/users/location.spec.js
index 92f08893e..c0525ca1c 100644
--- a/backend/src/schema/resolvers/users/location.spec.js
+++ b/backend/src/schema/resolvers/users/location.spec.js
@@ -121,7 +121,7 @@ describe('Location Service', () => {
     const result = await query({ query: queryLocations, variables })
     expect(result.data.queryLocations).toEqual(
       expect.arrayContaining([
-        { id: 'place.14094307404564380', place_name: 'Berlin, Germany' },
+        { id: expect.stringMatching(/^place\.[0-9]+$/), place_name: 'Berlin, Germany' },
         {
           id: expect.stringMatching(/^place\.[0-9]+$/),
           place_name: 'Berlin, Maryland, United States',
diff --git a/webapp/assets/_new/styles/tokens.scss b/webapp/assets/_new/styles/tokens.scss
index caca5953f..6aa6410df 100644
--- a/webapp/assets/_new/styles/tokens.scss
+++ b/webapp/assets/_new/styles/tokens.scss
@@ -374,4 +374,4 @@ $color-footer-link: $color-primary;
  * @tokens Locale Menu Color
  */
 
- $color-locale-menu: $text-color-soft;
\ No newline at end of file
+$color-locale-menu: $text-color-soft;
\ No newline at end of file
diff --git a/webapp/components/FilterMenu/CategoriesMenu.vue b/webapp/components/FilterMenu/CategoriesMenu.vue
index a9a36496b..0b5505503 100644
--- a/webapp/components/FilterMenu/CategoriesMenu.vue
+++ b/webapp/components/FilterMenu/CategoriesMenu.vue
@@ -1,13 +1,6 @@
 <template>
   <dropdown ref="category-menu" placement="top-start" :offset="8" class="category-menu">
-    <a
-      href="#"
-      slot="default"
-      :filled="filterActive"
-      :ghost="!filterActive"
-      slot-scope="{ toggleMenu }"
-      @click.prevent="toggleMenu()"
-    >
+    <a href="#" slot="default" slot-scope="{ toggleMenu }" @click.prevent="toggleMenu()">
       <ds-text bold size="large">{{ $t('admin.categories.name') }}</ds-text>
     </a>
     <template slot="popover">
@@ -41,6 +34,7 @@ export default {
   },
   computed: {
     ...mapGetters({
+      // TODO: implement visibility of active filter later on
       filterActive: 'posts/isActive',
     }),
   },