sc/source/ui/view/cellsh1.cxx |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 9330757e4c39b9cecfc2b08294de337129bca317
Author:     Julien Nabet <serval2...@yahoo.fr>
AuthorDate: Tue Sep 20 21:25:48 2022 +0200
Commit:     Adolfo Jayme Barrientos <fit...@ubuntu.com>
CommitDate: Mon Oct 3 16:06:00 2022 +0200

    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 <serval2...@yahoo.fr>
    (cherry picked from commit 45bee7678319325942664d23688367c639bdf83b)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/140882
    Reviewed-by: Adolfo Jayme Barrientos <fit...@ubuntu.com>

diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx
index 17c4936567f9..a48c7ece8e46 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;
                     }
                 }
 

Reply via email to