> On June 22, 2013, 3:24 p.m., Aaron J. Seigo wrote: > > For absolute completeness, the sortCaseSensitivity property should also > > have a NOTIFY signal .. but i don't see why anyone would ever bind to that > > signal and it would require a reimplementation of the set method, so i'm > > fine with it going in without .. so a +1 from me; if Marco can confirm then > > this is good to go in.
my blessings :p - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111148/#review34881 ----------------------------------------------------------- On June 22, 2013, 2:42 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111148/ > ----------------------------------------------------------- > > (Updated June 22, 2013, 2:42 p.m.) > > > Review request for Plasma. > > > Description > ------- > > This adds support for the sortCaseSensitivity property of > QSortFilterProxyModel to the PlasmaCore.SortFilterModel. Allows to sort items > case insensitive. > > > Diffs > ----- > > plasma/declarativeimports/core/datamodel.h 0943b68 > plasma/declarativeimports/core/datamodel.cpp f81e579 > > Diff: http://git.reviewboard.kde.org/r/111148/diff/ > > > Testing > ------- > > Tested with the battery monitor. Now "eva" comes before "Logitech" :) > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
