core.git: svx/source

[Jim Raykowski (via logerrit)]

 svx/source/dialog/ctredlin.cxx |    9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

New commits:
commit d9856b2ca5f3e80c978bd34330648b6345765202
Author:     Jim Raykowski <rayk...@gmail.com>
AuthorDate: Fri Jul 5 11:15:35 2024 -0800
Commit:     Jim Raykowski <rayk...@gmail.com>
CommitDate: Wed Jul 10 17:38:09 2024 +0200

    tdf#161811 related: Fix change manager dialog context popup menu
    
    Sort By > Document Position results
    
    With out the patch installed gtk sorts by the Action column. x11 and qt
    also do something other than expected.
    
    Change-Id: Ic36e79cf928e5ba142b7ff435999e806b1d56214
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170059
    Tested-by: Jenkins
    Reviewed-by: Jim Raykowski <rayk...@gmail.com>

diff --git a/svx/source/dialog/ctredlin.cxx b/svx/source/dialog/ctredlin.cxx
index 504cbe0f5256..49091e01578a 100644
--- a/svx/source/dialog/ctredlin.cxx
+++ b/svx/source/dialog/ctredlin.cxx
@@ -90,7 +90,12 @@ SvxRedlinTable::~SvxRedlinTable()
 
 IMPL_LINK(SvxRedlinTable, HeaderBarClick, int, nColumn, void)
 {
-    if (!bSorted)
+    if (nColumn == -1)
+    {
+        pTreeView->make_unsorted();
+        bSorted = false;
+    }
+    else if (!bSorted)
     {
         pTreeView->make_sorted();
         bSorted = true;
@@ -99,7 +104,7 @@ IMPL_LINK(SvxRedlinTable, HeaderBarClick, int, nColumn, void)
     bool bSortAtoZ = pTreeView->get_sort_order();
 
     //set new arrow positions in headerbar
-    if (nColumn == pTreeView->get_sort_column())
+    if (nColumn != -1 && nColumn == pTreeView->get_sort_column())
     {
         bSortAtoZ = !bSortAtoZ;
         pTreeView->set_sort_order(bSortAtoZ);