diff options
| author | Regina Henschel <rb.henschel@t-online.de> | 2023-07-02 18:14:54 +0200 |
|---|---|---|
| committer | Tomaž Vajngerl <quikee@gmail.com> | 2023-07-04 11:15:13 +0200 |
| commit | 7fede2c47f8a24b80700fe3b82f71a0f60bffd3d (patch) | |
| tree | 3994c43d82e4c02740d975fe7865efdc8e793ccc /svx | |
| parent | 8164915520b27b655372e7521b2a58978516f88d (diff) | |
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 <quikee@gmail.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/tbxctrls/PaletteManager.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
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; |