fix: 30 design issues

Signed-off-by: Luka Trovic <[email protected]>
nextcloud · Aug 29, 2024 · a4f2a99 · a4f2a99
1 parent e647faf
commit a4f2a99
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 3 deletions.
diff --git a/src/App.vue b/src/App.vue
@@ -192,7 +192,7 @@ export default {
 		min-width: 100%;
 		height: 100vh;
 		max-height: calc(100vh - 120px);
-		overflow: auto;
+		overflow: hidden;
 	}
 
 </style>
diff --git a/src/CardMoveDialog.vue b/src/CardMoveDialog.vue
@@ -92,7 +92,6 @@ export default {
 
 <style lang="scss" scoped>
 .modal__content {
-	width: 25vw;
 	min-width: 250px;
 	min-height: 120px;
 	margin: 20px 20px 100px 20px;

diff --git a/src/components/card/CardSidebar.vue b/src/components/card/CardSidebar.vue
@@ -209,6 +209,10 @@ section.app-sidebar__tab--active {
 	height: 100%;
 }
 
+.modal-container {
+	overflow: hidden;
+}
+
 // FIXME: Obivously we should at some point not randomly reuse the sidebar component
 // since this is not oficially supported
 .modal__card .app-sidebar {
@@ -222,10 +226,18 @@ section.app-sidebar__tab--active {
 	max-width: calc(100% - #{$modal-padding * 2});
 	padding: 0 14px;
 	height: 97%;
-	overflow: initial;
+	overflow: hidden !important;
 	user-select: text;
 	-webkit-user-select: text;
 
+	.app-sidebar-header__mainname-container {
+		padding-top: calc(var(--default-grid-baseline, 4px) * 2);
+	}
+
+	.app-sidebar-tabs {
+		max-height: 90%;
+	}
+
 	// FIXME: test
 	&:deep {
 		.app-sidebar-header {

diff --git a/src/components/navigation/AppNavigationBoard.vue b/src/components/navigation/AppNavigationBoard.vue
@@ -350,4 +350,10 @@ export default {
 	.forced-active {
 		box-shadow: inset 4px 0 var(--color-primary-element);
 	}
+
+	:deep(.active) {
+		.material-design-icon svg {
+			fill: var(--color-primary-element-text);
+		}
+	}
 </style>