pin sass back to 1.91 to prevent mixed-decls warnings (#2333)

also fix a couple small sass issues

Author: sawka

frontend/app/element/collapsiblemenu.scss

@@ -9,7 +9,6 @@
 }
 
 .collapsible-menu-item {
-    padding: 10px;
     cursor: pointer;
     user-select: none;
     padding: 0;

frontend/app/element/flyoutmenu.scss

@@ -23,6 +23,7 @@
 .menu-item {
     display: flex;
     align-items: center;
+    justify-content: space-between;
     padding: 4px 6px;
     cursor: pointer;
     color: var(--main-text-color);
@@ -33,9 +34,6 @@
     letter-spacing: -0.12px;
     width: 100%;
     border-radius: 2px;
-    display: flex;
-    align-items: center;
-    justify-content: space-between;
 
     .label {
         @include mixins.ellipsis();

frontend/app/notification/notificationitem.scss

@@ -2,6 +2,10 @@
 // SPDX-License-Identifier: Apache-2.0
 
 .notification {
+    &.hovered {
+        background: #292929;
+    }
+
     .notification-title {
         font-size: 13px;
         font-style: normal;
@@ -113,7 +117,4 @@
             margin-bottom: 7px;
         }
     }
-    &.hovered {
-        background: #292929;
-    }
 }

frontend/app/tab/tabbar.scss

@@ -14,14 +14,13 @@
 
 .config-error-message {
     max-width: 500px;
+    margin-bottom: 20px;
 
     h3 {
         font-weight: bold;
         font-size: 16px;
         margin-bottom: 10px;
     }
-
-    margin-bottom: 20px;
 }
 
 .tab-bar-wrapper {

frontend/app/view/chat/chatmessages.scss

@@ -12,7 +12,6 @@
 
 .chat-message {
     display: flex;
-    align-items: flex-start;
     align-items: center;
     margin-bottom: 8px;
     font-size: 1em;

frontend/app/view/chat/userlist.scss

@@ -4,7 +4,6 @@
 .user-list {
     display: flex;
     flex-direction: column;
-    padding: 10px;
     max-width: 250px;
     background-color: rgba(255, 255, 255, 0.025);
     padding: 5px;

frontend/app/view/preview/directorypreview.scss

@@ -238,7 +238,6 @@
     flex-direction: column;
     max-width: 90%;
     max-height: fit-content;
-    display: flex;
     padding: 10px;
     gap: 10px;
     border-radius: 4px;

package.json

@@ -72,7 +72,7 @@
         "prettier-plugin-jsdoc": "^1.3.3",
         "prettier-plugin-organize-imports": "^4.1.0",
         "rollup-plugin-flow": "^1.1.1",
-        "sass": "^1.92.0",
+        "sass": "1.91.0",
         "semver": "^7.7.2",
         "storybook": "^8.6.14",
         "storybook-dark-mode": "^4.0.2",

yarn.lock

@@ -20054,6 +20054,23 @@ __metadata:
   languageName: node
   linkType: hard
 
+"sass@npm:1.91.0":
+  version: 1.91.0
+  resolution: "sass@npm:1.91.0"
+  dependencies:
+    "@parcel/watcher": "npm:^2.4.1"
+    chokidar: "npm:^4.0.0"
+    immutable: "npm:^5.0.2"
+    source-map-js: "npm:>=0.6.2 <2.0.0"
+  dependenciesMeta:
+    "@parcel/watcher":
+      optional: true
+  bin:
+    sass: sass.js
+  checksum: 10c0/5be1c98f7a618cb5f90b62f63d2aa0f78f9bf369c93ec7cd9880752a26b0ead19aa63cc341e8a26ce6c74d080baa5705f1685dff52fe6a3f28a7828ae50182b6
+  languageName: node
+  linkType: hard
+
 "sass@npm:^1.92.0":
   version: 1.92.0
   resolution: "sass@npm:1.92.0"
@@ -22871,7 +22888,7 @@ __metadata:
     remark-github-blockquote-alert: "npm:^1.3.1"
     rollup-plugin-flow: "npm:^1.1.1"
     rxjs: "npm:^7.8.2"
-    sass: "npm:^1.92.0"
+    sass: "npm:1.91.0"
     semver: "npm:^7.7.2"
     shell-quote: "npm:^1.8.3"
     sprintf-js: "npm:^1.1.3"