[PATCH] Remove unused argument from gridwin.cxx

Cor Nouws oolst at nouenoff.nl
Tue Jun 12 23:20:10 PDT 2012


Hi,

Eike Rathke wrote (13-06-12 00:56)

> That's the old AutoFilter button dialog, now obsoleted by Kohei's new
> dialog implemented in LaunchAutoFilterMenu(). In fact
> LaunchDataSelectMenu() now is only called with bDataSelect=true so the
> entire !bDataSelect code branch and the parameter can be removed.

I see a big (and growing) difference in possibilities and use case for 
the old en new version of the autofilter. If the code still is there, I 
could try to add a Data > Filter > Autofilter simple as second item to 
the menu.
Would that be an idea to ask at UX, or will it introduce 
code-troubles/ugliness?

Cheers,


-- 
  - Cor
  - http://nl.libreoffice.org



More information about the LibreOffice mailing list