From fd6b9587566854d98899e95d881eff8de1496d8d Mon Sep 17 00:00:00 2001 From: Gursheen Anand Date: Wed, 4 Feb 2026 23:04:40 +0530 Subject: [PATCH 1/3] fix: border radius and hover state for list row --- frappe/public/scss/desk/list.scss | 37 ++++++++++++++----------------- 1 file changed, 17 insertions(+), 20 deletions(-) diff --git a/frappe/public/scss/desk/list.scss b/frappe/public/scss/desk/list.scss index f2ad07caad..57babd9f12 100644 --- a/frappe/public/scss/desk/list.scss +++ b/frappe/public/scss/desk/list.scss @@ -83,7 +83,8 @@ .list-row-container { display: flex; flex-direction: column; - border-bottom: 1px solid $border-color; + padding-inline: var(--padding-sm); + border-radius: var(--border-radius); &:focus { .list-row { @@ -92,8 +93,19 @@ } &:first-child { - padding-top: 0; - border-bottom: none; + padding: 0; + } + + &:hover:not(.list-row-head) { + background-color: var(--highlight-color); + + .level-right { + box-shadow: -5px 0px 5px var(--highlight-color); + } + } + + &:hover .level-right { + background-color: var(--highlight-color); } } @@ -127,28 +139,12 @@ } .list-row { - border-radius: var(--border-radius); + border-bottom: 1px solid var(--border-color); cursor: pointer; transition: color 0.2s; -webkit-transition: color 0.2s; @include get_textstyle("base", "regular"); - &:hover:not(.list-row-head) { - background-color: var(--highlight-color); - border-radius: unset; - .level-right { - box-shadow: -5px 0px 5px var(--highlight-color); - } - } - - &:hover .level-right { - background-color: var(--highlight-color); - } - - &:last-child { - border-bottom: 0px; - } - .level-left { flex: 4; min-width: 80%; @@ -228,6 +224,7 @@ background-color: var(--subtle-fg); border-radius: var(--border-radius); height: var(--list-row-height); + padding-inline: var(--padding-sm); .list-subject { font-weight: normal; From f8534ac39a3e8fdf5319ba347a944c6c0455b396 Mon Sep 17 00:00:00 2001 From: Gursheen Anand Date: Thu, 5 Feb 2026 00:04:49 +0530 Subject: [PATCH 2/3] fix: hover shadow in row --- frappe/public/scss/desk/list.scss | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/frappe/public/scss/desk/list.scss b/frappe/public/scss/desk/list.scss index 57babd9f12..064cb2c817 100644 --- a/frappe/public/scss/desk/list.scss +++ b/frappe/public/scss/desk/list.scss @@ -98,15 +98,15 @@ &:hover:not(.list-row-head) { background-color: var(--highlight-color); - - .level-right { - box-shadow: -5px 0px 5px var(--highlight-color); - } } &:hover .level-right { background-color: var(--highlight-color); } + + &:last-child { + border-bottom: 0px; + } } .frappe-list { @@ -145,6 +145,12 @@ -webkit-transition: color 0.2s; @include get_textstyle("base", "regular"); + &:hover:not(.list-row-head) { + .level-right { + box-shadow: -5px 0px 5px var(--highlight-color); + } + } + .level-left { flex: 4; min-width: 80%; @@ -235,7 +241,6 @@ } .level-right { background-color: var(--subtle-fg); - border-radius: var(--border-radius); height: var(--list-row-height); border-left: none; &:hover { From ff27ca9922b46e2e0bd9e35aa600254b4eb705cf Mon Sep 17 00:00:00 2001 From: Gursheen Anand Date: Thu, 5 Feb 2026 00:27:52 +0530 Subject: [PATCH 3/3] fix: remove unnecessary selector --- frappe/public/scss/desk/list.scss | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/frappe/public/scss/desk/list.scss b/frappe/public/scss/desk/list.scss index 064cb2c817..f8eb238bb8 100644 --- a/frappe/public/scss/desk/list.scss +++ b/frappe/public/scss/desk/list.scss @@ -96,17 +96,13 @@ padding: 0; } - &:hover:not(.list-row-head) { + &:hover { background-color: var(--highlight-color); } &:hover .level-right { background-color: var(--highlight-color); } - - &:last-child { - border-bottom: 0px; - } } .frappe-list {