mirror of
https://github.com/odoo/design-themes.git
synced 2025-10-07 01:18:52 +07:00
[FIX] *: fix duplicate palette crashing render in theme tab
*: theme_anelusia, theme_avantgarde, theme_bookstore, theme_buzzy, theme_real_estate, theme_treehouse Since website refactor [1], a silent error would occur while rendering the theme tab, because of the use of `t-foreach`, leading to nothing being displayed depending of the theme. This was caused by a duplicate key in the `$o-selected-color-palettes-names` scss variable, added by some theme, which would make the `t-foreach` crash in `website.ThemeColorsOption`. Steps to reproduce: - Create a website using any of the above themes, through the wizard - Enter edit mode - Click on theme tab - Nothing is rendered [1]: https://github.com/odoo/odoo/commit/9fe45e2b7ddbbfd0445ffe25a859e67a316d02b2 opw-5020952 opw-5040694 closes odoo/design-themes#1132 Related: odoo/odoo#224076 Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
This commit is contained in:
committed by
qsm-odoo
parent
59f35e4dde
commit
d53ae16a85
@@ -131,8 +131,6 @@ $o-theme-color-palettes: map-merge($o-theme-color-palettes,
|
||||
)
|
||||
);
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-25');
|
||||
|
||||
$o-color-palettes-compatibility-indexes: (
|
||||
1: 'anelusia-1',
|
||||
2: 'anelusia-2',
|
||||
|
||||
@@ -186,8 +186,6 @@ $o-website-values-palettes: (
|
||||
),
|
||||
);
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-15');
|
||||
|
||||
$o-color-palettes-compatibility-indexes: (
|
||||
1: 'avantgarde-1',
|
||||
2: 'avantgarde-2',
|
||||
|
||||
@@ -74,8 +74,6 @@ $o-website-values-palettes: (
|
||||
),
|
||||
);
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-26');
|
||||
|
||||
$o-color-palettes-compatibility-indexes: (
|
||||
1: 'bookstore-1',
|
||||
2: 'bookstore-2',
|
||||
|
||||
@@ -32,12 +32,6 @@ $o-website-values-palettes: (
|
||||
),
|
||||
);
|
||||
|
||||
//------------------------------------------------------------------------------
|
||||
// Palettes
|
||||
//------------------------------------------------------------------------------
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-24');
|
||||
|
||||
//------------------------------------------------------------------------------
|
||||
// Fonts
|
||||
//------------------------------------------------------------------------------
|
||||
|
||||
@@ -93,8 +93,6 @@ $o-website-values-palettes: (
|
||||
),
|
||||
);
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-28');
|
||||
|
||||
$o-color-palettes-compatibility-indexes: (
|
||||
1: 'real-estate-1',
|
||||
2: 'real-estate-2',
|
||||
|
||||
@@ -159,8 +159,6 @@ $o-theme-color-palettes: map-merge($o-theme-color-palettes,
|
||||
)
|
||||
);
|
||||
|
||||
$o-selected-color-palettes-names: append($o-selected-color-palettes-names, 'default-28');
|
||||
|
||||
$o-color-palettes-compatibility-indexes: (
|
||||
1: 'treehouse-1',
|
||||
2: 'treehouse-2',
|
||||
|
||||
Reference in New Issue
Block a user