Re: Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-06 Thread Ovidiu-Florin BOGDAN
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125061/ --- (Updated Sept. 7, 2015, 8:57 a.m.) Status -- This change has been di

Re: Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-06 Thread Ovidiu-Florin BOGDAN
> On Sept. 6, 2015, 5:25 p.m., Vishesh Handa wrote: > > I'm fairly against this. Maybe others can chime in? > > > > My rationale is that when configuring Baloo, I don't want the first > > impression to be - Here is how you can disable it. I would prefer it to be > > something along the lines o

Re: Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-06 Thread Ovidiu-Florin BOGDAN
> On Sept. 6, 2015, 8:01 p.m., Heiko Tietze wrote: > > The design does not fit the (preliminary) KCM guidelines [1] and has a > > questionale workflow as Vishesh commented wrt. the checkbox plus the > > stacked lists are not a common design. To solve the problkem I'm thinking > > about using t

Re: Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-06 Thread Heiko Tietze
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125061/#review84901 --- The design does not fit the (preliminary) KCM guidelines [1] a

Re: Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-06 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125061/#review84897 --- I'm fairly against this. Maybe others can chime in? My ration

Review Request 125061: Moved indexing checkbox to the top of the KCM

2015-09-05 Thread Ovidiu-Florin BOGDAN
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125061/ --- Review request for Plasma and KDE Usability. Repository: plasma-desktop