Merge pull request #2208 from nextcloud/bugfix/design-fixes

src/components/board/SharingTabSidebar.vue

@@ -198,7 +198,7 @@ export default {
 	}
 
 	.avatardiv {
-		background-color: #f5f5f5;
+		background-color: var(--color-background-dark);
 		border-radius: 16px;
 		width: 32px;
 		height: 32px;

src/components/boards/BoardItem.vue

@@ -25,7 +25,7 @@
 		:title="board.title"
 		:to="routeTo"
 		class="board-list-row"
-		tag="div">
+		tag="a">
 		<div class="board-list-bullet-cell">
 			<div :style="{ 'background-color': `#${board.color}` }" class="board-list-bullet" />
 		</div>
@@ -78,6 +78,13 @@ export default {
 
 <style lang="scss" scoped>
 
+	.board-list-row {
+		&:hover, &:focus {
+			background-color: var(--color-background-hover);
+		}
+		cursor: pointer;
+	}
+
 	.board-list-bullet-cell {
 		padding: 6px 15px;
 

src/components/boards/Boards.vue

@@ -79,10 +79,11 @@ export default {
 
 <style lang="scss">
 	.board-list {
+		margin-top: -44px;
 
 		.board-list-row {
 			align-items: center;
-			border-bottom: 1px solid #ededed;
+			border-bottom: 1px solid var(--color-border);
 			display: flex;
 		}
 
@@ -93,6 +94,7 @@ export default {
 
 		.board-list-header-row {
 			color: var(--color-text-lighter);
+			height: 44px;
 		}
 
 		.board-list-bullet-cell,

src/components/cards/CardItem.vue

@@ -189,11 +189,15 @@ export default {
 	.card {
 		transition: box-shadow 0.1s ease-in-out;
 		box-shadow: 0 0 2px 0 var(--color-box-shadow);
-		border-radius: 3px;
+		border-radius: var(--border-radius-large);
 		font-size: 100%;
 		background-color: var(--color-main-background);
 		margin-bottom: $card-spacing;
 
+		body.dark &, body.theme--dark & {
+			border: 2px solid var(--color-border);
+		}
+
 		&.current-card {
 			box-shadow: 0 0 5px 0 var(--color-box-shadow);
 		}