On Tue, Dec 4, 2012 at 11:21 PM, Gabriel Poesia <[email protected]>wrote:
> Would putting the button in the "Indexing" tab be a good idea? In the > advanced tab, I would have to add another QGroupBox to contextualize the > button (or it would appear too different from the Memory Usage box), but I > don't know what to put in the group besides the button, or its title (a > short label, maybe, but the button is quite self-explanatory, so it looks > unnecessary). I'm not good at UI design either. I think another group box under advanced would be okay. Just that button is okay, you can add a spacer and a button if you want. I'll ask some of the UI experts to clear it up later. > > > 2012/12/4 Vishesh Handa <[email protected]> > >> This is an automatically generated e-mail. To reply, visit: >> http://git.reviewboard.kde.org/r/107553/ >> >> Review board has a way to add screenshots. It's kinda cool. >> >> Anyway, I'm not sure if adding it to the status widget it a good idea. The >> same status widget is also uses in the Nepomuk Controller. I was hoping that >> it could be added in the advanced section of the kcm. We could even add >> other advanced stuff like clearing all the indexed data. >> >> I should point out, that I'm not so great with UI stuff. >> >> >> - Vishesh >> >> On December 2nd, 2012, 6:34 p.m., Gabriel Poesia wrote: >> Review request for Nepomuk, Vishesh Handa and Sebastian Trueg. >> By Gabriel Poesia. >> >> *Updated Dec. 2, 2012, 6:34 p.m.* >> Description >> >> Add a button to the KCM Nepomuk as suggested in bug #303658. When clicked, >> the indexer executes the updateAllFolders() method. >> The widget was placed in the Nepomuk Repository Details dialog, along with >> the "Suspend/Resume File Indexing". I'm not sure if it violates >> any design principles behind KDE, KCM or Nepomuk (if it should have an icon, >> for example, like the two other buttons in the dialog). >> >> Testing >> >> Compiled, installed, ran kcmshell4 nepomuk. The file indexer was idle. >> Clicking the button changed the status message to "Scanning for recent >> changes in [...]". >> >> *Bugs: * 303658 <http://bugs.kde.org/show_bug.cgi?id=303658> >> Diffs >> >> - nepomuk/kcm/statuswidget.h (088799e) >> - nepomuk/kcm/statuswidget.cpp (36e66f5) >> - nepomuk/kcm/statuswidget.ui (46e4d26) >> >> View Diff <http://git.reviewboard.kde.org/r/107553/diff/> >> > > -- Vishesh Handa
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
