Review Request 126544: [Kickoff] Hide history tab if empty

[Aleix Pol Gonzalez]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126544/#review90210
-----------------------------------------------------------


Maybe show it as disabled at least? It's weird that UI structure change every now and then.

- Aleix Pol Gonzalez


On Dec. 28, 2015, 3:01 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126544/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2015, 3:01 p.m.)
> 
> 
> Review request for Plasma, KDE Usability and Eike Hein.
> 
> 
> Bugs: 347633
>     https://bugs.kde.org/show_bug.cgi?id=347633
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> This hides the "History" tab if it is empty.
> 
> This can permanently happen if you use disable tracking in activity settings or administrative means.
> 
> 
> Diffs
> -----
> 
>   applets/kickoff/package/contents/ui/FavoritesView.qml 5b7a2cf 
>   applets/kickoff/package/contents/ui/FullRepresentation.qml c2f5ad8 
>   applets/kickoff/package/contents/ui/RecentlyUsedView.qml 7169622 
> 
> Diff: https://git.reviewboard.kde.org/r/126544/diff/
> 
> 
> Testing
> -------
> 
> Button comes and goes as I add and remove history entries.
> 
> When I clear the history, it switches to the Favorites view so you don't end up with a broken view.
> 
> It doesn't seem to track application launches through Kickoff but I think this is unrelated.
> 
> 
> File Attachments
> ----------------
> 
> No history tab
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/12/28/3f2f0697-b5ce-420a-bfc9-8e949b345439__hiddenhistory.jpg
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151228/b96c523b/attachment-0001.html>