fix: 30 design issues

Signed-off-by: Luka Trovic <luka@nextcloud.com>
nextcloud · Aug 21, 2024 · bae066e · bae066e
1 parent 67bd5dc
commit bae066e
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 1 deletion.
diff --git a/src/components/Controls.vue b/src/components/Controls.vue
@@ -496,7 +496,6 @@ export default {
 				border-radius: 50%;
 				background-color: transparent;
 				margin: 12px;
-				margin-left: -4px;
 			}
 		}
 

diff --git a/src/components/board/Board.vue b/src/components/board/Board.vue
@@ -284,6 +284,7 @@ export default {
 					padding-top: 15px;
 					margin-top: -10px;
 					scrollbar-gutter: stable;
+					max-height: 94%;
 				}
 
 				.smooth-dnd-container.vertical > .smooth-dnd-draggable-wrapper {

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 {
@@ -226,6 +230,10 @@ section.app-sidebar__tab--active {
 	user-select: text;
 	-webkit-user-select: text;
 
+	.app-sidebar-header__mainname-container {
+		padding-top: calc(var(--default-grid-baseline, 4px) * 2);
+	}
+
 	// 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>