[Libreoffice-bugs] [Bug 155726] Calc - change the location of the 'Sort by Color' option in the automatic filter menu

2023-06-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=155726

Julien Nabet  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|libreoffice-b...@lists.free |serval2...@yahoo.fr
   |desktop.org |

--- Comment #4 from Julien Nabet  ---
Thank you Heiko for the quick feedback, I've submitted the patch on gerrit:
https://gerrit.libreoffice.org/c/core/+/152734

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 155726] Calc - change the location of the 'Sort by Color' option in the automatic filter menu

2023-06-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=155726

--- Comment #3 from Heiko Tietze  ---
Yes, makes sense. 

(Color vs. Colour... I hope the British English comes from a localization and
the American English is missing yet)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 155726] Calc - change the location of the 'Sort by Color' option in the automatic filter menu

2023-06-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=155726

Julien Nabet  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 CC||heiko.tietze@documentfounda
   ||tion.org,
   ||serval2...@yahoo.fr,
   ||xiscofa...@libreoffice.org
 Ever confirmed|0   |1

--- Comment #2 from Julien Nabet  ---
On pc Debian x86-64 with master sources updated today, I could reproduce this.

Heiko/Xisco: if ok, I can do the change with this straightforward patch:
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 50b85f1596c3..e68300c0b369 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -1077,9 +1077,9 @@ void ScGridWindow::LaunchAutoFilterMenu(SCCOL nCol, SCROW
nRow)
 mpAutoFilterPopup->addMenuItem(
 ScResId(STR_MENU_SORT_DESC),
 new AutoFilterAction(this, AutoFilterMode::SortDescending));
-mpAutoFilterPopup->addSeparator();
 if (ScListSubMenuControl* pSubMenu =
mpAutoFilterPopup->addSubMenuItem(ScResId(SCSTR_SORT_COLOR), true, true))
 pSubMenu->setPopupStartAction(new
AutoFilterColorPopupStartAction(this, pSubMenu, false));
+mpAutoFilterPopup->addSeparator();
 if (ScListSubMenuControl* pSubMenu =
mpAutoFilterPopup->addSubMenuItem(ScResId(SCSTR_FILTER_COLOR), true, true))
 pSubMenu->setPopupStartAction(new
AutoFilterColorPopupStartAction(this, pSubMenu, true));
 if (ScListSubMenuControl* pSubMenu =
mpAutoFilterPopup->addSubMenuItem(ScResId(SCSTR_FILTER_CONDITION), true,
false))

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 155726] Calc - change the location of the 'Sort by Color' option in the automatic filter menu

2023-06-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=155726

--- Comment #1 from kabilo  ---
Created attachment 187775
  --> https://bugs.documentfoundation.org/attachment.cgi?id=187775=edit
change the location of the Sort by Color option

-- 
You are receiving this mail because:
You are the assignee for the bug.