svx/source/tbxctrls/PaletteManager.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 7fede2c47f8a24b80700fe3b82f71a0f60bffd3d
Author: Regina Henschel <rb.hensc...@t-online.de>
AuthorDate: Sun Jul 2 18:14:54 2023 +0200
Commit: Tomaž Vajngerl <qui...@gmail.com>
CommitDate: Tue Jul 4 11:15:13 2023 +0200
tdf#153487 First color item needs mnId = 1
The SvxColorValueSet for the other color palettes is build so that the
first item gets value 1 in its mnId component. But for the new theme
color palette it was 0. That results in the error, that the first
color (top-left in the dialog) could not be selected. An Id of value 0
is handled as 'fail' in various places. Thus use starting mnId = 1 for
theme color palette too.
Change-Id: Ic0b63fee641d4424cabaf25a7f803acd734cbfa3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/153860
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <qui...@gmail.com>
diff --git a/svx/source/tbxctrls/PaletteManager.cxx
b/svx/source/tbxctrls/PaletteManager.cxx
index 607bd60df921..9fcefd34d1c8 100644
--- a/svx/source/tbxctrls/PaletteManager.cxx
+++ b/svx/source/tbxctrls/PaletteManager.cxx
@@ -207,7 +207,7 @@ void PaletteManager::ReloadColorSet(SvxColorValueSet
&rColorSet)
auto pColorSet = pObjectShell->GetThemeColors();
mnColorCount = 12;
rColorSet.Clear();
- sal_uInt16 nItemId = 0;
+ sal_uInt16 nItemId = 1;
if (!pColorSet)
return;
