diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2022-09-20 21:25:48 +0200 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2022-09-24 11:32:15 +0200 |
| commit | 45bee7678319325942664d23688367c639bdf83b (patch) | |
| tree | 84353145a0ecf66da105124597c4b45cb84eebff /sc/source | |
| parent | 5895fb9dac86ecf0cca42dbbb414f69caceae52e (diff) | |
tdf#117764: fix context menu item "Conditional formatting"
Respond to https://bugs.documentfoundation.org/show_bug.cgi?id=117764#c1
a) the function should be *available* on "clean" cells (removed after comment in gerrit)
b) answering "No", one would not edit the existing format, *but will edit a new format instead*
c) answering "Yes", one would edit the existing format
Change-Id: I8c578a59084c557939a853d4746efffbaf4e56e3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140266
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Diffstat (limited to 'sc/source')
| -rw-r--r-- | sc/source/ui/view/cellsh1.cxx | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx index e0ccc37d5545..b3e03a488a6a 100644 --- a/sc/source/ui/view/cellsh1.cxx +++ b/sc/source/ui/view/cellsh1.cxx @@ -2213,6 +2213,7 @@ void ScCellShell::ExecuteEdit( SfxRequest& rReq ) { pCondFormat = pList->GetFormat(rCondFormats[0]); assert(pCondFormat); + nIndex = pCondFormat->GetKey(); bCondFormatDlg = true; } else @@ -2225,7 +2226,9 @@ void ScCellShell::ExecuteEdit( SfxRequest& rReq ) else { // define an overlapping conditional format - // does not need to be handled here + pCondFormat = pList->GetFormat(rCondFormats[0]); + assert(pCondFormat); + bCondFormatDlg = true; } } |