Skip to content

Commit a8e0d6d

Browse files
committed
[IMP] web: improve date time picker week days colors
In commit[1] the days and week cells got a background, however in darkmode it breaks the elevation. Since neither bg-200 nor bg-400 provides enough contrast it's better to match the background color of the popover in darkmode. task-5238000 [1]: odoo/odoo@f65d9a8 closes odoo#235351 Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
1 parent 5428723 commit a8e0d6d

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
// = DateTime Picker
2+
// ============================================================================
3+
// No CSS hacks, variables overrides only
4+
5+
.o_datetime_picker {
6+
--DateTimePicker__day-week-cell-bg: #{$gray-300};
7+
}

addons/web/static/src/core/datetime/datetime_picker.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,10 @@
8686
}
8787
}
8888

89+
.o_day_of_week_cell, .o_week_number_cell {
90+
background-color: var(--DateTimePicker__day-week-cell-bg, #{$gray-100});
91+
}
92+
8993
.o_week_number_cell {
9094
font-variant: tabular-nums;
9195
}

addons/web/static/src/core/datetime/datetime_picker.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
>
1010
<t t-foreach="month.daysOfWeek" t-as="dayOfWeek" t-key="dayOfWeek[0]">
1111
<div
12-
class="o_day_of_week_cell bg-100 fw-bolder d-flex align-items-center justify-content-center"
12+
class="o_day_of_week_cell fw-bolder d-flex align-items-center justify-content-center"
1313
t-att-title="dayOfWeek[1]"
1414
>
1515
<div class="text-nowrap overflow-hidden" t-esc="props.daysOfWeekFormat === 'narrow' ? dayOfWeek[2] : dayOfWeek[0]"/>
@@ -18,7 +18,7 @@
1818
<t t-foreach="month.weeks" t-as="week" t-key="week.number">
1919
<t t-if="props.showWeekNumbers">
2020
<div
21-
class="o_week_number_cell d-flex justify-content-end align-items-center pe-3 bg-100 fw-bolder"
21+
class="o_week_number_cell d-flex justify-content-center align-items-center fw-bolder"
2222
t-esc="week.number"
2323
/>
2424
</t>

0 commit comments

Comments
 (0)