Review Request 106112: Pager QML port
Shaun Reich
shaun.reich at kdemail.net
Mon Feb 11 21:39:43 UTC 2013
> On Feb. 9, 2013, 6:49 p.m., Mark Gaiser wrote:
> > Just wanted to add that it has a regression that slipped in the 4.10.0 release. https://bugs.kde.org/show_bug.cgi?id=312684
> >
> > If you have only one virtual desktop the pager just shows an empty area. That is obviously wrong. It is actually very clearly visible in this - long - review: http://www.muktware.com/5194/kde-410-review-time-switch-kde. Just look at the first image, the gap between activity icon and dolphin is the pager.
>
> Luís Gabriel Lima wrote:
> Actually it's not a regression. As you can see in the history of this review, it was requested to hide the pager when there is only one virtual desktop. Although I can see a bug in the current implementation, the pager should be considerably small (practically hidden). Right now it's just making the virtual desktop rect invisible and keeping the same size.
Oh it most certainly is on at least some level. Yes, if the pager is going to be hidden when one desktop is there, it shouldn't ghostily take up any room. But I don't think hiding the widget just because there is one desktop is a solution. In fact, I have seen users get confused around this, because their system is set to 1 VD and they add a pager widget, and what happens? Nothing. So they add it a few more times. Then they try adding it to a different panel, it still isn't working.
Turns out I had to tell them that they had to change the VD count to > 1, and surprise surprise! there's 20 pager widgets all over the desktop.
This is another case of someone trying to make something look clean and minimalistic, but not thinking about the consequences when it comes to user interaction.
If you don't like how the pager widget looks with 1 desktop, then don't ship it by default with 1 desktop, or don't ship the pager widget at all on a default desktop (which honestly I don't think many users(aka average) would use it in a default desktop).
Turning the pager widget into a ghost widget is definitely not the answer and we're going to get a lot of issues because of that, as I noted above, because I witnessed them first hand... Making it invisible really just pushes the confusion upward a bit more.
- Shaun
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106112/#review27093
-----------------------------------------------------------
On Aug. 30, 2012, 2:30 a.m., Luís Gabriel Lima wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106112/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2012, 2:30 a.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> This patch contains the QML port of the Pager plasmoid done during the GSoC 2012.
>
> In this port basically I used QML to paint the Pager UI and deal with the user interaction.
> The geometry calculation of the desktop and window rectangles was kept in C++ as well as other routines that needs to interact with classes like KWindowSystem, QDbusConnection and so on.
> This patch also introduces the PagerModel, a QAIM subclass that holds the desktop/window geometries and is used by the QML part to fill the UI.
>
>
> Diffs
> -----
>
> plasma/desktop/applets/pager/CMakeLists.txt 5d80514
> plasma/desktop/applets/pager/model.h PRE-CREATION
> plasma/desktop/applets/pager/model.cpp PRE-CREATION
> plasma/desktop/applets/pager/package/contents/ui/main.qml PRE-CREATION
> plasma/desktop/applets/pager/package/contents/ui/utils.js PRE-CREATION
> plasma/desktop/applets/pager/package/metadata.desktop PRE-CREATION
> plasma/desktop/applets/pager/pager.h 6c7c045
> plasma/desktop/applets/pager/pager.cpp 74dc529
>
> Diff: http://git.reviewboard.kde.org/r/106112/diff/
>
>
> Testing
> -------
>
> - Tested inside panels and floating on desktop, sizing works as expected
> - Mouse interactions (move windows around, change desktop, etc)
>
>
> Screenshots
> -----------
>
>
> http://git.reviewboard.kde.org/r/106112/s/691/
>
> http://git.reviewboard.kde.org/r/106112/s/692/
>
>
> Thanks,
>
> Luís Gabriel Lima
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130211/15395912/attachment-0001.html>
More information about the Plasma-devel
mailing list