Review Request 128402: Kickoff: Add option to hide Favorites tab
John Salatas
jsalatas at gmail.com
Sat Jul 9 18:48:08 UTC 2016
> On July 8, 2016, 9:35 a.m., Kai Uwe Broulik wrote:
> > -1
> >
> > What I would instead like is the ability to re-arrange and hide individual tabs, see Bug 361210, but not a corner-case solution for just the favorites.
>
> John Salatas wrote:
> Yes! That would be great! Not sure if I can implement it through drag and drop, but I could give it a try :\
>
> Kai Uwe Broulik wrote:
> As a first step already having the infrastructure in the applet, ie. a stringlist setting <default>favorites,applications,history,...</default> and the applet placing the tabs as configured would be nice.
>
> A fancy settings UI with drag and drop could be added at a later stage.
>
> David Edmundson wrote:
> Or should we just hide if a model is empty?
>
> We have an existing bug on the leave tab existing even if all the entries are disabled with kiosk.
>
> Kai Uwe Broulik wrote:
> I made a patch for that a while ago (eg. hide History tab when empty) but it was rejected for... "performance" reasons.
Patch updated as per Kai Uwe Broulik's suggestion.
Not sure if I did it the correct way but I guess I didn't as it is actually my first qml application :\
I would appreciate your feedback!
- John
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128402/#review97199
-----------------------------------------------------------
On July 9, 2016, 6:48 p.m., John Salatas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128402/
> -----------------------------------------------------------
>
> (Updated July 9, 2016, 6:48 p.m.)
>
>
> Review request for Plasma.
>
>
> Bugs: 361210
> https://bugs.kde.org/show_bug.cgi?id=361210
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> I'm using the Icons-Only Task Manager and it seems to me that the Favorites tab in kickoff is somewhat redundant, as I have added launchers for all of my favorites in the Icons-Only Task Manager.
> So in this patch I'm introducing an option to hide any menu and also according to bug 36210 to reorganize them.
>
> In the attached screenshot you can see
>
> 1) the configuration options: These are just the menu buttons than can be rearranged via drag n drop and the also include a checkbox (checked: show it, unchecked: hide it).
> 2) The Kickoff menu with just 3 menu items (the other two are hidden).
>
>
> Diffs
> -----
>
> applets/kickoff/package/contents/code/ButtonTools.js PRE-CREATION
> applets/kickoff/package/contents/config/main.xml 2de73ba
> applets/kickoff/package/contents/ui/ConfigButtons.qml PRE-CREATION
> applets/kickoff/package/contents/ui/ConfigGeneral.qml 011b76b
> applets/kickoff/package/contents/ui/FullRepresentation.qml 8ac029c
> applets/kickoff/package/contents/ui/KickoffButton.qml 6b3a2b7
> applets/kickoff/package/contents/ui/KickoffConfigurationButton.qml PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/128402/diff/
>
>
> Testing
> -------
>
> Tested in KDE Neon Developer Stable (as of July 8, 2016)
>
>
> File Attachments
> ----------------
>
> Screenshot
> https://git.reviewboard.kde.org/media/uploaded/files/2016/07/09/3090c598-745d-41d5-9b5c-34191cc85b04__kickoff.png
>
>
> Thanks,
>
> John Salatas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160709/0db93ae5/attachment-0001.html>
More information about the Plasma-devel
mailing list