From wengxt at gmail.com Thu Aug 1 01:23:12 2013
From: wengxt at gmail.com (Xuetian Weng)
Date: Thu, 01 Aug 2013 01:23:12 -0000
Subject: Review Request 111828: fix link click in notification
Message-ID: <20130801012312.18279.55171@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111828/
-----------------------------------------------------------
Review request for Plasma and Marco Martin.
Description
-------
handle the linkActivated signal in both popup and notification history.
now click on link can open browser (or anything it should be) correctly.
Diffs
-----
plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml ca7564f
plasma/generic/applets/notifications/contents/ui/LastNotificationPopup.qml 29a6a2d
Diff: http://git.reviewboard.kde.org/r/111828/diff/
Testing
-------
No problem, and seems the drag operation is not affected (if not dragging from link).
test with:
notify-send "test link"
Thanks,
Xuetian Weng
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Thu Aug 1 09:16:11 2013
From: sebas at kde.org (=?utf-8?q?Sebastian_K=C3=BCgler?=)
Date: Thu, 01 Aug 2013 09:16:11 -0000
Subject: Review Request 111828: fix link click in notification
In-Reply-To: <20130801012312.18279.55171@vidsolbach.de>
References: <20130801012312.18279.55171@vidsolbach.de>
Message-ID: <20130801091611.19831.44732@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111828/#review36897
-----------------------------------------------------------
Ship it!
Ship It!
- Sebastian K?gler
On Aug. 1, 2013, 1:23 a.m., Xuetian Weng wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111828/
> -----------------------------------------------------------
>
> (Updated Aug. 1, 2013, 1:23 a.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> handle the linkActivated signal in both popup and notification history.
>
> now click on link can open browser (or anything it should be) correctly.
>
>
> Diffs
> -----
>
> plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml ca7564f
> plasma/generic/applets/notifications/contents/ui/LastNotificationPopup.qml 29a6a2d
>
> Diff: http://git.reviewboard.kde.org/r/111828/diff/
>
>
> Testing
> -------
>
> No problem, and seems the drag operation is not affected (if not dragging from link).
>
> test with:
> notify-send "test link"
>
>
> Thanks,
>
> Xuetian Weng
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Thu Aug 1 20:28:21 2013
From: null at kde.org (Commit Hook)
Date: Thu, 01 Aug 2013 20:28:21 -0000
Subject: Review Request 111828: fix link click in notification
In-Reply-To: <20130801012312.18279.55171@vidsolbach.de>
References: <20130801012312.18279.55171@vidsolbach.de>
Message-ID: <20130801202821.16329.19385@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111828/#review36944
-----------------------------------------------------------
This review has been submitted with commit 20765b4479db5d3788ef799fa45bad5503075a09 by Weng Xuetian to branch KDE/4.11.
- Commit Hook
On Aug. 1, 2013, 1:23 a.m., Xuetian Weng wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111828/
> -----------------------------------------------------------
>
> (Updated Aug. 1, 2013, 1:23 a.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> handle the linkActivated signal in both popup and notification history.
>
> now click on link can open browser (or anything it should be) correctly.
>
>
> Diffs
> -----
>
> plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml ca7564f
> plasma/generic/applets/notifications/contents/ui/LastNotificationPopup.qml 29a6a2d
>
> Diff: http://git.reviewboard.kde.org/r/111828/diff/
>
>
> Testing
> -------
>
> No problem, and seems the drag operation is not affected (if not dragging from link).
>
> test with:
> notify-send "test link"
>
>
> Thanks,
>
> Xuetian Weng
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Thu Aug 1 20:28:25 2013
From: null at kde.org (Commit Hook)
Date: Thu, 01 Aug 2013 20:28:25 -0000
Subject: Review Request 111828: fix link click in notification
In-Reply-To: <20130801012312.18279.55171@vidsolbach.de>
References: <20130801012312.18279.55171@vidsolbach.de>
Message-ID: <20130801202825.16880.20989@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111828/
-----------------------------------------------------------
(Updated Aug. 1, 2013, 8:28 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Marco Martin.
Description
-------
handle the linkActivated signal in both popup and notification history.
now click on link can open browser (or anything it should be) correctly.
Diffs
-----
plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml ca7564f
plasma/generic/applets/notifications/contents/ui/LastNotificationPopup.qml 29a6a2d
Diff: http://git.reviewboard.kde.org/r/111828/diff/
Testing
-------
No problem, and seems the drag operation is not affected (if not dragging from link).
test with:
notify-send "test link"
Thanks,
Xuetian Weng
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From bugzilla_noreply at kde.org Fri Aug 2 06:00:10 2013
From: bugzilla_noreply at kde.org (bugzilla_noreply at kde.org)
Date: Fri, 02 Aug 2013 06:00:10 +0000
Subject: [KDE Bugtracking System] REMINDER: current Plasma regressions
Message-ID:
Please find below a list of the current regressions reported for Plasma. This is a weekly reminder.
This search was scheduled by myriam at kde.org.
Plasma regressions
------------------
Bug 301424:
https://bugs.kde.org/show_bug.cgi?id=301424
Priority: NOR Severity: normal Platform: openSUSE RPMs
Assignee: plasma-bugs at kde.org
Status: REOPENED
Summary: Cannot open battery monitor applet if set to hidden in systray
Bug 311354:
https://bugs.kde.org/show_bug.cgi?id=311354
Priority: NOR Severity: normal Platform: unspecified
Assignee: plasma-bugs at kde.org
Status: CONFIRMED
Summary: System Tray widget can't be resized to fit all icons in one row when added to the desktop
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Fri Aug 2 13:37:46 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 02 Aug 2013 15:37:46 +0200
Subject: Add custom first week support to KDE digital clock.
In-Reply-To:
References:
<91612708.EgajKWTZLQ@monet>
Message-ID: <1543638.izhrCgAe5k@monet>
Hi Hu Zheng,
Thanks for the patch, and welcome to Plasma development!
On Friday, August 02, 2013 20:01:19 Hu Zheng wrote:
> Here is the custom first week patch. Can anyone add it to the source
> repository?
We use reviewboard to review changes to Plasma source code. This way, we can
look at the changes in detail, improve things that need fixing, and review for
example changes in the UI.
Could you post your patch to our Reviewboard instance at
http://git.reviewboard.kde.org and assign it to the Plasma group?
A good explanation of what your code tries to achieve and how it does that
helps reviewers to understand the changes.
As your code introduces visual changes, please also attach screenshots (before
and after) to the review request.
Also, a note on our development cycle. kde-workspace is feature-frozen, as we
move to development of the upcoming version of Plasma. This means that any new
feature has to target Plasma Workspaces 2, which we intend to release next
summer.
Kind regards,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From jpwhiting at kde.org Fri Aug 2 20:14:15 2013
From: jpwhiting at kde.org (Jeremy Whiting)
Date: Fri, 2 Aug 2013 14:14:15 -0600
Subject: Fixing plasma theme removal
Message-ID:
Hello,
Today I applied and pushed a fix to make uninstalling from KNS3 for most
cases work again, however https://bugs.kde.org/show_bug.cgi?id=149479 is
still broken for plasma theme uninstallation and here's why:
PlasmaStructure doesn't accept application/x-xz-compressed-tar format files
which 'Helium' theme is. Then plasmapkg -t theme -r filename is failing
since it's actually expecting the name of the theme rather than a plasma
package archive file. Looking at the plasma-themes.knsrc we use plasmapkg
-t theme -i %f to install which works fine for .xz .gz and such archives,
but then uninstall we use plasmapkg -t theme -r %f with the same filename
and it fails.
So we either need to add a new command to plasmapkg to inspect a plasma
package to get it's name then we could do something like 'plasmapkg -r
`plasmapkg -n %f`' for removals if we added -n to get a plasma package's
name.
This "solution" has a problem though because opening the archives of plasma
themes fails since their metadata.desktop file's are in a subfolder of the
archive. Even for plasmapkg -i themefile there's a warning saying the
metadata.desktop file wasn't found where it was expected. So we also need
to make plasmapkg extract the KTar directory's sources without the top
level folder, or make PlasmaStructure dive a folder deeper to look for the
metadata.desktop.
Any suggestions on which approach I should take would be greatly
appreciated.
thanks,
Jeremy
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From huzheng001 at gmail.com Sat Aug 3 03:32:32 2013
From: huzheng001 at gmail.com (Hu Zheng)
Date: Sat, 03 Aug 2013 03:32:32 -0000
Subject: Review Request 111849: custom first week for plasmaclock.
Message-ID: <20130803033232.19709.65658@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111849/
-----------------------------------------------------------
Review request for Plasma.
Description
-------
In normal case, the first week is the first week in the year, but the teachers may want to custom the first week, for example, make the 8th week as the first week, then the 9th week will be the second week, and so on. Then the teacher can easily do their work as a custom teaching period!
Diffs
-----
Diff: http://git.reviewboard.kde.org/r/111849/diff/
Testing
-------
cd kde-workspace-4.10.5/libs/plasmaclock
patch -p1 < customfirstweek.patch
The save configuration codes are not very beautiful, but it works fine!
Thanks,
Hu Zheng
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From huzheng001 at gmail.com Sat Aug 3 03:36:33 2013
From: huzheng001 at gmail.com (Hu Zheng)
Date: Sat, 03 Aug 2013 03:36:33 -0000
Subject: Review Request 111849: custom first week for plasmaclock.
In-Reply-To: <20130803033232.19709.65658@vidsolbach.de>
References: <20130803033232.19709.65658@vidsolbach.de>
Message-ID: <20130803033633.19898.17573@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111849/
-----------------------------------------------------------
(Updated Aug. 3, 2013, 3:36 a.m.)
Review request for Plasma.
Changes
-------
Add patch.
Description
-------
In normal case, the first week is the first week in the year, but the teachers may want to custom the first week, for example, make the 8th week as the first week, then the 9th week will be the second week, and so on. Then the teacher can easily do their work as a custom teaching period!
Diffs
-----
Diff: http://git.reviewboard.kde.org/r/111849/diff/
Testing
-------
cd kde-workspace-4.10.5/libs/plasmaclock
patch -p1 < customfirstweek.patch
The save configuration codes are not very beautiful, but it works fine!
File Attachments (updated)
----------------
patch
http://git.reviewboard.kde.org/media/uploaded/files/2013/08/03/customfirstweek.patch
Thanks,
Hu Zheng
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Sat Aug 3 10:02:25 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Sat, 3 Aug 2013 10:02:25 +0000
Subject: dirmodel in plasma-framework
Message-ID:
Hi ,
Is dirModel ported to plasma2 ? As when in dirModelplugin.cpp I changed
qmlRegisterType(uri, 0, 1, "DirModel") to
qmlRegisterType(uri, 2,0, "DirModel") I am getting an output in
my trashcan plasmoid that url trash:/ can not be listed .However it worked
fine with plasma1 .
Further I want to know how shall I be able to use data engines in plasma2
as window-list requires tasks data engine .
Regards
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Sat Aug 3 10:07:29 2013
From: notmart at gmail.com (Marco Martin)
Date: Sat, 3 Aug 2013 12:07:29 +0200
Subject: dirmodel in plasma-framework
In-Reply-To:
References:
Message-ID: <201308031207.30002.notmart@gmail.com>
On Saturday 03 August 2013, Heena Mahour wrote:
> Hi ,
> Is dirModel ported to plasma2 ? As when in dirModelplugin.cpp I changed
> qmlRegisterType(uri, 0, 1, "DirModel") to
> qmlRegisterType(uri, 2,0, "DirModel") I am getting an output in
> my trashcan plasmoid that url trash:/ can not be listed .However it worked
> fine with plasma1 .
> Further I want to know how shall I be able to use data engines in plasma2
> as window-list requires tasks data engine .
> Regards
yes, dirmodel exists
plasma-framework/src/declarativeimports/dirmodel/
(and yes, changing the version to 2,0 is correct, i forgot that, can you
push?)
the reason this doesn't work yet, is probably that most kioslaves aren't
ported yet, so it will need a bit of blind navigation still :/
--
Marco Martin
From heena393 at gmail.com Sat Aug 3 11:41:44 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Sat, 3 Aug 2013 11:41:44 +0000
Subject: dirmodel in plasma-framework
In-Reply-To: <201308031207.30002.notmart@gmail.com>
References:
<201308031207.30002.notmart@gmail.com>
Message-ID:
I have pushed it in branch heenamahour/gsoc (plasma-framework)
http://quickgit.kde.org/?p=plasma-framework.git&a=commit&h=18266f3ab78108a3246bac3540cb3aef6f97bbe6
>the reason this doesn't work yet, is probably that most kioslaves aren't
ported yet, so it will need a bit of blind navigation still
So, shall I port the plasma2 plasmoid I ported I guess it will work
completely when plasma-framework will be build completely ?
On Sat, Aug 3, 2013 at 10:07 AM, Marco Martin wrote:
> On Saturday 03 August 2013, Heena Mahour wrote:
> > Hi ,
> > Is dirModel ported to plasma2 ? As when in dirModelplugin.cpp I changed
> > qmlRegisterType(uri, 0, 1, "DirModel") to
> > qmlRegisterType(uri, 2,0, "DirModel") I am getting an output
> in
> > my trashcan plasmoid that url trash:/ can not be listed .However it
> worked
> > fine with plasma1 .
> > Further I want to know how shall I be able to use data engines in plasma2
> > as window-list requires tasks data engine .
> > Regards
> yes, dirmodel exists
> plasma-framework/src/declarativeimports/dirmodel/
> (and yes, changing the version to 2,0 is correct, i forgot that, can you
> push?)
>
> the reason this doesn't work yet, is probably that most kioslaves aren't
> ported yet, so it will need a bit of blind navigation still :/
>
> --
> Marco Martin
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ivan.cukic at kde.org Sat Aug 3 17:34:32 2013
From: ivan.cukic at kde.org (Ivan =?utf-8?B?xIx1a2nEhw==?=)
Date: Sat, 03 Aug 2013 19:34:32 +0200
Subject: Branch review request
Message-ID: <1967303.8KXXA0zMFk@drako>
Hi all,
I've realized that it will be painful if I wait until everything is finished
to merge into master, and that there is not much reason for it to be so since
the master is in a playground mode anyway.
One of the reasons for the request is that I'd like to do a few things in
parallel to the main work - cleaning up the whole plasma code, use allowed
levels of C++11 (and the 'spiffy' d-pointers as Aaron called them)...
So, I'm requesting a review of the ivan/shell-switching branch.
There are some things that are removed and should be returned, and there are
some things that should be removed before the merger:
- panels (to return after merger, a bit more complex)
- desktopcorona (to be removed after the panels are added to shellcorona)
- src/shells folder can be merged into qmlpackages*(1) (and binary parts of
src/shells removed - those are not needed, but was a good example/test of more
advanced mechanism that can be achieved - for example, shells providing custom
components etc.) - this will be greatly simplified very soon!
- change config names
What works:
(required: git at git.kde.org:scratch/ivan/solidx,
kde:scratch/ivan/platformcomponents - these should be merged into
declarativeimports)
Currently, we have two shells - the org.kde.desktop and org.kde.blank. When
you plug in a touch-screen, the blank one is loaded. And the desktop is loaded
again when you remove the device.
There is a dbus interface for fake hw:
qdbus org.kde.SolidXTest /FakeInput addDevice fake FakeTouch 2 2
qdbus org.kde.SolidXTest /FakeInput removeDevice fake
Obviously, the loading conditions will be changed when we get more into the
shell production mode. Now, those are only there for testing purposes.
*(1) Shells folder can be merged with the parts of qmlpackages that are
concerning shells. What I'd like to do is to move those into a separate folder
- to be more obvious what is where.
But the problem is that it would mean having src/shells and src/shell as it is
currently in the branch. What I'd like to have is a following structure:
lib/plasma
lib/declarativeimports
lib/kpart
application (or similar - for plasma-shell - core, app, main, ...)
something/shells
something/qmlpackages
something/plasmapkg (have no idea what 'something' could be)
*(2) The other thing I wanted to ask is can I rename org.kde.desktop to
org.kde.plasma.desktop?
Cheerio,
Ivan
--
I don't really trust a sane person.
-- Lyle Alzado
From akshay_ratan at yahoo.com Sat Aug 3 18:10:44 2013
From: akshay_ratan at yahoo.com (Akshay Ratan)
Date: Sat, 03 Aug 2013 18:10:44 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
Message-ID: <20130803181044.23880.16241@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/
-----------------------------------------------------------
Review request for Plasma, Shantanu Tushar and Sinny Kumari.
Description
-------
Folder Contents(Images) are previewed when we browse the local directories for Pictures.
Diffs
-----
browsingbackends/localfiles/localpictures/CMakeLists.txt 2acbd59
browsingbackends/localfiles/localpictures/localpicturesmodel.h d61f2a3
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp afb852f
browsingbackends/localfiles/localpictures/localpicturesthumbnailprovider.h PRE-CREATION
browsingbackends/localfiles/localpictures/localpicturesthumbnailprovider.cpp PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111863/diff/
Testing
-------
Yes. Works fine.
Thanks,
Akshay Ratan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ivan.cukic at kde.org Sun Aug 4 06:58:37 2013
From: ivan.cukic at kde.org (=?UTF-8?B?SXZhbiDEjHVracSH?=)
Date: Sun, 4 Aug 2013 08:58:37 +0200
Subject: Branch review request
In-Reply-To: <1967303.8KXXA0zMFk@drako>
References: <1967303.8KXXA0zMFk@drako>
Message-ID:
A small addendum - the ivan/shell-switching branch is sufficient - no
need for the scratch repos.
The loaders are quite simplified.
The blank shell behaves like failsafe - it loads xterm application
when shown - don't know how it behaves when xterm is not installed.
--
Cheerio,
Ivan
--
While you were hanging yourself on someone else's words
Dying to believe in what you heard
I was staring straight into the shining sun
From sebas at kde.org Sun Aug 4 14:21:27 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Sun, 04 Aug 2013 16:21:27 +0200
Subject: Add custom first week support to KDE digital clock.
In-Reply-To:
References:
<1543638.izhrCgAe5k@monet>
Message-ID: <2117394.I5ydiA5Mtz@monet>
Hi Hu Zheng,
On Saturday, August 03, 2013 11:42:22 Hu Zheng wrote:
> Done! See:
> https://git.reviewboard.kde.org/r/111849/
Please submit the patch as patch on reviewboard, not as attachment. That way,
it can be read easily from the web interface.
Please also attach screenshots of any UI that has been changed.
Thanks,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From huzheng001 at gmail.com Fri Aug 2 12:01:19 2013
From: huzheng001 at gmail.com (Hu Zheng)
Date: Fri, 2 Aug 2013 20:01:19 +0800
Subject: Add custom first week support to KDE digital clock.
In-Reply-To: <91612708.EgajKWTZLQ@monet>
References:
<2138507.narW42gxtJ@monet>
<91612708.EgajKWTZLQ@monet>
Message-ID:
Here is the custom first week patch. Can anyone add it to the source
repository?
cd kde-workspace-4.10.5/libs/plasmaclock
patch -p1 < customfirstweek.patch
The save configuration codes are not very beautiful, but it works fine!
2013/7/18 Sebastian K?gler
> On Sunday, July 14, 2013 15:49:04 Hu Zheng wrote:
> > "Now I am not sure what you mean with first week support, maybe you could
> > explain that?"
> > In normal case, the first week is the first week in the year, but the
> > teachers may want to custom the first week, for example, make the 8th
> week
> > as the first week, then the 9th week will be the second week, and so on.
> > Then the teacher can easily do their work as a custom teaching period!
>
> Thanks for the explanation!
> --
> sebas
>
> http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
>
--
http://www.huzheng.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: customfirstweek.patch
Type: application/octet-stream
Size: 12956 bytes
Desc: not available
URL:
From frank78ac at googlemail.com Fri Aug 2 15:45:58 2013
From: frank78ac at googlemail.com (Frank Reininghaus)
Date: Fri, 2 Aug 2013 17:45:58 +0200
Subject: Unintended changes at community.kde.org?
Message-ID:
Hi everyone,
(please CC me in any replies, I'm not subscribed)
I recently had a look at the Dolphin pages at community.kde.org. We
don't actually use them much - Peter and I mostly used
http://community.kde.org/Dolphin/Dolphin-2.0-Status to list the things
that still needed to be done for the new view engine before the KDE
4.8 release (the page is mostly empty now because the vast majority of
these issues had been resolved a long time ago).
I found out that a certain user 'Cyberhacker' has recently started
adding links to bug reports to the page:
http://community.kde.org/index.php?title=Dolphin/Dolphin-2.0-Status&action=history
It's not hard to guess which bugs.kde.org user 'Cyberhacker' is.
The reason why I'm telling you about this is that he did the same
thing with the "Plasma/Tasks" page:
http://community.kde.org/index.php?title=Plasma/Tasks&action=history
I don't know how much you use that page, but if you do, be aware that
not all content on the page is made by the Plasma team.
Moreover, 'Cyberhacker' has set up this page:
http://community.kde.org/Back_to_basics
which is linked to from the community.kde.org start page with the
comment "Let's make KDE better!" - I'm not sure if we want that.
I wish everyone a nice weekend!
Best regards,
Frank
From huzheng001 at gmail.com Sat Aug 3 03:42:22 2013
From: huzheng001 at gmail.com (Hu Zheng)
Date: Sat, 3 Aug 2013 11:42:22 +0800
Subject: Add custom first week support to KDE digital clock.
In-Reply-To: <1543638.izhrCgAe5k@monet>
References:
<91612708.EgajKWTZLQ@monet>
<1543638.izhrCgAe5k@monet>
Message-ID:
Done! See:
https://git.reviewboard.kde.org/r/111849/
Thank you very much!
2013/8/2 Sebastian K?gler
> Hi Hu Zheng,
>
> Thanks for the patch, and welcome to Plasma development!
>
> On Friday, August 02, 2013 20:01:19 Hu Zheng wrote:
> > Here is the custom first week patch. Can anyone add it to the source
> > repository?
>
> We use reviewboard to review changes to Plasma source code. This way, we
> can
> look at the changes in detail, improve things that need fixing, and review
> for
> example changes in the UI.
>
> Could you post your patch to our Reviewboard instance at
> http://git.reviewboard.kde.org and assign it to the Plasma group?
>
> A good explanation of what your code tries to achieve and how it does that
> helps reviewers to understand the changes.
> As your code introduces visual changes, please also attach screenshots
> (before
> and after) to the review request.
>
> Also, a note on our development cycle. kde-workspace is feature-frozen, as
> we
> move to development of the upcoming version of Plasma. This means that any
> new
> feature has to target Plasma Workspaces 2, which we intend to release next
> summer.
>
> Kind regards,
> --
> sebas
>
> http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
>
--
http://www.huzheng.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From akshay_ratan at yahoo.com Mon Aug 5 16:33:42 2013
From: akshay_ratan at yahoo.com (Akshay Ratan)
Date: Mon, 05 Aug 2013 16:33:42 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
In-Reply-To: <20130803181044.23880.16241@vidsolbach.de>
References: <20130803181044.23880.16241@vidsolbach.de>
Message-ID: <20130805163342.31306.78845@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/
-----------------------------------------------------------
(Updated Aug. 5, 2013, 4:33 p.m.)
Review request for Plasma, Shantanu Tushar and Sinny Kumari.
Changes
-------
Thumbnail Provider class relocated so that now both picturemodel and video model includes and use it :)
Renaming of the class done and CMakeLists.txt edited accordingly !
Description
-------
Folder Contents(Images) are previewed when we browse the local directories for Pictures.
Diffs (updated)
-----
browsingbackends/localfiles/localpictures/CMakeLists.txt 2acbd59
browsingbackends/localfiles/localpictures/localpicturesmodel.h d61f2a3
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp afb852f
browsingbackends/localfiles/localthumbnailprovider.h PRE-CREATION
browsingbackends/localfiles/localthumbnailprovider.cpp PRE-CREATION
browsingbackends/localfiles/localvideos/CMakeLists.txt 3cac82c
browsingbackends/localfiles/localvideos/localvideosmodel.h 3b4b668
browsingbackends/localfiles/localvideos/localvideosmodel.cpp 6bdf386
browsingbackends/localfiles/localvideos/localvideothumbnailprovider.h d485b11
browsingbackends/localfiles/localvideos/localvideothumbnailprovider.cpp d666ac4
Diff: http://git.reviewboard.kde.org/r/111863/diff/
Testing
-------
Yes. Works fine.
Thanks,
Akshay Ratan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Mon Aug 5 16:41:09 2013
From: null at kde.org (Commit Hook)
Date: Mon, 05 Aug 2013 16:41:09 -0000
Subject: Review Request 111691: Add KPluginFactory compatible ctor to
dataengine
In-Reply-To: <20130728232447.10329.98727@vidsolbach.de>
References: <20130728232447.10329.98727@vidsolbach.de>
Message-ID: <20130805164109.31831.53021@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111691/#review37144
-----------------------------------------------------------
This review has been submitted with commit 0368d7423c98c1b9ffcacc16b28dec415d76e8d5 by Sebastian K?gler to branch master.
- Commit Hook
On July 28, 2013, 11:24 p.m., Sebastian K?gler wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111691/
> -----------------------------------------------------------
>
> (Updated July 28, 2013, 11:24 p.m.)
>
>
> Review request for Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> This adds back a ctor that is useful for creating a KPluginFactory, which has the metadata baked into the plugin. With corresponding patches to kdelibs, this allows to create a dataengine, with valid KPluginInfo from a Qt plugin (i.e. without using KService)
>
>
> Diffs
> -----
>
> src/plasma/dataengine.h d3baad3
> src/plasma/dataengine.cpp a2c68cb
> src/plasma/private/dataengine_p.h d8ade83
>
> Diff: http://git.reviewboard.kde.org/r/111691/diff/
>
>
> Testing
> -------
>
> Build and loaded plugin, verified metadata is completely inside KPluginInfo
>
>
> Thanks,
>
> Sebastian K?gler
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Mon Aug 5 16:41:12 2013
From: null at kde.org (Commit Hook)
Date: Mon, 05 Aug 2013 16:41:12 -0000
Subject: Review Request 111691: Add KPluginFactory compatible ctor to
dataengine
In-Reply-To: <20130728232447.10329.98727@vidsolbach.de>
References: <20130728232447.10329.98727@vidsolbach.de>
Message-ID: <20130805164112.31781.36569@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111691/
-----------------------------------------------------------
(Updated Aug. 5, 2013, 4:41 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Aaron J. Seigo.
Description
-------
This adds back a ctor that is useful for creating a KPluginFactory, which has the metadata baked into the plugin. With corresponding patches to kdelibs, this allows to create a dataengine, with valid KPluginInfo from a Qt plugin (i.e. without using KService)
Diffs
-----
src/plasma/dataengine.h d3baad3
src/plasma/dataengine.cpp a2c68cb
src/plasma/private/dataengine_p.h d8ade83
Diff: http://git.reviewboard.kde.org/r/111691/diff/
Testing
-------
Build and loaded plugin, verified metadata is completely inside KPluginInfo
Thanks,
Sebastian K?gler
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From afiestas at kde.org Tue Aug 6 07:02:19 2013
From: afiestas at kde.org (=?utf-8?q?=C3=80lex_Fiestas?=)
Date: Tue, 06 Aug 2013 07:02:19 -0000
Subject: Review Request 111899: Make QGuiPlatformPlugin react to
iconChanges
In-Reply-To: <20130805195129.2180.63646@vidsolbach.de>
References: <20130805195129.2180.63646@vidsolbach.de>
Message-ID: <20130806070219.19310.95079@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111899/
-----------------------------------------------------------
(Updated Aug. 6, 2013, 7:02 a.m.)
Review request for Plasma and Olivier Goffart.
Description
-------
When KDE changes the iconSize, send a StyleChange event to QToolbar and QMainWindow (this one is required for QToolBar that are children of it).
Diffs
-----
qguiplatformplugin_kde/qguiplatformplugin_kde.cpp cc74dc0
Diff: http://git.reviewboard.kde.org/r/111899/diff/
Testing
-------
Played a while with assistant, designer and quasselclient, seems to work fine.
Thanks,
?lex Fiestas
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ogoffart at kde.org Tue Aug 6 08:19:06 2013
From: ogoffart at kde.org (Olivier Goffart)
Date: Tue, 06 Aug 2013 08:19:06 -0000
Subject: Review Request 111899: Make QGuiPlatformPlugin react to
iconChanges
In-Reply-To: <20130806070219.19310.95079@vidsolbach.de>
References: <20130806070219.19310.95079@vidsolbach.de>
Message-ID: <20130806081906.32398.71237@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111899/#review37191
-----------------------------------------------------------
Ship it!
Ship It!
- Olivier Goffart
On Aug. 6, 2013, 7:02 a.m., ?lex Fiestas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111899/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 7:02 a.m.)
>
>
> Review request for Plasma and Olivier Goffart.
>
>
> Description
> -------
>
> When KDE changes the iconSize, send a StyleChange event to QToolbar and QMainWindow (this one is required for QToolBar that are children of it).
>
>
> Diffs
> -----
>
> qguiplatformplugin_kde/qguiplatformplugin_kde.cpp cc74dc0
>
> Diff: http://git.reviewboard.kde.org/r/111899/diff/
>
>
> Testing
> -------
>
> Played a while with assistant, designer and quasselclient, seems to work fine.
>
>
> Thanks,
>
> ?lex Fiestas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Tue Aug 6 09:51:11 2013
From: notmart at gmail.com (Marco Martin)
Date: Tue, 06 Aug 2013 09:51:11 -0000
Subject: Review Request 111748: Better wrap QMimeData by retaining support
for QByteArray
In-Reply-To: <20130727184112.22041.90400@vidsolbach.de>
References: <20130727184112.22041.90400@vidsolbach.de>
Message-ID: <20130806095111.12920.35074@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111748/#review37196
-----------------------------------------------------------
Ship it!
Ship It!
- Marco Martin
On July 27, 2013, 6:41 p.m., Eike Hein wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111748/
> -----------------------------------------------------------
>
> (Updated July 27, 2013, 6:41 p.m.)
>
>
> Review request for Plasma, Marco Martin and Sebastian K?gler.
>
>
> Description
> -------
>
> DeclarativeMimeData::setData() is QString type, QString data, while QMimeData::setData() is QString type, QByteArray data. The former ends up calling the latter with data.toLatin1().
>
> I have a need to set a QByteArray payload in the taskbar - X11 WIds, as understood e.g. by pagers.
>
> The proposed patch changes the DeclarativeMimeData::setData() signature to QString type, QVariant data. The implementation checks whether the QVariant is a byte array, and if so, hands it to the base class implementation. If it's not a byte array, it checks if it can be converted to a QString, and if so, hands the string version's toLatin1() to the base class. If the QVariant is neither a byte array nor convertable to a string, nothing is done.
>
> This means behavior changes as follows:
> - Byte array support is back.
> - Anything that QVariant knows how to convert to a QString is converted. This maybe subtly different from how QML "casts" to QString, but I think fits the original sentiment of changing the signature to use QString.
> - Non-bytearray/string-convertable data is silently dropped.
>
> If this patch is accepted, I would like to backport it to KDE/4.11 as well and make use of the improved implementation to restore drag-task-to-pager behavior in the new QML task manager.
>
>
> This addresses bug 322567.
> http://bugs.kde.org/show_bug.cgi?id=322567
>
>
> Diffs
> -----
>
> src/declarativeimports/draganddrop/DeclarativeMimeData.h ec725af
> src/declarativeimports/draganddrop/DeclarativeMimeData.cpp e7cab10
>
> Diff: http://git.reviewboard.kde.org/r/111748/diff/
>
>
> Testing
> -------
>
> I haven't even built the thing. It's terribad.
>
>
> Thanks,
>
> Eike Hein
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 6 10:01:52 2013
From: null at kde.org (Commit Hook)
Date: Tue, 06 Aug 2013 10:01:52 -0000
Subject: Review Request 111748: Better wrap QMimeData by retaining support
for QByteArray
In-Reply-To: <20130727184112.22041.90400@vidsolbach.de>
References: <20130727184112.22041.90400@vidsolbach.de>
Message-ID: <20130806100152.14324.3587@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111748/#review37198
-----------------------------------------------------------
This review has been submitted with commit f63357bd29944c777de2954289d9573922fe8222 by Eike Hein to branch master.
- Commit Hook
On July 27, 2013, 6:41 p.m., Eike Hein wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111748/
> -----------------------------------------------------------
>
> (Updated July 27, 2013, 6:41 p.m.)
>
>
> Review request for Plasma, Marco Martin and Sebastian K?gler.
>
>
> Description
> -------
>
> DeclarativeMimeData::setData() is QString type, QString data, while QMimeData::setData() is QString type, QByteArray data. The former ends up calling the latter with data.toLatin1().
>
> I have a need to set a QByteArray payload in the taskbar - X11 WIds, as understood e.g. by pagers.
>
> The proposed patch changes the DeclarativeMimeData::setData() signature to QString type, QVariant data. The implementation checks whether the QVariant is a byte array, and if so, hands it to the base class implementation. If it's not a byte array, it checks if it can be converted to a QString, and if so, hands the string version's toLatin1() to the base class. If the QVariant is neither a byte array nor convertable to a string, nothing is done.
>
> This means behavior changes as follows:
> - Byte array support is back.
> - Anything that QVariant knows how to convert to a QString is converted. This maybe subtly different from how QML "casts" to QString, but I think fits the original sentiment of changing the signature to use QString.
> - Non-bytearray/string-convertable data is silently dropped.
>
> If this patch is accepted, I would like to backport it to KDE/4.11 as well and make use of the improved implementation to restore drag-task-to-pager behavior in the new QML task manager.
>
>
> This addresses bug 322567.
> http://bugs.kde.org/show_bug.cgi?id=322567
>
>
> Diffs
> -----
>
> src/declarativeimports/draganddrop/DeclarativeMimeData.h ec725af
> src/declarativeimports/draganddrop/DeclarativeMimeData.cpp e7cab10
>
> Diff: http://git.reviewboard.kde.org/r/111748/diff/
>
>
> Testing
> -------
>
> I haven't even built the thing. It's terribad.
>
>
> Thanks,
>
> Eike Hein
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 6 10:01:55 2013
From: null at kde.org (Commit Hook)
Date: Tue, 06 Aug 2013 10:01:55 -0000
Subject: Review Request 111748: Better wrap QMimeData by retaining support
for QByteArray
In-Reply-To: <20130727184112.22041.90400@vidsolbach.de>
References: <20130727184112.22041.90400@vidsolbach.de>
Message-ID: <20130806100155.14607.84206@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111748/
-----------------------------------------------------------
(Updated Aug. 6, 2013, 10:01 a.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma, Marco Martin and Sebastian K?gler.
Description
-------
DeclarativeMimeData::setData() is QString type, QString data, while QMimeData::setData() is QString type, QByteArray data. The former ends up calling the latter with data.toLatin1().
I have a need to set a QByteArray payload in the taskbar - X11 WIds, as understood e.g. by pagers.
The proposed patch changes the DeclarativeMimeData::setData() signature to QString type, QVariant data. The implementation checks whether the QVariant is a byte array, and if so, hands it to the base class implementation. If it's not a byte array, it checks if it can be converted to a QString, and if so, hands the string version's toLatin1() to the base class. If the QVariant is neither a byte array nor convertable to a string, nothing is done.
This means behavior changes as follows:
- Byte array support is back.
- Anything that QVariant knows how to convert to a QString is converted. This maybe subtly different from how QML "casts" to QString, but I think fits the original sentiment of changing the signature to use QString.
- Non-bytearray/string-convertable data is silently dropped.
If this patch is accepted, I would like to backport it to KDE/4.11 as well and make use of the improved implementation to restore drag-task-to-pager behavior in the new QML task manager.
This addresses bug 322567.
http://bugs.kde.org/show_bug.cgi?id=322567
Diffs
-----
src/declarativeimports/draganddrop/DeclarativeMimeData.h ec725af
src/declarativeimports/draganddrop/DeclarativeMimeData.cpp e7cab10
Diff: http://git.reviewboard.kde.org/r/111748/diff/
Testing
-------
I haven't even built the thing. It's terribad.
Thanks,
Eike Hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ksinny at gmail.com Tue Aug 6 11:06:35 2013
From: ksinny at gmail.com (Sinny Kumari)
Date: Tue, 06 Aug 2013 11:06:35 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
In-Reply-To: <20130805163342.31306.78845@vidsolbach.de>
References: <20130805163342.31306.78845@vidsolbach.de>
Message-ID: <20130806110635.26351.94195@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/#review37204
-----------------------------------------------------------
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp
I am getting some weird behaviour in browse pictures thumbnails. Now, it doesn't show thumbnail for all pictures http://imgur.com/0hZlaqI . please try to reproduce same because I see similar behaviour at many places.
Right now, reason for same could be that we are asking thumbnail in this manner for all image files and directories. Directory preview works fine now. So, what I am suggesting is call new way of getting decoration only
if MediaCenter::IsExpandableRole role is false i.e for directory else keep previous logic (i.e files) for getting thumbnail of pictures.
- Sinny Kumari
On Aug. 5, 2013, 4:33 p.m., Akshay Ratan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111863/
> -----------------------------------------------------------
>
> (Updated Aug. 5, 2013, 4:33 p.m.)
>
>
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
>
>
> Description
> -------
>
> Folder Contents(Images) are previewed when we browse the local directories for Pictures.
>
>
> Diffs
> -----
>
> browsingbackends/localfiles/localpictures/CMakeLists.txt 2acbd59
> browsingbackends/localfiles/localpictures/localpicturesmodel.h d61f2a3
> browsingbackends/localfiles/localpictures/localpicturesmodel.cpp afb852f
> browsingbackends/localfiles/localthumbnailprovider.h PRE-CREATION
> browsingbackends/localfiles/localthumbnailprovider.cpp PRE-CREATION
> browsingbackends/localfiles/localvideos/CMakeLists.txt 3cac82c
> browsingbackends/localfiles/localvideos/localvideosmodel.h 3b4b668
> browsingbackends/localfiles/localvideos/localvideosmodel.cpp 6bdf386
> browsingbackends/localfiles/localvideos/localvideothumbnailprovider.h d485b11
> browsingbackends/localfiles/localvideos/localvideothumbnailprovider.cpp d666ac4
>
> Diff: http://git.reviewboard.kde.org/r/111863/diff/
>
>
> Testing
> -------
>
> Yes. Works fine.
>
>
> Thanks,
>
> Akshay Ratan
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Tue Aug 6 12:25:41 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Tue, 06 Aug 2013 12:25:41 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
Message-ID: <20130806122541.5222.16387@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/
-----------------------------------------------------------
Review request for KDE Frameworks and Plasma.
Description
-------
Add KConfigLoader from Plasma Framework to KConfigGui
The ConfigLoader is way to awesome to not be directly in KConfig.
Diffs
-----
tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111908/diff/
Testing
-------
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From david at davidedmundson.co.uk Tue Aug 6 14:52:16 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Tue, 06 Aug 2013 14:52:16 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806122541.5222.16387@vidsolbach.de>
References: <20130806122541.5222.16387@vidsolbach.de>
Message-ID: <20130806145216.23350.89021@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
As a non-Plasma user of this class, I fully support it being here.
tier1/kconfig/src/gui/kconfigloader.h
This looks like it should be const
I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
tier1/kconfig/src/gui/kconfigloader.h
const?
See above.
tier1/kconfig/src/gui/kconfigloader.h
const?
See above.
- David Edmundson
On Aug. 6, 2013, 12:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 12:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Tue Aug 6 15:06:16 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Tue, 06 Aug 2013 15:06:16 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806145216.23350.89021@vidsolbach.de>
References: <20130806145216.23350.89021@vidsolbach.de>
Message-ID: <20130806150616.24530.4213@vidsolbach.de>
> On Aug. 6, 2013, 4:52 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.h, line 112
> >
> >
> > This looks like it should be const
> >
> > I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
please note that I just moved the code over and changed the name from Plasma::ConfigLoader to KConfigLoader. I did not change the code and I think it would be wrong to change the code in the same step. Because of that I drop the three issues, though it would probably make sense to do these changes after this review is merged.
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
On Aug. 6, 2013, 2:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 2:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From aleixpol at kde.org Tue Aug 6 17:04:54 2013
From: aleixpol at kde.org (Aleix Pol Gonzalez)
Date: Tue, 06 Aug 2013 17:04:54 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806145216.23350.89021@vidsolbach.de>
References: <20130806145216.23350.89021@vidsolbach.de>
Message-ID: <20130806170454.11233.30914@vidsolbach.de>
> On Aug. 6, 2013, 2:52 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.h, line 112
> >
> >
> > This looks like it should be const
> >
> > I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
>
> Martin Gr??lin wrote:
> please note that I just moved the code over and changed the name from Plasma::ConfigLoader to KConfigLoader. I did not change the code and I think it would be wrong to change the code in the same step. Because of that I drop the three issues, though it would probably make sense to do these changes after this review is merged.
Make you can add a comment or warning?
- Aleix
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
On Aug. 6, 2013, 12:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 12:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From aacid at kde.org Tue Aug 6 19:14:03 2013
From: aacid at kde.org (Albert Astals Cid)
Date: Tue, 06 Aug 2013 19:14:03 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806145216.23350.89021@vidsolbach.de>
References: <20130806145216.23350.89021@vidsolbach.de>
Message-ID: <20130806191403.10113.31308@vidsolbach.de>
> On Aug. 6, 2013, 2:52 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.h, line 112
> >
> >
> > This looks like it should be const
> >
> > I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
>
> Martin Gr??lin wrote:
> please note that I just moved the code over and changed the name from Plasma::ConfigLoader to KConfigLoader. I did not change the code and I think it would be wrong to change the code in the same step. Because of that I drop the three issues, though it would probably make sense to do these changes after this review is merged.
>
> Aleix Pol Gonzalez wrote:
> Make you can add a comment or warning?
What's the point of this review request if you're not going to accept any review? IOW what are you expecting people to say in this review request?
- Albert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
On Aug. 6, 2013, 12:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 12:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Wed Aug 7 04:56:08 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Wed, 07 Aug 2013 04:56:08 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806145216.23350.89021@vidsolbach.de>
References: <20130806145216.23350.89021@vidsolbach.de>
Message-ID: <20130807045608.3438.10599@vidsolbach.de>
> On Aug. 6, 2013, 4:52 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.h, line 112
> >
> >
> > This looks like it should be const
> >
> > I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
>
> Martin Gr??lin wrote:
> please note that I just moved the code over and changed the name from Plasma::ConfigLoader to KConfigLoader. I did not change the code and I think it would be wrong to change the code in the same step. Because of that I drop the three issues, though it would probably make sense to do these changes after this review is merged.
>
> Aleix Pol Gonzalez wrote:
> Make you can add a comment or warning?
>
> Albert Astals Cid wrote:
> What's the point of this review request if you're not going to accept any review? IOW what are you expecting people to say in this review request?
> IOW what are you expecting people to say in this review request?
That it's OK to move the code to this location.
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
On Aug. 6, 2013, 2:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 2:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Wed Aug 7 06:26:53 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Wed, 07 Aug 2013 06:26:53 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806145216.23350.89021@vidsolbach.de>
References: <20130806145216.23350.89021@vidsolbach.de>
Message-ID: <20130807062653.2734.70943@vidsolbach.de>
> On Aug. 6, 2013, 4:52 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.h, line 112
> >
> >
> > This looks like it should be const
> >
> > I suspect it wasn't because KCoreConfigSeleton::findItem was not const in KDE4, but it appears to be now.
>
> Martin Gr??lin wrote:
> please note that I just moved the code over and changed the name from Plasma::ConfigLoader to KConfigLoader. I did not change the code and I think it would be wrong to change the code in the same step. Because of that I drop the three issues, though it would probably make sense to do these changes after this review is merged.
>
> Aleix Pol Gonzalez wrote:
> Make you can add a comment or warning?
>
> Albert Astals Cid wrote:
> What's the point of this review request if you're not going to accept any review? IOW what are you expecting people to say in this review request?
>
> Martin Gr??lin wrote:
> > IOW what are you expecting people to say in this review request?
> That it's OK to move the code to this location.
or other put: I'm not willing to invest time into doing the changes before I know that it will overall be accepted. Because if it doesn't get accepted the changes David requested should be done to the code in Plasma Frameworks.
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37219
-----------------------------------------------------------
On Aug. 6, 2013, 2:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 2:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From faure at kde.org Wed Aug 7 15:41:41 2013
From: faure at kde.org (David Faure)
Date: Wed, 07 Aug 2013 15:41:41 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806122541.5222.16387@vidsolbach.de>
References: <20130806122541.5222.16387@vidsolbach.de>
Message-ID: <20130807154141.31833.42677@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37284
-----------------------------------------------------------
tier1/kconfig/autotests/kconfigloadertest.cpp
I have trouble understanding the purpose of this class. How is this different from
QCOMPARE(configGroup.readEntry("DefaultBoolItem", true), true);
?
OK the one difference is that the default value comes from the XML file instead of coming from the code, but apart from that?
KConfigXT's entire purpose was to make things statically checked (compile-time), on top of the dynamic (string-based) KConfig. And now this is another layer on top, which makes things dynamic (string-based) again? I'm confused :-)
Ah, is this actually only about introspecting KConfigXT xml files, to extract the defaults from it? But what would be the purpose of that? (isn't this accessible in the KConfigXT-generated code too?)
Please expand the class documentation to make it clear for dummies like me, what is the actual purpose of the class, and in which case it should be used.
- David Faure
On Aug. 6, 2013, 12:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 12:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Thu Aug 8 04:58:14 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Thu, 08 Aug 2013 04:58:14 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130807154141.31833.42677@vidsolbach.de>
References: <20130807154141.31833.42677@vidsolbach.de>
Message-ID: <20130808045814.17630.34068@vidsolbach.de>
> On Aug. 7, 2013, 5:41 p.m., David Faure wrote:
> > tier1/kconfig/autotests/kconfigloadertest.cpp, line 56
> >
> >
> > I have trouble understanding the purpose of this class. How is this different from
> >
> > QCOMPARE(configGroup.readEntry("DefaultBoolItem", true), true);
> >
> > ?
> >
> > OK the one difference is that the default value comes from the XML file instead of coming from the code, but apart from that?
> >
> > KConfigXT's entire purpose was to make things statically checked (compile-time), on top of the dynamic (string-based) KConfig. And now this is another layer on top, which makes things dynamic (string-based) again? I'm confused :-)
> >
> > Ah, is this actually only about introspecting KConfigXT xml files, to extract the defaults from it? But what would be the purpose of that? (isn't this accessible in the KConfigXT-generated code too?)
> >
> > Please expand the class documentation to make it clear for dummies like me, what is the actual purpose of the class, and in which case it should be used.
> >
I think it's best explained to think of cases where you don't have any code in the first place. Examples are plasmoids or KWin scripts which just ship a kconfigxt file and a ui file and with the help of the KConfigLoader we are able to provide a working config interface dynamically loaded.
@Aaron: do you have a suggestion on how to improve the documentation?
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37284
-----------------------------------------------------------
On Aug. 6, 2013, 2:25 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 2:25 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Thu Aug 8 04:58:32 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Thu, 08 Aug 2013 04:58:32 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130806122541.5222.16387@vidsolbach.de>
References: <20130806122541.5222.16387@vidsolbach.de>
Message-ID: <20130808045832.17630.90464@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/
-----------------------------------------------------------
(Updated Aug. 8, 2013, 6:58 a.m.)
Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
Changes
-------
Added Aaron to get some feedback
Description
-------
Add KConfigLoader from Plasma Framework to KConfigGui
The ConfigLoader is way to awesome to not be directly in KConfig.
Diffs
-----
tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111908/diff/
Testing
-------
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From faure at kde.org Thu Aug 8 08:23:10 2013
From: faure at kde.org (David Faure)
Date: Thu, 08 Aug 2013 08:23:10 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130807154141.31833.42677@vidsolbach.de>
References: <20130807154141.31833.42677@vidsolbach.de>
Message-ID: <20130808082310.9774.47902@vidsolbach.de>
> On Aug. 7, 2013, 3:41 p.m., David Faure wrote:
> > tier1/kconfig/autotests/kconfigloadertest.cpp, line 56
> >
> >
> > I have trouble understanding the purpose of this class. How is this different from
> >
> > QCOMPARE(configGroup.readEntry("DefaultBoolItem", true), true);
> >
> > ?
> >
> > OK the one difference is that the default value comes from the XML file instead of coming from the code, but apart from that?
> >
> > KConfigXT's entire purpose was to make things statically checked (compile-time), on top of the dynamic (string-based) KConfig. And now this is another layer on top, which makes things dynamic (string-based) again? I'm confused :-)
> >
> > Ah, is this actually only about introspecting KConfigXT xml files, to extract the defaults from it? But what would be the purpose of that? (isn't this accessible in the KConfigXT-generated code too?)
> >
> > Please expand the class documentation to make it clear for dummies like me, what is the actual purpose of the class, and in which case it should be used.
> >
>
> Martin Gr??lin wrote:
> I think it's best explained to think of cases where you don't have any code in the first place. Examples are plasmoids or KWin scripts which just ship a kconfigxt file and a ui file and with the help of the KConfigLoader we are able to provide a working config interface dynamically loaded.
>
> @Aaron: do you have a suggestion on how to improve the documentation?
Ah, GUI generation, I see. A bit like KConfigDialog then, but you separated the parsing and the UI generation, and you don't need the compile-time generated KCoreConfigSkeleton).
So I was wrong, it's not on top of the KConfigXT-generated code, it's instead of that.
It just seems to me that KConfigLoader alone isn't really useful, but OK, it's a component in the overall architecture.
It definitely needs documentation about what it can be used for, and how.
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37284
-----------------------------------------------------------
On Aug. 8, 2013, 4:58 a.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2013, 4:58 a.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Thu Aug 8 12:56:15 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Thu, 08 Aug 2013 12:56:15 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130808045832.17630.90464@vidsolbach.de>
References: <20130808045832.17630.90464@vidsolbach.de>
Message-ID: <20130808125615.32577.31521@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/
-----------------------------------------------------------
(Updated Aug. 8, 2013, 2:56 p.m.)
Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
Changes
-------
Extended the documentation to show how one can use this class.
Description
-------
Add KConfigLoader from Plasma Framework to KConfigGui
The ConfigLoader is way to awesome to not be directly in KConfig.
Diffs (updated)
-----
tier1/kconfig/autotests/CMakeLists.txt c913da3
tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
tier1/kconfig/src/gui/CMakeLists.txt 0913349
tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111908/diff/
Testing
-------
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From akshay_ratan at yahoo.com Thu Aug 8 14:20:18 2013
From: akshay_ratan at yahoo.com (Akshay Ratan)
Date: Thu, 08 Aug 2013 14:20:18 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
In-Reply-To: <20130805163342.31306.78845@vidsolbach.de>
References: <20130805163342.31306.78845@vidsolbach.de>
Message-ID: <20130808142018.10951.75987@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/
-----------------------------------------------------------
(Updated Aug. 8, 2013, 2:20 p.m.)
Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
Changes
-------
Fixed the issue which Sinny pointed out about the inability of high resolution pictures to be shown as a thumbnail.
Please test it once again and let me know if there are any changes required before pushing it in the separate branch:: akshay/preview !
Description
-------
Folder Contents(Images) are previewed when we browse the local directories for Pictures.
Diffs (updated)
-----
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp 6a1239c
Diff: http://git.reviewboard.kde.org/r/111863/diff/
Testing
-------
Yes. Works fine.
Thanks,
Akshay Ratan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From akshay_ratan at yahoo.com Thu Aug 8 14:30:31 2013
From: akshay_ratan at yahoo.com (Akshay Ratan)
Date: Thu, 08 Aug 2013 14:30:31 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
In-Reply-To: <20130808142018.10951.75987@vidsolbach.de>
References: <20130808142018.10951.75987@vidsolbach.de>
Message-ID: <20130808143031.19636.86073@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/
-----------------------------------------------------------
(Updated Aug. 8, 2013, 2:30 p.m.)
Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
Changes
-------
Full diff output for the branch.
Description
-------
Folder Contents(Images) are previewed when we browse the local directories for Pictures.
Diffs (updated)
-----
browsingbackends/localfiles/localpictures/CMakeLists.txt 2acbd59
browsingbackends/localfiles/localpictures/localpicturesmodel.h d61f2a3
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp afb852f
browsingbackends/localfiles/localthumbnailprovider.h PRE-CREATION
browsingbackends/localfiles/localthumbnailprovider.cpp PRE-CREATION
browsingbackends/localfiles/localvideos/CMakeLists.txt 3cac82c
browsingbackends/localfiles/localvideos/localvideosmodel.h 3b4b668
browsingbackends/localfiles/localvideos/localvideosmodel.cpp 6bdf386
browsingbackends/localfiles/localvideos/localvideothumbnailprovider.h d485b11
browsingbackends/localfiles/localvideos/localvideothumbnailprovider.cpp d666ac4
Diff: http://git.reviewboard.kde.org/r/111863/diff/
Testing
-------
Yes. Works fine.
Thanks,
Akshay Ratan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ervin at kde.org Thu Aug 8 15:51:37 2013
From: ervin at kde.org (Kevin Ottens)
Date: Thu, 08 Aug 2013 15:51:37 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130808125615.32577.31521@vidsolbach.de>
References: <20130808125615.32577.31521@vidsolbach.de>
Message-ID: <20130808155137.2959.79662@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37360
-----------------------------------------------------------
Totally makes sense to have that in KConfigGui. Still the issues raised by David E. should be addressed.
tier1/kconfig/src/gui/kconfigloader.h
Should be kconfiggui_export.h since you move it to KConfigGui.
tier1/kconfig/src/gui/kconfigloader.h
Should be KCONFIGGUI
tier1/kconfig/src/gui/kconfigloader.h
Insert an empty line before that one.
- Kevin Ottens
On Aug. 8, 2013, 12:56 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2013, 12:56 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From shantanu at kde.org Thu Aug 8 18:18:41 2013
From: shantanu at kde.org (Shantanu Tushar)
Date: Thu, 08 Aug 2013 18:18:41 -0000
Subject: Review Request 111863: Feature: Folder Preview in Local Pictures
Browsing
In-Reply-To: <20130808143031.19636.86073@vidsolbach.de>
References: <20130808143031.19636.86073@vidsolbach.de>
Message-ID: <20130808181841.12579.52405@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111863/#review37374
-----------------------------------------------------------
Ship it!
Works fine, just fix the little issues below and push
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp
if needs braces {}
browsingbackends/localfiles/localpictures/localpicturesmodel.cpp
indentation needs to be fixed
- Shantanu Tushar
On Aug. 8, 2013, 2:30 p.m., Akshay Ratan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111863/
> -----------------------------------------------------------
>
> (Updated Aug. 8, 2013, 2:30 p.m.)
>
>
> Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
>
>
> Description
> -------
>
> Folder Contents(Images) are previewed when we browse the local directories for Pictures.
>
>
> Diffs
> -----
>
> browsingbackends/localfiles/localpictures/CMakeLists.txt 2acbd59
> browsingbackends/localfiles/localpictures/localpicturesmodel.h d61f2a3
> browsingbackends/localfiles/localpictures/localpicturesmodel.cpp afb852f
> browsingbackends/localfiles/localthumbnailprovider.h PRE-CREATION
> browsingbackends/localfiles/localthumbnailprovider.cpp PRE-CREATION
> browsingbackends/localfiles/localvideos/CMakeLists.txt 3cac82c
> browsingbackends/localfiles/localvideos/localvideosmodel.h 3b4b668
> browsingbackends/localfiles/localvideos/localvideosmodel.cpp 6bdf386
> browsingbackends/localfiles/localvideos/localvideothumbnailprovider.h d485b11
> browsingbackends/localfiles/localvideos/localvideothumbnailprovider.cpp d666ac4
>
> Diff: http://git.reviewboard.kde.org/r/111863/diff/
>
>
> Testing
> -------
>
> Yes. Works fine.
>
>
> Thanks,
>
> Akshay Ratan
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Fri Aug 9 05:20:52 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Fri, 09 Aug 2013 05:20:52 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130808125615.32577.31521@vidsolbach.de>
References: <20130808125615.32577.31521@vidsolbach.de>
Message-ID: <20130809052052.7356.99488@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/
-----------------------------------------------------------
(Updated Aug. 9, 2013, 7:20 a.m.)
Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
Changes
-------
fixed the issues outlined by David E. and Kevin
Description
-------
Add KConfigLoader from Plasma Framework to KConfigGui
The ConfigLoader is way to awesome to not be directly in KConfig.
Diffs (updated)
-----
tier1/kconfig/autotests/CMakeLists.txt c913da3
tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
tier1/kconfig/src/gui/CMakeLists.txt 0913349
tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111908/diff/
Testing
-------
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ervin at kde.org Fri Aug 9 11:59:40 2013
From: ervin at kde.org (Kevin Ottens)
Date: Fri, 09 Aug 2013 11:59:40 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809052052.7356.99488@vidsolbach.de>
References: <20130809052052.7356.99488@vidsolbach.de>
Message-ID: <20130809115940.17061.85582@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37410
-----------------------------------------------------------
tier1/kconfig/src/gui/kconfigloader.h
Didn't spot it the first time, sorry...
We generally use KConfigGroup by value if I'm not mistaken. So should be a const ref instead of a pointer.
- Kevin Ottens
On Aug. 9, 2013, 5:20 a.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 5:20 a.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Fri Aug 9 12:09:29 2013
From: mgraesslin at kde.org (Martin Graesslin)
Date: Fri, 09 Aug 2013 14:09:29 +0200
Subject: Libtaskmanager ported to Qt5/KF5
Message-ID: <6883935.jgcEjcLMyN@martin-thinkpad>
Hi all,
on special request by Eike I looked into porting libtaskmanager to Qt5/KF5.
And after a surprisingly easy port I just pushed to frameworks-scratch branch
a commit which enables building of libtaskmanager again.
As a side effect I also had to port ksysguard/processcore.
Obviously I have not tested whether the code works but we should notice soon
enough when the tasks applet gets ported ;-)
Cheers
Martin
From mgraesslin at kde.org Fri Aug 9 12:38:15 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Fri, 09 Aug 2013 12:38:15 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809052052.7356.99488@vidsolbach.de>
References: <20130809052052.7356.99488@vidsolbach.de>
Message-ID: <20130809123815.27040.19018@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/
-----------------------------------------------------------
(Updated Aug. 9, 2013, 2:38 p.m.)
Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
Changes
-------
const ref instead of pointer
Description
-------
Add KConfigLoader from Plasma Framework to KConfigGui
The ConfigLoader is way to awesome to not be directly in KConfig.
Diffs (updated)
-----
tier1/kconfig/autotests/CMakeLists.txt c913da3
tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
tier1/kconfig/src/gui/CMakeLists.txt 0913349
tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/111908/diff/
Testing
-------
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Fri Aug 9 13:13:14 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 09 Aug 2013 15:13:14 +0200
Subject: Branch review request
In-Reply-To: <1967303.8KXXA0zMFk@drako>
References: <1967303.8KXXA0zMFk@drako>
Message-ID: <2126645.EOmZGENg7g@miro.vizzzion.net>
On Saturday, August 03, 2013 19:34:32 Ivan ?uki? wrote:
> I've realized that it will be painful if I wait until everything is
> finished to merge into master, and that there is not much reason for it to
> be so since the master is in a playground mode anyway.
>
> One of the reasons for the request is that I'd like to do a few things in
> parallel to the main work - cleaning up the whole plasma code, use allowed
> levels of C++11 (and the 'spiffy' d-pointers as Aaron called them)...
Some comments:
src/declarativeimports/hardware/:
- this is solid material, Alex wants this in Solid, in fact he wants to make
Solid "awesome to use from QML". Let's not introduce yet another wrapper that
we have to maintain, but get it into Solid.
(You know this one, as you're already merging stuff to Solid, good!)
Changes to src/plasma:
- needs proper apidocs, arguments, @since, etc. It would also be nice to tell
about race conditions, can someone, at any given moment just set another
package? What is the expected behavior then?
src/platformstatus:
- single line conditional also get curly braces
src/utils:
- A custom d-pointer implementation? This seems like overkill and makes the
code a lot less readable. Just remove this and use a QScopedPointer?
> *(2) The other thing I wanted to ask is can I rename org.kde.desktop to
> org.kde.plasma.desktop?
How about org.kde.desktopshell? That would make it clearer what it is compared
to containments.
This is not a full review, but some points that need addressing anyway.
Cheers,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From ivan.cukic at kde.org Fri Aug 9 13:30:08 2013
From: ivan.cukic at kde.org (=?UTF-8?B?SXZhbiDEjHVracSH?=)
Date: Fri, 9 Aug 2013 15:30:08 +0200
Subject: Branch review request
In-Reply-To: <2126645.EOmZGENg7g@miro.vizzzion.net>
References: <1967303.8KXXA0zMFk@drako> <2126645.EOmZGENg7g@miro.vizzzion.net>
Message-ID:
> - A custom d-pointer implementation? This seems like overkill and
To quote Aaron:
"would love to see all the dptrs move to your spiffy d_ptr smart
pointer as well."
after this blog post of mine:
http://ivan.fomentgroup.org/blog/2013/06/22/d-ptr-the-modern-way/
>> *(2) The other thing I wanted to ask is can I rename org.kde.desktop to
>> org.kde.plasma.desktop?
>
> How about org.kde.desktopshell? That would make it clearer what
Ok, though I wanted it to say plasma to advertise the project of kde
it belongs to.
From sebas at kde.org Fri Aug 9 13:33:57 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 09 Aug 2013 15:33:57 +0200
Subject: Branch review request
In-Reply-To:
References: <1967303.8KXXA0zMFk@drako> <2126645.EOmZGENg7g@miro.vizzzion.net>
Message-ID: <7040054.vGMVU3JldC@miro.vizzzion.net>
On Friday, August 09, 2013 15:30:08 Ivan ?uki? wrote:
> > - A custom d-pointer implementation? This seems like overkill and
>
> To quote Aaron:
> "would love to see all the dptrs move to your spiffy d_ptr smart
> pointer as well."
> after this blog post of mine:
> http://ivan.fomentgroup.org/blog/2013/06/22/d-ptr-the-modern-way/
It doesn't belong into plasma-framework, and you didn't answer my "why not
QScopedPointer" question. :)
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From sebas at kde.org Fri Aug 9 13:34:28 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 09 Aug 2013 15:34:28 +0200
Subject: Libtaskmanager ported to Qt5/KF5
In-Reply-To: <6883935.jgcEjcLMyN@martin-thinkpad>
References: <6883935.jgcEjcLMyN@martin-thinkpad>
Message-ID: <1467844.N4HGVqxSQ5@miro.vizzzion.net>
On Friday, August 09, 2013 14:09:29 Martin Graesslin wrote:
> on special request by Eike I looked into porting libtaskmanager to Qt5/KF5.
> And after a surprisingly easy port I just pushed to frameworks-scratch
> branch a commit which enables building of libtaskmanager again.
>
> As a side effect I also had to port ksysguard/processcore.
>
> Obviously I have not tested whether the code works but we should notice
> soon enough when the tasks applet gets ported
Rocking! Thanks,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From ivan.cukic at kde.org Fri Aug 9 13:49:33 2013
From: ivan.cukic at kde.org (=?UTF-8?B?SXZhbiDEjHVracSH?=)
Date: Fri, 9 Aug 2013 15:49:33 +0200
Subject: Branch review request
In-Reply-To: <7040054.vGMVU3JldC@miro.vizzzion.net>
References: <1967303.8KXXA0zMFk@drako> <2126645.EOmZGENg7g@miro.vizzzion.net>
<7040054.vGMVU3JldC@miro.vizzzion.net>
Message-ID:
> It doesn't belong into plasma-framework,
The original mail *was* about plasma-framework - the thread was
started when Kevin removed the add_definitions("-std=c++0x") from the
repo's top CMakeLists.txt
http://mail.kde.org/pipermail/plasma-devel/2013-June/025747.html
> and you didn't answer my "why not
> QScopedPointer" question. :)
It is not really about QSP or unique_ptr - they /can/ be used directly
for this. They don't have the same features though (btw, the
implementation does use unique_ptr in the back)
Features
- if SomeClass::Private has a no-argument constructor, you don't need
to initialize it at all
SomeClass::SomeClass()
// This:
// : d(new Private())
// becomes this:
{
}
- if it has arguments, for example Private(int, int)
SomeClass::SomeClass()
// This:
// : d(new Private(2,3))
// becomes:
: d(2,3)
{
}
- it is safe. You can't not accidentally screw up the pointer it
points to. You can't accidentally access the raw pointer. You can not
delete it etc. (the only way to access the raw pointer is to use
operator->() - and you can not do that accidentally :) )
- it ensures that it points to something, you can not leave it empty
by accident.
So, in the same way QSP and u_p are improvements over a raw pointer,
this is an improvement over them - for *this* use-case - it is
tailored for this use-case.
--
Cheerio,
Ivan
--
While you were hanging yourself on someone else's words
Dying to believe in what you heard
I was staring straight into the shining sun
From ivan.cukic at kde.org Fri Aug 9 13:51:32 2013
From: ivan.cukic at kde.org (=?UTF-8?B?SXZhbiDEjHVracSH?=)
Date: Fri, 9 Aug 2013 15:51:32 +0200
Subject: Branch review request
In-Reply-To:
References: <1967303.8KXXA0zMFk@drako> <2126645.EOmZGENg7g@miro.vizzzion.net>
<7040054.vGMVU3JldC@miro.vizzzion.net>
Message-ID:
p.s. I'm a bit of a safe-coding buff lately - if something removes
potential "stupid bugs" and doesn't introduce a performance overhead -
I'm all into it (like for vs foreach and similar).
From sebas at kde.org Fri Aug 9 13:53:20 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 09 Aug 2013 15:53:20 +0200
Subject: Branch review request
In-Reply-To:
References: <1967303.8KXXA0zMFk@drako> <7040054.vGMVU3JldC@miro.vizzzion.net>
Message-ID: <1446809.3aKeaM8MNn@miro.vizzzion.net>
On Friday, August 09, 2013 15:49:33 Ivan ?uki? wrote:
> > It doesn't belong into plasma-framework,
>
> The original mail *was* about plasma-framework - the thread was
> started when Kevin removed the add_definitions("-std=c++0x") from the
> repo's top CMakeLists.txt
> http://mail.kde.org/pipermail/plasma-devel/2013-June/025747.html
Well, my point is that this is too low level (or generally applicable if you
wish :P) for plasma-framework.
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From bruce.oy at gmail.com Fri Aug 9 14:22:34 2013
From: bruce.oy at gmail.com (Bruce Ouyang)
Date: Fri, 09 Aug 2013 22:22:34 +0800
Subject: how to draw a transparent / semi-transparent area in plasma-qml
=?iso-8859-1?Q?=3F?=
Message-ID:
i have tried rectrangle with opacity:0.5 and color:"transparent", but this doesn't work.
looking forward for your reply!
thanks advance!
From heena393 at gmail.com Fri Aug 9 14:29:44 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Fri, 9 Aug 2013 14:29:44 +0000
Subject: how to draw a transparent / semi-transparent area in plasma-qml ?
In-Reply-To:
References:
Message-ID:
Ahem , no use of transparent rectangles ,we don't do that way.Anyways you
can play with opacity by behaviour on opacity also .
Regards
On Fri, Aug 9, 2013 at 2:22 PM, Bruce Ouyang wrote:
> i have tried rectrangle with opacity:0.5 and color:"transparent", but this
> doesn't work.
> looking forward for your reply!
> thanks advance!
> ______________________________**_________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/**listinfo/plasma-devel
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From dantti12 at gmail.com Fri Aug 9 14:31:15 2013
From: dantti12 at gmail.com (Daniel Nicoletti)
Date: Fri, 9 Aug 2013 11:31:15 -0300
Subject: how to draw a transparent / semi-transparent area in plasma-qml ?
In-Reply-To:
References:
Message-ID:
a rectangle with color: "transparent" will be fully transparent
regardless of it's opacity,
though the opacity will be applied to all of it's children.
if you want something semi-transparent you can try color: "white" and
opacity: 0.5
2013/8/9 Bruce Ouyang :
> i have tried rectrangle with opacity:0.5 and color:"transparent", but this
> doesn't work.
> looking forward for your reply!
> thanks advance!
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
--
Daniel Nicoletti
KDE Developer - http://dantti.wordpress.com
From sebas at kde.org Fri Aug 9 14:43:45 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Fri, 09 Aug 2013 16:43:45 +0200
Subject: how to draw a transparent / semi-transparent area in plasma-qml ?
In-Reply-To:
References:
Message-ID: <2553544.EAUeI0L01O@miro.vizzzion.net>
On Friday, August 09, 2013 22:22:34 Bruce Ouyang wrote:
> i have tried rectrangle with opacity:0.5 and color:"transparent", but this
> doesn't work. looking forward for your reply!
If you could explain your usecase a bit more detailed, that'd help.
Otherwise, from the little info you give, PlasmaComponents.Highlight might be
what you're looking for.
Cheers,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From david at davidedmundson.co.uk Fri Aug 9 14:55:21 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Fri, 09 Aug 2013 14:55:21 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809123815.27040.19018@vidsolbach.de>
References: <20130809123815.27040.19018@vidsolbach.de>
Message-ID: <20130809145521.13568.69836@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37413
-----------------------------------------------------------
tier1/kconfig/src/gui/kconfigloader.cpp
This would be best with use of QLatin1String() thoughout this file.
- David Edmundson
On Aug. 9, 2013, 12:38 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 12:38 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From kde at privat.broulik.de Fri Aug 9 15:04:29 2013
From: kde at privat.broulik.de (Kai Uwe Broulik)
Date: Fri, 09 Aug 2013 15:04:29 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809145521.13568.69836@vidsolbach.de>
References: <20130809145521.13568.69836@vidsolbach.de>
Message-ID: <20130809150429.23997.73144@vidsolbach.de>
> On Aug. 9, 2013, 2:55 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.cpp, line 65
> >
> >
> > This would be best with use of QLatin1String() thoughout this file.
> >
> >
QStringLiteral?
- Kai Uwe
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37413
-----------------------------------------------------------
On Aug. 9, 2013, 12:38 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 12:38 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From david at davidedmundson.co.uk Fri Aug 9 15:07:38 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Fri, 09 Aug 2013 15:07:38 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809145521.13568.69836@vidsolbach.de>
References: <20130809145521.13568.69836@vidsolbach.de>
Message-ID: <20130809150738.27181.33488@vidsolbach.de>
> On Aug. 9, 2013, 2:55 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.cpp, line 65
> >
> >
> > This would be best with use of QLatin1String() thoughout this file.
> >
> >
>
> Kai Uwe Broulik wrote:
> QStringLiteral?
No, here we are only comparing a QString with a char* so we want to avoid creating a QString object and copying all the text just to do == on it.
QStringLiteral is for when you know you want to make a QString object at some point in the future.
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37413
-----------------------------------------------------------
On Aug. 9, 2013, 12:38 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 12:38 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From david at davidedmundson.co.uk Fri Aug 9 15:08:33 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Fri, 09 Aug 2013 15:08:33 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809145521.13568.69836@vidsolbach.de>
References: <20130809145521.13568.69836@vidsolbach.de>
Message-ID: <20130809150833.31694.22290@vidsolbach.de>
> On Aug. 9, 2013, 2:55 p.m., David Edmundson wrote:
> > tier1/kconfig/src/gui/kconfigloader.cpp, line 65
> >
> >
> > This would be best with use of QLatin1String() thoughout this file.
> >
> >
>
> Kai Uwe Broulik wrote:
> QStringLiteral?
>
> David Edmundson wrote:
> No, here we are only comparing a QString with a char* so we want to avoid creating a QString object and copying all the text just to do == on it.
>
> QStringLiteral is for when you know you want to make a QString object at some point in the future.
Edit: I'm happy with this being done after it's merged. No need to hold things up.
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37413
-----------------------------------------------------------
On Aug. 9, 2013, 12:38 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 12:38 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From faure at kde.org Fri Aug 9 19:27:23 2013
From: faure at kde.org (David Faure)
Date: Fri, 09 Aug 2013 19:27:23 -0000
Subject: Review Request 111908: Add KConfigLoader from Plasma Framework to
KConfigGui
In-Reply-To: <20130809123815.27040.19018@vidsolbach.de>
References: <20130809123815.27040.19018@vidsolbach.de>
Message-ID: <20130809192723.16448.12501@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111908/#review37445
-----------------------------------------------------------
tier1/kconfig/src/gui/kconfigloader.h
I wonder if this should really derive from KConfigSkeleton, rather than encapsulate it.
For instance it means you can't get a "core only" version of it (see KCoreConfigSkeleton).
It doesn't reimplement any virtual methods, so this inheritance seems unnecessary. On the other hand I'm not sure how it would work by composition anyway; maybe the caller would have to create it themselves (which would be a bit cumbersome?).
The use of QColor in the code makes core/gui separation difficult anyway, it would need to be provided by a runtime hook.
Just an idea.
Don't know how much core/gui separation is useful for this class.
- David Faure
On Aug. 9, 2013, 12:38 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111908/
> -----------------------------------------------------------
>
> (Updated Aug. 9, 2013, 12:38 p.m.)
>
>
> Review request for KDE Frameworks, Plasma and Aaron J. Seigo.
>
>
> Description
> -------
>
> Add KConfigLoader from Plasma Framework to KConfigGui
>
> The ConfigLoader is way to awesome to not be directly in KConfig.
>
>
> Diffs
> -----
>
> tier1/kconfig/autotests/CMakeLists.txt c913da3
> tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.cpp PRE-CREATION
> tier1/kconfig/autotests/kconfigloadertest.xml PRE-CREATION
> tier1/kconfig/src/gui/CMakeLists.txt 0913349
> tier1/kconfig/src/gui/kconfigloader.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader.cpp PRE-CREATION
> tier1/kconfig/src/gui/kconfigloader_p.h PRE-CREATION
> tier1/kconfig/src/gui/kconfigloaderhandler_p.h PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/111908/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Fri Aug 9 23:28:26 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Sat, 10 Aug 2013 01:28:26 +0200
Subject: Libtaskmanager ported to Qt5/KF5
In-Reply-To: <6883935.jgcEjcLMyN@martin-thinkpad>
References: <6883935.jgcEjcLMyN@martin-thinkpad>
Message-ID: <1543323.eP51yocMUY@monet>
On Friday, August 09, 2013 14:09:29 Martin Graesslin wrote:
> on special request by Eike I looked into porting libtaskmanager to Qt5/KF5.
> And after a surprisingly easy port I just pushed to frameworks-scratch
> branch a commit which enables building of libtaskmanager again.
>
> As a side effect I also had to port ksysguard/processcore.
>
> Obviously I have not tested whether the code works but we should notice
> soon enough when the tasks applet gets ported ;-)
.. or the tasks dataengine. :) I've just ported it, it builds, loads, reacts
to "virtualDesktops", but not to "tasks". I haven't looked into it in more
detail, but we can debug this from here, as plasmaengineexplorer is also
ported (in plasmate[frameworks-scratch]).
Progress!
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From hein at kde.org Sat Aug 10 00:13:56 2013
From: hein at kde.org (Eike Hein)
Date: Sat, 10 Aug 2013 02:13:56 +0200
Subject: Libtaskmanager ported to Qt5/KF5
In-Reply-To: <6883935.jgcEjcLMyN@martin-thinkpad>
References: <6883935.jgcEjcLMyN@martin-thinkpad>
Message-ID: <1513590.TuRnP4SBVJ@ehm1.ehn>
On Friday 09 August 2013 14:09:29 you wrote:
> Hi all,
>
> on special request by Eike I looked into porting libtaskmanager to Qt5/KF5.
> And after a surprisingly easy port I just pushed to frameworks-scratch
> branch a commit which enables building of libtaskmanager again.
Thanks :)
Nice to hear it was that easy - I had hoped as much, there's
not really that much Xlib usage in there, but since you were
doing xcb porting on kwin already I figured it made sense to
tap into that knowledge :).
Cheers,
Eike
From null at kde.org Sat Aug 10 13:10:22 2013
From: null at kde.org (Commit Hook)
Date: Sat, 10 Aug 2013 13:10:22 -0000
Subject: Review Request 110467: Plasma Wallpaper BackgroundListModel:
Avoid using KFileMetaInfo
In-Reply-To: <20130517180300.28244.6876@vidsolbach.de>
References: <20130517180300.28244.6876@vidsolbach.de>
Message-ID: <20130810131022.11157.85938@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110467/#review37461
-----------------------------------------------------------
This review has been submitted with commit 13ebf3c4a2f47827d54f61a19b30cfe49b127ad4 by Hrvoje Senjan on behalf of Vishesh Handa to branch frameworks-scratch.
- Commit Hook
On May 17, 2013, 6:02 p.m., Vishesh Handa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110467/
> -----------------------------------------------------------
>
> (Updated May 17, 2013, 6:02 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Plasma Wallpaper BackgroundListModel: Avoid using KFileMetaInfo
>
> 1. The Xesam width and height properties are never returned by strigi
> and haven't been returned for a very very long time. It currently
> returns the nfo:width and nfo:height property instead.
>
> 2. KFileMetadataWidget is blocking and blocks while it is calculating
> the height and width. It is better to open the file and calculate it in
> another thread as done by the ImageSizeFinder
>
> 3. QImage in more reliable and actively maintained in contrast to
> strigi.
>
>
> Diffs
> -----
>
> plasma/generic/wallpapers/image/backgroundlistmodel.cpp c4093a7
>
> Diff: http://git.reviewboard.kde.org/r/110467/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Vishesh Handa
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Sun Aug 11 10:39:56 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Sun, 11 Aug 2013 12:39:56 +0200
Subject: Review Request 111992: Activity bar in QML.
In-Reply-To: <20130810171855.9101.84068@vidsolbach.de>
References: <20130810170400.7039.54207@vidsolbach.de>
<20130810171855.9101.84068@vidsolbach.de>
Message-ID: <1842169.LUNnsqFlyY@miro.vizzzion.net>
Hi Bhushan,
[The right list for this discussion is plasma-devel at kde.org, I'm CC:ing this
list, let's move the discussion there.]
On Saturday, August 10, 2013 17:18:55 Bhushan Shah wrote:
> Yes! I am showing all activities because if someone wants to switch to
> activity which is not running then he/she have to do it from activity
> manager. That don't make sense to me..
We usually do not just remove features, because we don't understand them. In
the case of the activity bar, I can easily enlighten you, as I'm using it, and
I'm relying on exactly that feature of not showing stopped activities. Likely,
there are many others in the same situation.
I have about 20 Activities configured on this system, I usually use 2 to 3.
The rest are all rather specific that I might use for a couple of days, and
then will not use for weeks or months. Having to scan 20 items every time I
switch between activities would make this very cumbersome.
Our usual and proven approach to porting widgets to QML is to port them
feature-by-feature, wether we like and understand a feature, or we don't --
it's not about our understanding or preferences, it's about what is useful to
the user.
Adding this "only show running Activities" back in is therefore essential for
merging this widget.
There is another catch, however. kde-workspace 4.x is frozen, we are moving
development to the new, Plasma 2 version. Now porting a widget to QML is a
good intermediate step, from there it will be quite easy to port it to
QtQuick2 and Plasma 2, but I thought I'd tell nevertheless.
Cheers,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From bhush94 at gmail.com Sun Aug 11 10:59:42 2013
From: bhush94 at gmail.com (Bhushan Shah)
Date: Sun, 11 Aug 2013 16:29:42 +0530
Subject: Fwd: Review Request 111992: Activity bar in QML.
In-Reply-To:
References: <20130810170400.7039.54207@vidsolbach.de>
<20130810171855.9101.84068@vidsolbach.de>
<1842169.LUNnsqFlyY@miro.vizzzion.net>
Message-ID:
Hello,
On Sun, Aug 11, 2013 at 4:09 PM, Sebastian K?gler wrote:
> I have about 20 Activities configured on this system, I usually use 2 to 3.
> The rest are all rather specific that I might use for a couple of days, and
> then will not use for weeks or months. Having to scan 20 items every time I
> switch between activities would make this very cumbersome.
>
If that's user case then I am happy to use SortFilterModel instead of the
DataModel, Later we can add configuration option "Show all activities".
Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Sun Aug 11 15:02:31 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Sun, 11 Aug 2013 15:02:31 +0000
Subject: Libtaskmanager ported to Qt5/KF5
In-Reply-To: <1513590.TuRnP4SBVJ@ehm1.ehn>
References: <6883935.jgcEjcLMyN@martin-thinkpad> <1513590.TuRnP4SBVJ@ehm1.ehn>
Message-ID:
Hi,
Does that mean now window-list can be ported to plasma2 (it requires tasks
data engine)..?
Regards
On Sat, Aug 10, 2013 at 12:13 AM, Eike Hein wrote:
> On Friday 09 August 2013 14:09:29 you wrote:
> > Hi all,
> >
> > on special request by Eike I looked into porting libtaskmanager to
> Qt5/KF5.
> > And after a surprisingly easy port I just pushed to frameworks-scratch
> > branch a commit which enables building of libtaskmanager again.
>
> Thanks :)
>
> Nice to hear it was that easy - I had hoped as much, there's
> not really that much Xlib usage in there, but since you were
> doing xcb porting on kwin already I figured it made sense to
> tap into that knowledge :).
>
>
> Cheers,
> Eike
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From esken at kde.org Sun Aug 11 21:58:50 2013
From: esken at kde.org (Christian Esken)
Date: Sun, 11 Aug 2013 21:58:50 -0000
Subject: Review Request 6928: KMix Declarative Applet - First attempt
In-Reply-To: <20130726223042.16841.14778@vidsolbach.de>
References: <20130726223042.16841.14778@vidsolbach.de>
Message-ID: <20130811215850.28467.4923@vidsolbach.de>
> On July 26, 2013, 10:30 p.m., Christian Esken wrote:
> > I have not followed this request, but it looks like I have not receieved any followups on my last comment. I would like to know what to do with this review request. Diego, Igor? Diego, are you still following up on this. Care to contact Igor?
>
> Diego Casella wrote:
> Hi Christian, after hearing that someone else was working on it, I left the development of that plasmoid; I didnt' want to step over Igor's work.
> Anyway, by looking at the current kmix status, it's still missing the conversion to a full kded service: otherwise there will be two entries in the tray: the "old" kmix, and its plasmoid counterpart. We also need to extend its dbus interface and add a method to pop-up the full kmix gui: in that way, the plasmoid can show the user interface, allowing the end-user to configure kmix properly.
> As soon as those changes will be peformed (if you agree with it of course, and if you want to implement the new kmix as a qml plasmoid), I'd be happy to continue what I did so far :)
Commenting on:
"if you want to implement the new kmix as a qml plasmoid"
Well, I have no knowledge in QML and no desire to learn it. If you still want to do so - focussing on the tray popup/plasmoid - then it should be much more functional than your prototype - especially: supporting multiple controls, adding Media player control like in todays KMix.
The KMix Mainwindow will surely always stay a normal application, everything else is a complete new application.
- Christian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6928/#review11062
-----------------------------------------------------------
On March 28, 2012, 6:49 p.m., Diego Casella wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6928/
> -----------------------------------------------------------
>
> (Updated March 28, 2012, 6:49 p.m.)
>
>
> Review request for Plasma, Aaron Seigo, Christian Esken, and Marco Martin.
>
>
> Description
> -------
>
> First attempt of making a declarative kmix applet for plasma.
> What the apple does right now:
> * modifies the volume level and the mute/unmute status of the master channel;
> * reacts to changes of the volume level/status (i.e. made with multimedia keys);
> * disables the slider if the channel gets muted, and enables it back as soon as the channel gets unmuted;
> * collapses gracefully in a popup icon when placed inside the panel.
>
>
> Diffs
> -----
>
> trunk/KDE/kdemultimedia/kmix/plasma/CMakeLists.txt 1287513
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/VerticalControl.qml PRE-CREATION
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/kmixapplet.qml PRE-CREATION
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/metadata.desktop PRE-CREATION
>
> Diff: http://svn.reviewboard.kde.org/r/6928/diff/
>
>
> Testing
> -------
>
> Tested against r1287510. For basic audio management it works great imho.
>
> However, there is a lot of room for improvements, but this is gonna need some extra work outside the kmix applet scope:
> * first of all, the applet need kmix executable to run in order to perform the dbus calls. You can of course disable KMix tray icon feature but, at every login, KMix mainwindow will be shown and the user must closeby hand. This is a kind of ugly behavior that should be avoided;
> * it will be great to great to add an action to allow the user to select the master channel (by reusing KMix "Select Master Channel" widget), but this will require tweaking KMix dbus interface;
> * as you noticed in the screenshots, the applet in the panel and in the desktop have different size even if it __is__ actually the same: something is going wrong when plasma shows the PopupApplet. This behavior was even worse when I started implementing a "flip" action to change the layout from horizontal to vertical and vice-versa, and for this reason I gave up and simply stick with the vertical layout.
>
> Could this applet be shipped in the current status, or should we wait for all the aforementioned improvements?
> Comments/ideas/suggestions?
>
> Cheers :)
>
>
> Screenshots
> -----------
>
> Applet look in panel and desktop
> http://svn.reviewboard.kde.org/r/6928/s/627/
> Applet look in panel and desktop - audio muted
> http://svn.reviewboard.kde.org/r/6928/s/628/
>
>
> Thanks,
>
> Diego Casella
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Sun Aug 11 22:11:35 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Mon, 12 Aug 2013 00:11:35 +0200
Subject: Libtaskmanager ported to Qt5/KF5
In-Reply-To:
References: <6883935.jgcEjcLMyN@martin-thinkpad> <1513590.TuRnP4SBVJ@ehm1.ehn>
Message-ID: <7142615.YVMgdT9dZB@monet>
On Sunday, August 11, 2013 15:02:31 Heena Mahour wrote:
> Does that mean now window-list can be ported to plasma2 (it requires tasks
> data engine)..?
Yes, at least as far as the tasks engine in Plasma 2 works, I think it
doesn't. (That'd then be the first thing to fix.)
For this you need kdelibs[frameworks], plasma-framework[master] and kde-
workspace[frameworks-scratch], optionally (two more wallpaper plugins)
kdeplasma-addons[frameworks-scratch] and plasma[frameworks-scratch] for the
engineexplorer.
Cheers,
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From sebas at kde.org Sun Aug 11 22:11:52 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Mon, 12 Aug 2013 00:11:52 +0200
Subject: Fwd: Review Request 111992: Activity bar in QML.
In-Reply-To:
References: <20130810170400.7039.54207@vidsolbach.de>
Message-ID: <1952866.9xbaRdiV0B@monet>
On Sunday, August 11, 2013 16:29:42 Bhushan Shah wrote:
> On Sun, Aug 11, 2013 at 4:09 PM, Sebastian K?gler wrote:
> I have about 20 Activities configured on this system, I usually use 2 to 3.
> The rest are all rather specific that I might use for a couple of days, and
> then will not use for weeks or months. Having to scan 20 items every time I
> switch between activities would make this very cumbersome.
>
> If that's user case then I am happy to use SortFilterModel instead of the
> DataModel, Later we can add configuration option "Show all activities".
Okay, cool. :)
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From sebas at kde.org Sun Aug 11 22:12:46 2013
From: sebas at kde.org (Sebastian =?ISO-8859-1?Q?K=FCgler?=)
Date: Mon, 12 Aug 2013 00:12:46 +0200
Subject: mondaily Plasma hangout at 12:00 Europe/Amsterdam
Message-ID: <2737004.p0jRpmrzXG@monet>
see subject, just a reminder. Pop up in #plasma on IRC if you want to join.
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
From polentino911 at gmail.com Mon Aug 12 07:33:47 2013
From: polentino911 at gmail.com (Diego Casella)
Date: Mon, 12 Aug 2013 07:33:47 -0000
Subject: Review Request 6928: KMix Declarative Applet - First attempt
In-Reply-To: <20130726223042.16841.14778@vidsolbach.de>
References: <20130726223042.16841.14778@vidsolbach.de>
Message-ID: <20130812073347.3612.99715@vidsolbach.de>
> On July 26, 2013, 10:30 p.m., Christian Esken wrote:
> > I have not followed this request, but it looks like I have not receieved any followups on my last comment. I would like to know what to do with this review request. Diego, Igor? Diego, are you still following up on this. Care to contact Igor?
>
> Diego Casella wrote:
> Hi Christian, after hearing that someone else was working on it, I left the development of that plasmoid; I didnt' want to step over Igor's work.
> Anyway, by looking at the current kmix status, it's still missing the conversion to a full kded service: otherwise there will be two entries in the tray: the "old" kmix, and its plasmoid counterpart. We also need to extend its dbus interface and add a method to pop-up the full kmix gui: in that way, the plasmoid can show the user interface, allowing the end-user to configure kmix properly.
> As soon as those changes will be peformed (if you agree with it of course, and if you want to implement the new kmix as a qml plasmoid), I'd be happy to continue what I did so far :)
>
> Christian Esken wrote:
> Commenting on:
> "if you want to implement the new kmix as a qml plasmoid"
>
> Well, I have no knowledge in QML and no desire to learn it. If you still want to do so - focussing on the tray popup/plasmoid - then it should be much more functional than your prototype - especially: supporting multiple controls, adding Media player control like in todays KMix.
> The KMix Mainwindow will surely always stay a normal application, everything else is a complete new application.
> Commenting on:
> "if you want to implement the new kmix as a qml plasmoid"
>
> Well, I have no knowledge in QML and no desire to learn it.
Sorry, I meant to say "if you want me to implement the new kmix as a qml plasmoid", I though I was sufficiently clear since then I continued with "I'd be happy to continue[..]" :)
About multiple controls and Media player controls I agree: they are rather easy to implement, so I'll go for it.
> The KMix Mainwindow will surely always stay a normal application, everything else is a complete new application.
Okay, I guess then it will be modified to not show the tray icon by default - otherwise there will be its tray icon and the plasma applet, kinda confusing for the user - right? There's this annoyance then: when KMix is set to not show the tray icon, at every login it will display its GUI. That should not happen, unless the user explicitly executes KMix or my applet calls it.
What do you think?
PS: since kmix moved to git and this review request is old, once we agreed on what should be done I'll start a brand new one and and link to this one as reference :)
- Diego
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6928/#review11062
-----------------------------------------------------------
On March 28, 2012, 6:49 p.m., Diego Casella wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6928/
> -----------------------------------------------------------
>
> (Updated March 28, 2012, 6:49 p.m.)
>
>
> Review request for Plasma, Aaron Seigo, Christian Esken, and Marco Martin.
>
>
> Description
> -------
>
> First attempt of making a declarative kmix applet for plasma.
> What the apple does right now:
> * modifies the volume level and the mute/unmute status of the master channel;
> * reacts to changes of the volume level/status (i.e. made with multimedia keys);
> * disables the slider if the channel gets muted, and enables it back as soon as the channel gets unmuted;
> * collapses gracefully in a popup icon when placed inside the panel.
>
>
> Diffs
> -----
>
> trunk/KDE/kdemultimedia/kmix/plasma/CMakeLists.txt 1287513
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/VerticalControl.qml PRE-CREATION
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/kmixapplet.qml PRE-CREATION
> trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/metadata.desktop PRE-CREATION
>
> Diff: http://svn.reviewboard.kde.org/r/6928/diff/
>
>
> Testing
> -------
>
> Tested against r1287510. For basic audio management it works great imho.
>
> However, there is a lot of room for improvements, but this is gonna need some extra work outside the kmix applet scope:
> * first of all, the applet need kmix executable to run in order to perform the dbus calls. You can of course disable KMix tray icon feature but, at every login, KMix mainwindow will be shown and the user must closeby hand. This is a kind of ugly behavior that should be avoided;
> * it will be great to great to add an action to allow the user to select the master channel (by reusing KMix "Select Master Channel" widget), but this will require tweaking KMix dbus interface;
> * as you noticed in the screenshots, the applet in the panel and in the desktop have different size even if it __is__ actually the same: something is going wrong when plasma shows the PopupApplet. This behavior was even worse when I started implementing a "flip" action to change the layout from horizontal to vertical and vice-versa, and for this reason I gave up and simply stick with the vertical layout.
>
> Could this applet be shipped in the current status, or should we wait for all the aforementioned improvements?
> Comments/ideas/suggestions?
>
> Cheers :)
>
>
> Screenshots
> -----------
>
> Applet look in panel and desktop
> http://svn.reviewboard.kde.org/r/6928/s/627/
> Applet look in panel and desktop - audio muted
> http://svn.reviewboard.kde.org/r/6928/s/628/
>
>
> Thanks,
>
> Diego Casella
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Mon Aug 12 17:34:45 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Mon, 12 Aug 2013 17:34:45 -0000
Subject: Review Request 112033: Drop windowId property from PlasmaCore.Dialog
Message-ID: <20130812173445.10948.86699@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/
-----------------------------------------------------------
Review request for Plasma and Marco Martin.
Description
-------
Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
Diffs
-----
src/declarativeimports/core/dialog.h 7392b71
src/declarativeimports/core/dialog.cpp eaf3dce
Diff: http://git.reviewboard.kde.org/r/112033/diff/
Testing
-------
Yes, KWin's ThumnailItem is still working
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ivan.cukic at kde.org Mon Aug 12 17:50:31 2013
From: ivan.cukic at kde.org (=?utf-8?b?SXZhbiDEjHVracSH?=)
Date: Mon, 12 Aug 2013 17:50:31 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812173445.10948.86699@vidsolbach.de>
References: <20130812173445.10948.86699@vidsolbach.de>
Message-ID: <20130812175031.11937.37398@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37602
-----------------------------------------------------------
Isn't it also used for reporting wid to share-like-connect (activity manager) in some applications?
- Ivan ?uki?
On Aug. 12, 2013, 5:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 5:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Mon Aug 12 18:53:56 2013
From: mgraesslin at kde.org (=?utf-8?q?Martin_Gr=C3=A4=C3=9Flin?=)
Date: Mon, 12 Aug 2013 18:53:56 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812175031.11937.37398@vidsolbach.de>
References: <20130812175031.11937.37398@vidsolbach.de>
Message-ID: <20130812185356.24351.22490@vidsolbach.de>
> On Aug. 12, 2013, 7:50 p.m., Ivan ?uki? wrote:
> > Isn't it also used for reporting wid to share-like-connect (activity manager) in some applications?
well it got introduced for KWin:
commit 52bc2bd662b0e82191922d8fc81b74cc2895d3c0
Author: Martin Gr??lin
Date: Sat Mar 24 10:51:12 2012 +0100
Export WindowId on Plasma Dialog
Exporting the window Id allows to use it in declarative KWin scripts
to embedd window thumbnails on the dialog.
REVIEW: 104393
---
do you have an example of how it is used? Because most likely it should be easy to port in that case, too.
- Martin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37602
-----------------------------------------------------------
On Aug. 12, 2013, 7:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 7:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From ivan.cukic at kde.org Mon Aug 12 18:59:27 2013
From: ivan.cukic at kde.org (=?utf-8?b?SXZhbiDEjHVracSH?=)
Date: Mon, 12 Aug 2013 18:59:27 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812175031.11937.37398@vidsolbach.de>
References: <20130812175031.11937.37398@vidsolbach.de>
Message-ID: <20130812185927.24576.61988@vidsolbach.de>
> On Aug. 12, 2013, 5:50 p.m., Ivan ?uki? wrote:
> > Isn't it also used for reporting wid to share-like-connect (activity manager) in some applications?
>
> Martin Gr??lin wrote:
> well it got introduced for KWin:
>
> commit 52bc2bd662b0e82191922d8fc81b74cc2895d3c0
> Author: Martin Gr??lin
> Date: Sat Mar 24 10:51:12 2012 +0100
>
> Export WindowId on Plasma Dialog
>
> Exporting the window Id allows to use it in declarative KWin scripts
> to embedd window thumbnails on the dialog.
>
> REVIEW: 104393
>
> ---
> do you have an example of how it is used? Because most likely it should be easy to port in that case, too.
>
This was a question for Marco - sorry for not explicitely specifying it. I'm just wondering whther he used it in image viewer, web browser ect. of Plasma Active.
- Ivan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37602
-----------------------------------------------------------
On Aug. 12, 2013, 5:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 5:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Mon Aug 12 21:59:42 2013
From: notmart at gmail.com (Marco Martin)
Date: Mon, 12 Aug 2013 21:59:42 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812175031.11937.37398@vidsolbach.de>
References: <20130812175031.11937.37398@vidsolbach.de>
Message-ID: <20130812215942.10911.22546@vidsolbach.de>
> On Aug. 12, 2013, 5:50 p.m., Ivan ?uki? wrote:
> > Isn't it also used for reporting wid to share-like-connect (activity manager) in some applications?
>
> Martin Gr??lin wrote:
> well it got introduced for KWin:
>
> commit 52bc2bd662b0e82191922d8fc81b74cc2895d3c0
> Author: Martin Gr??lin
> Date: Sat Mar 24 10:51:12 2012 +0100
>
> Export WindowId on Plasma Dialog
>
> Exporting the window Id allows to use it in declarative KWin scripts
> to embedd window thumbnails on the dialog.
>
> REVIEW: 104393
>
> ---
> do you have an example of how it is used? Because most likely it should be easy to port in that case, too.
>
>
> Ivan ?uki? wrote:
> This was a question for Marco - sorry for not explicitely specifying it. I'm just wondering whther he used it in image viewer, web browser ect. of Plasma Active.
as far i know for slc in qml usually the resourceinstance component is used, that one fetches it from its c++ implementation
- Marco
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37602
-----------------------------------------------------------
On Aug. 12, 2013, 5:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 5:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Mon Aug 12 22:00:40 2013
From: notmart at gmail.com (Marco Martin)
Date: Mon, 12 Aug 2013 22:00:40 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812173445.10948.86699@vidsolbach.de>
References: <20130812173445.10948.86699@vidsolbach.de>
Message-ID: <20130812220040.10984.4822@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37620
-----------------------------------------------------------
Ship it!
Ship It!
- Marco Martin
On Aug. 12, 2013, 5:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 5:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 13 04:47:08 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 04:47:08 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812173445.10948.86699@vidsolbach.de>
References: <20130812173445.10948.86699@vidsolbach.de>
Message-ID: <20130813044708.8271.44162@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/#review37635
-----------------------------------------------------------
This review has been submitted with commit f50a5412ba83058dcbfa3acff4b602cc87675d9a by Martin Gr??lin to branch master.
- Commit Hook
On Aug. 12, 2013, 5:34 p.m., Martin Gr??lin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112033/
> -----------------------------------------------------------
>
> (Updated Aug. 12, 2013, 5:34 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
>
>
> Diffs
> -----
>
> src/declarativeimports/core/dialog.h 7392b71
> src/declarativeimports/core/dialog.cpp eaf3dce
>
> Diff: http://git.reviewboard.kde.org/r/112033/diff/
>
>
> Testing
> -------
>
> Yes, KWin's ThumnailItem is still working
>
>
> Thanks,
>
> Martin Gr??lin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 13 04:47:12 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 04:47:12 -0000
Subject: Review Request 112033: Drop windowId property from
PlasmaCore.Dialog
In-Reply-To: <20130812173445.10948.86699@vidsolbach.de>
References: <20130812173445.10948.86699@vidsolbach.de>
Message-ID: <20130813044712.8439.84592@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/
-----------------------------------------------------------
(Updated Aug. 13, 2013, 4:47 a.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Marco Martin.
Description
-------
Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
Diffs
-----
src/declarativeimports/core/dialog.h 7392b71
src/declarativeimports/core/dialog.cpp eaf3dce
Diff: http://git.reviewboard.kde.org/r/112033/diff/
Testing
-------
Yes, KWin's ThumnailItem is still working
Thanks,
Martin Gr??lin
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From david at davidedmundson.co.uk Tue Aug 13 13:33:18 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Tue, 13 Aug 2013 13:33:18 -0000
Subject: Review Request 112058: Enable apps dataengine for KF5
Message-ID: <20130813133318.4737.96143@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112058/
-----------------------------------------------------------
Review request for Plasma.
Description
-------
Enable apps dataengine for KF5
Diffs
-----
plasma/generic/dataengines/CMakeLists.txt 923a519bd6ee2c460968ac9de71550c126425ee8
plasma/generic/dataengines/apps/CMakeLists.txt 05d139cbfd306a2e2003f75280e09958af46dcf2
plasma/generic/dataengines/apps/appjob.cpp 3a91b1e4de6119e0f13013dd255b7e15415284bd
plasma/generic/dataengines/apps/appsengine.cpp 340705c05e091c75909665180f51a62cb69acb14
Diff: http://git.reviewboard.kde.org/r/112058/diff/
Testing
-------
Checked it appeared in plasmaengineexplorer
I have not verified it shows apps as my kf5 build dir does not contains any.
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Tue Aug 13 13:39:28 2013
From: notmart at gmail.com (Marco Martin)
Date: Tue, 13 Aug 2013 13:39:28 -0000
Subject: Review Request 112058: Enable apps dataengine for KF5
In-Reply-To: <20130813133318.4737.96143@vidsolbach.de>
References: <20130813133318.4737.96143@vidsolbach.de>
Message-ID: <20130813133928.5551.66561@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112058/#review37692
-----------------------------------------------------------
Ship it!
Ship It!
- Marco Martin
On Aug. 13, 2013, 1:33 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112058/
> -----------------------------------------------------------
>
> (Updated Aug. 13, 2013, 1:33 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Enable apps dataengine for KF5
>
>
> Diffs
> -----
>
> plasma/generic/dataengines/CMakeLists.txt 923a519bd6ee2c460968ac9de71550c126425ee8
> plasma/generic/dataengines/apps/CMakeLists.txt 05d139cbfd306a2e2003f75280e09958af46dcf2
> plasma/generic/dataengines/apps/appjob.cpp 3a91b1e4de6119e0f13013dd255b7e15415284bd
> plasma/generic/dataengines/apps/appsengine.cpp 340705c05e091c75909665180f51a62cb69acb14
>
> Diff: http://git.reviewboard.kde.org/r/112058/diff/
>
>
> Testing
> -------
>
> Checked it appeared in plasmaengineexplorer
> I have not verified it shows apps as my kf5 build dir does not contains any.
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 13 13:48:25 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 13:48:25 -0000
Subject: Review Request 112058: Enable apps dataengine for KF5
In-Reply-To: <20130813133318.4737.96143@vidsolbach.de>
References: <20130813133318.4737.96143@vidsolbach.de>
Message-ID: <20130813134825.6122.74911@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112058/#review37693
-----------------------------------------------------------
This review has been submitted with commit 6b7471fcb032ef44f095dcd5c2927325e50b3afc by David Edmundson to branch frameworks-scratch.
- Commit Hook
On Aug. 13, 2013, 1:33 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112058/
> -----------------------------------------------------------
>
> (Updated Aug. 13, 2013, 1:33 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Enable apps dataengine for KF5
>
>
> Diffs
> -----
>
> plasma/generic/dataengines/CMakeLists.txt 923a519bd6ee2c460968ac9de71550c126425ee8
> plasma/generic/dataengines/apps/CMakeLists.txt 05d139cbfd306a2e2003f75280e09958af46dcf2
> plasma/generic/dataengines/apps/appjob.cpp 3a91b1e4de6119e0f13013dd255b7e15415284bd
> plasma/generic/dataengines/apps/appsengine.cpp 340705c05e091c75909665180f51a62cb69acb14
>
> Diff: http://git.reviewboard.kde.org/r/112058/diff/
>
>
> Testing
> -------
>
> Checked it appeared in plasmaengineexplorer
> I have not verified it shows apps as my kf5 build dir does not contains any.
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 13 13:48:27 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 13:48:27 -0000
Subject: Review Request 112058: Enable apps dataengine for KF5
In-Reply-To: <20130813133318.4737.96143@vidsolbach.de>
References: <20130813133318.4737.96143@vidsolbach.de>
Message-ID: <20130813134827.6099.6809@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112058/
-----------------------------------------------------------
(Updated Aug. 13, 2013, 1:48 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma.
Description
-------
Enable apps dataengine for KF5
Diffs
-----
plasma/generic/dataengines/CMakeLists.txt 923a519bd6ee2c460968ac9de71550c126425ee8
plasma/generic/dataengines/apps/CMakeLists.txt 05d139cbfd306a2e2003f75280e09958af46dcf2
plasma/generic/dataengines/apps/appjob.cpp 3a91b1e4de6119e0f13013dd255b7e15415284bd
plasma/generic/dataengines/apps/appsengine.cpp 340705c05e091c75909665180f51a62cb69acb14
Diff: http://git.reviewboard.kde.org/r/112058/diff/
Testing
-------
Checked it appeared in plasmaengineexplorer
I have not verified it shows apps as my kf5 build dir does not contains any.
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From david at davidedmundson.co.uk Tue Aug 13 13:49:30 2013
From: david at davidedmundson.co.uk (David Edmundson)
Date: Tue, 13 Aug 2013 13:49:30 -0000
Subject: Review Request 112059: Add reviewboardrc file
Message-ID: <20130813134930.6235.47936@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112059/
-----------------------------------------------------------
Review request for Plasma.
Description
-------
Add reviewboardrc file
Diffs
-----
.reviewboardrc PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/112059/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Tue Aug 13 14:56:15 2013
From: sebas at kde.org (=?utf-8?q?Sebastian_K=C3=BCgler?=)
Date: Tue, 13 Aug 2013 14:56:15 -0000
Subject: Review Request 112059: Add reviewboardrc file
In-Reply-To: <20130813134930.6235.47936@vidsolbach.de>
References: <20130813134930.6235.47936@vidsolbach.de>
Message-ID: <20130813145615.19041.54650@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112059/#review37697
-----------------------------------------------------------
Ship it!
Ship It!
- Sebastian K?gler
On Aug. 13, 2013, 1:49 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112059/
> -----------------------------------------------------------
>
> (Updated Aug. 13, 2013, 1:49 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Add reviewboardrc file
>
>
> Diffs
> -----
>
> .reviewboardrc PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/112059/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Tue Aug 13 16:26:08 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Tue, 13 Aug 2013 16:26:08 +0000
Subject: plasma-framework cmake error after git pull --rebase
Message-ID:
Hi ,
In order to make use of data engine tasks in plasma 2 .I did git pull
--rebase in extra-cmake-module and kdelibs and rebuild it using runtime
setup .Then I did the same with plasma-framework .But getting cmake error
on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn
Please help me out :p (plasma framework takes time ;) )
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Tue Aug 13 19:36:09 2013
From: notmart at gmail.com (Marco Martin)
Date: Tue, 13 Aug 2013 12:36:09 -0700 (PDT)
Subject: plasma-framework cmake error after git pull --rebase
In-Reply-To:
References:
Message-ID: <4110686.Uf8LMm1XLv@deimos.diaunet>
On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote:
> Hi ,
> In order to make use of data engine tasks in plasma 2 .I did git pull
> --rebase in extra-cmake-module and kdelibs and rebuild it using runtime
> setup .Then I did the same with plasma-framework .But getting cmake error
> on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn
> Please help me out :p (plasma framework takes time ;) )
try to add -DCMAKE_PREFIX_PATH=$KF5 to the cmake command line, should tell to
actually search for kdelibs under $KF5
--
Marco Martin
From null at kde.org Tue Aug 13 20:11:44 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 20:11:44 -0000
Subject: Review Request 112059: Add reviewboardrc file
In-Reply-To: <20130813134930.6235.47936@vidsolbach.de>
References: <20130813134930.6235.47936@vidsolbach.de>
Message-ID: <20130813201144.32200.8681@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112059/#review37724
-----------------------------------------------------------
This review has been submitted with commit 366d988f7d681cf52c741a8bec3bb22e2b7232b7 by David Edmundson to branch frameworks-scratch.
- Commit Hook
On Aug. 13, 2013, 1:49 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112059/
> -----------------------------------------------------------
>
> (Updated Aug. 13, 2013, 1:49 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> Add reviewboardrc file
>
>
> Diffs
> -----
>
> .reviewboardrc PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/112059/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Tue Aug 13 20:11:47 2013
From: null at kde.org (Commit Hook)
Date: Tue, 13 Aug 2013 20:11:47 -0000
Subject: Review Request 112059: Add reviewboardrc file
In-Reply-To: <20130813134930.6235.47936@vidsolbach.de>
References: <20130813134930.6235.47936@vidsolbach.de>
Message-ID: <20130813201147.32208.19389@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112059/
-----------------------------------------------------------
(Updated Aug. 13, 2013, 8:11 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma.
Description
-------
Add reviewboardrc file
Diffs
-----
.reviewboardrc PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/112059/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From hrvoje.senjan at gmail.com Tue Aug 13 23:20:58 2013
From: hrvoje.senjan at gmail.com (Hrvoje Senjan)
Date: Tue, 13 Aug 2013 23:20:58 -0000
Subject: Review Request 112069: Call KPluginInfo::KPluginInfo with the resource
argument
Message-ID: <20130813232058.23737.56418@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112069/
-----------------------------------------------------------
Review request for Plasma.
Description
-------
>From the bug report:
"I have analyzed the crashed application in gdb, and I believe this code in Plasma::ThemePrivate::useCache() is the real culprit:
230 if (isRegularTheme) {
231 const QString path = KStandardDirs::locate("data", "desktoptheme/" + themeName + "/metadata.desktop");
232 const KPluginInfo pluginInfo(path);
Note that "path" is a relative path and KPluginInfo::KPluginInfo is called with only one argument. However, KPluginInfo constructor is declared as:
* @param filename The filename of the .desktop file.
* @param resource If filename is relative, you need to specify a resource type
* (e.g. "service", "apps"... KStandardDirs). Otherwise,
* resource isn't used.
*/
explicit KPluginInfo( const QString & filename, const char* resource = 0 );
So, if you call it without the "resource" argument, the default value is used, which is NULL. This is invalid for a relative path, and indeed causes the crash further down the stack in KStandardDirs::saveLocation():
1499 if (dirs.isEmpty() && (
1500 (strcmp(type, "socket") == 0) ||
1501 (strcmp(type, "tmp") == 0) ||
1502 (strcmp(type, "cache") == 0) ))
Unsurprisingly, calling strcmp(NULL, something) will cause a SIGSEGV."
Note that it's not my patch, but i feel it's a waste it's just sitting there in the report, and not here ;-)
Issue is triggered very easily by explcitly definining a plasma theme which does not exist (with e.g. adding
[Theme]
name=fake-non-existant-theme
in plasmarc)
This addresses bugs 320855 and 322446.
http://bugs.kde.org/show_bug.cgi?id=320855
http://bugs.kde.org/show_bug.cgi?id=322446
Diffs
-----
plasma/theme.cpp 4554de7
Diff: http://git.reviewboard.kde.org/r/112069/diff/
Testing
-------
Tried the above example with the patch, plasma doesn't crash
Thanks,
Hrvoje Senjan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From hrvoje.senjan at gmail.com Wed Aug 14 00:01:17 2013
From: hrvoje.senjan at gmail.com (Hrvoje Senjan)
Date: Wed, 14 Aug 2013 00:01:17 -0000
Subject: Review Request 112073: Partially revert commit 83d3018
Message-ID: <20130814000117.13131.43688@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112073/
-----------------------------------------------------------
Review request for Plasma and David Faure.
Description
-------
While the commit may be technically correct, it breaks ksplash for 4.11 release...
I've changed unique to multi, if there is a better solution, please hint it ;-)
(maybe actually someone meant multi, when typing wait :-)
This addresses bug 321695.
http://bugs.kde.org/show_bug.cgi?id=321695
Diffs
-----
plasma/desktop/shell/data/plasma-desktop.desktop f37dc57
Diff: http://git.reviewboard.kde.org/r/112073/diff/
Testing
-------
Yes... starts "normally" as previously/ksplash isn't interupted
Thanks,
Hrvoje Senjan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Wed Aug 14 05:57:43 2013
From: mgraesslin at kde.org (Martin Graesslin)
Date: Wed, 14 Aug 2013 07:57:43 +0200
Subject: Announcing kwindoweffectsplugin
Message-ID: <2154403.svqbvbK2BQ@martin-thinkpad>
Hi all,
to help me with working on KWindowEffects I decided to write a small qml plugin
which exports all the methods of KWindowEffects to qml.
I just pushed this plugin to a personal scratch repository:
kde:scratch/graesslin/kwindoweffectsplugin [1]
It contains an example.qml which can be loaded in qmlscene and shows how to
use the plugin and also is a nice example of what we can do with the
KWindowEffects.
I am not sure whether this plugin is of use for more things than just an
example. So feel free to do with it as you wish or tell me that it is useful
and what I should do with it.
Cheers
Martin
[1]
http://commits.kde.org/scratch/graesslin/kwindoweffectsplugin/edc6bbb6c23d1cde45362f8a5433a548a2651081
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL:
From bhush94 at gmail.com Wed Aug 14 07:06:18 2013
From: bhush94 at gmail.com (Bhushan Shah)
Date: Wed, 14 Aug 2013 12:36:18 +0530
Subject: Compiling QT5 for KF5
Message-ID:
Hello,
I want to compile QT 5.2 for compiling kde-framework 5, So want to ask if I
can ignore some submodules to speedup compiling process? Like qt multimedia
package etc?
Which packages from Qt5 are required to build KF5?
Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From mgraesslin at kde.org Wed Aug 14 07:33:53 2013
From: mgraesslin at kde.org (Martin Graesslin)
Date: Wed, 14 Aug 2013 09:33:53 +0200
Subject: Compiling QT5 for KF5
In-Reply-To:
References:
Message-ID: <8100541.VleHKN6BFS@martin-thinkpad>
On Wednesday 14 August 2013 12:36:18 Bhushan Shah wrote:
> Hello,
>
> I want to compile QT 5.2 for compiling kde-framework 5, So want to ask if I
> can ignore some submodules to speedup compiling process? Like qt multimedia
> package etc?
depending on your distribution there might be "daily" built packages of Qt 5.2
available. So which distro are you using?
Cheers
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL:
From bhush94 at gmail.com Wed Aug 14 07:43:19 2013
From: bhush94 at gmail.com (Bhushan Shah)
Date: Wed, 14 Aug 2013 13:13:19 +0530
Subject: Compiling QT5 for KF5
In-Reply-To: <8100541.VleHKN6BFS@martin-thinkpad>
References:
<8100541.VleHKN6BFS@martin-thinkpad>
Message-ID:
Hello, I am using Arch Linux i686. Thanks!
On 8/14/13, Martin Graesslin wrote:
> On Wednesday 14 August 2013 12:36:18 Bhushan Shah wrote:
>> Hello,
>>
>> I want to compile QT 5.2 for compiling kde-framework 5, So want to ask if
>> I
>> can ignore some submodules to speedup compiling process? Like qt
>> multimedia
>> package etc?
> depending on your distribution there might be "daily" built packages of Qt
> 5.2
> available. So which distro are you using?
>
> Cheers
> Martin
>
From mgraesslin at kde.org Wed Aug 14 08:44:37 2013
From: mgraesslin at kde.org (Martin Graesslin)
Date: Wed, 14 Aug 2013 10:44:37 +0200
Subject: Compiling QT5 for KF5
In-Reply-To:
References:
<8100541.VleHKN6BFS@martin-thinkpad>
Message-ID: <1531398.4GBTIWfQFz@martin-thinkpad>
On Wednesday 14 August 2013 13:13:19 Bhushan Shah wrote:
> Hello, I am using Arch Linux i686. Thanks!
I'm not the Arch expert, but it seems like something exists:
https://bbs.archlinux.org/viewtopic.php?id=164647
Cheers
Martin
>
> On 8/14/13, Martin Graesslin wrote:
> > On Wednesday 14 August 2013 12:36:18 Bhushan Shah wrote:
> >> Hello,
> >>
> >> I want to compile QT 5.2 for compiling kde-framework 5, So want to ask if
> >> I
> >> can ignore some submodules to speedup compiling process? Like qt
> >> multimedia
> >> package etc?
> >
> > depending on your distribution there might be "daily" built packages of Qt
> > 5.2
> > available. So which distro are you using?
> >
> > Cheers
> > Martin
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL:
From bhush94 at gmail.com Wed Aug 14 08:53:12 2013
From: bhush94 at gmail.com (Bhushan Shah)
Date: Wed, 14 Aug 2013 14:23:12 +0530
Subject: Compiling QT5 for KF5
In-Reply-To: <1531398.4GBTIWfQFz@martin-thinkpad>
References:
<8100541.VleHKN6BFS@martin-thinkpad>
<1531398.4GBTIWfQFz@martin-thinkpad>
Message-ID:
Hello,
On Wed, Aug 14, 2013 at 2:14 PM, Martin Graesslin wrote:
> I'm not the Arch expert, but it seems like something exists:
> https://bbs.archlinux.org/viewtopic.php?id=164647
>
> Cheers
> Martin
>
Wish I could get PKGBUILDs, Its available for 64 bit only. I asked Andrea
Scarpino but no reply yet, May be different time zone! :-)
Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From hrvoje.senjan at gmail.com Wed Aug 14 10:56:33 2013
From: hrvoje.senjan at gmail.com (Hrvoje Senjan)
Date: Wed, 14 Aug 2013 10:56:33 -0000
Subject: Review Request 112069: Call KPluginInfo::KPluginInfo with the
resource argument
In-Reply-To: <20130813232058.23737.56418@vidsolbach.de>
References: <20130813232058.23737.56418@vidsolbach.de>
Message-ID: <20130814105633.14143.82256@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112069/
-----------------------------------------------------------
(Updated Aug. 14, 2013, 10:56 a.m.)
Status
------
This change has been discarded.
Review request for Plasma.
Description
-------
>From the bug report:
"I have analyzed the crashed application in gdb, and I believe this code in Plasma::ThemePrivate::useCache() is the real culprit:
230 if (isRegularTheme) {
231 const QString path = KStandardDirs::locate("data", "desktoptheme/" + themeName + "/metadata.desktop");
232 const KPluginInfo pluginInfo(path);
Note that "path" is a relative path and KPluginInfo::KPluginInfo is called with only one argument. However, KPluginInfo constructor is declared as:
* @param filename The filename of the .desktop file.
* @param resource If filename is relative, you need to specify a resource type
* (e.g. "service", "apps"... KStandardDirs). Otherwise,
* resource isn't used.
*/
explicit KPluginInfo( const QString & filename, const char* resource = 0 );
So, if you call it without the "resource" argument, the default value is used, which is NULL. This is invalid for a relative path, and indeed causes the crash further down the stack in KStandardDirs::saveLocation():
1499 if (dirs.isEmpty() && (
1500 (strcmp(type, "socket") == 0) ||
1501 (strcmp(type, "tmp") == 0) ||
1502 (strcmp(type, "cache") == 0) ))
Unsurprisingly, calling strcmp(NULL, something) will cause a SIGSEGV."
Note that it's not my patch, but i feel it's a waste it's just sitting there in the report, and not here ;-)
Issue is triggered very easily by explcitly definining a plasma theme which does not exist (with e.g. adding
[Theme]
name=fake-non-existant-theme
in plasmarc)
This addresses bugs 320855 and 322446.
http://bugs.kde.org/show_bug.cgi?id=320855
http://bugs.kde.org/show_bug.cgi?id=322446
Diffs
-----
plasma/theme.cpp 4554de7
Diff: http://git.reviewboard.kde.org/r/112069/diff/
Testing
-------
Tried the above example with the patch, plasma doesn't crash
Thanks,
Hrvoje Senjan
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Wed Aug 14 11:49:07 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Wed, 14 Aug 2013 11:49:07 +0000
Subject: plasma-framework cmake error after git pull --rebase
In-Reply-To: <4110686.Uf8LMm1XLv@deimos.diaunet>
References:
<4110686.Uf8LMm1XLv@deimos.diaunet>
Message-ID:
I used cmake -DCMAKE_INSTALL_PREFIX=$KF5 ..
On Tue, Aug 13, 2013 at 7:36 PM, Marco Martin wrote:
> On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote:
> > Hi ,
> > In order to make use of data engine tasks in plasma 2 .I did git pull
> > --rebase in extra-cmake-module and kdelibs and rebuild it using runtime
> > setup .Then I did the same with plasma-framework .But getting cmake error
> > on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn
> > Please help me out :p (plasma framework takes time ;) )
>
> try to add -DCMAKE_PREFIX_PATH=$KF5 to the cmake command line, should tell
> to
> actually search for kdelibs under $KF5
>
> --
> Marco Martin
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Wed Aug 14 14:56:37 2013
From: null at kde.org (Commit Hook)
Date: Wed, 14 Aug 2013 14:56:37 -0000
Subject: Review Request 111899: Make QGuiPlatformPlugin react to
iconChanges
In-Reply-To: <20130806070219.19310.95079@vidsolbach.de>
References: <20130806070219.19310.95079@vidsolbach.de>
Message-ID: <20130814145637.18071.54418@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111899/#review37767
-----------------------------------------------------------
This review has been submitted with commit e74a264325e8af834d800ee650c4bf208e1b2fb1 by ?lex Fiestas to branch KDE/4.11.
- Commit Hook
On Aug. 6, 2013, 7:02 a.m., ?lex Fiestas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111899/
> -----------------------------------------------------------
>
> (Updated Aug. 6, 2013, 7:02 a.m.)
>
>
> Review request for Plasma and Olivier Goffart.
>
>
> Description
> -------
>
> When KDE changes the iconSize, send a StyleChange event to QToolbar and QMainWindow (this one is required for QToolBar that are children of it).
>
>
> Diffs
> -----
>
> qguiplatformplugin_kde/qguiplatformplugin_kde.cpp cc74dc0
>
> Diff: http://git.reviewboard.kde.org/r/111899/diff/
>
>
> Testing
> -------
>
> Played a while with assistant, designer and quasselclient, seems to work fine.
>
>
> Thanks,
>
> ?lex Fiestas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From null at kde.org Wed Aug 14 14:56:52 2013
From: null at kde.org (Commit Hook)
Date: Wed, 14 Aug 2013 14:56:52 -0000
Subject: Review Request 111899: Make QGuiPlatformPlugin react to
iconChanges
In-Reply-To: <20130806070219.19310.95079@vidsolbach.de>
References: <20130806070219.19310.95079@vidsolbach.de>
Message-ID: <20130814145652.18093.34588@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111899/
-----------------------------------------------------------
(Updated Aug. 14, 2013, 2:56 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Olivier Goffart.
Description
-------
When KDE changes the iconSize, send a StyleChange event to QToolbar and QMainWindow (this one is required for QToolBar that are children of it).
Diffs
-----
qguiplatformplugin_kde/qguiplatformplugin_kde.cpp cc74dc0
Diff: http://git.reviewboard.kde.org/r/111899/diff/
Testing
-------
Played a while with assistant, designer and quasselclient, seems to work fine.
Thanks,
?lex Fiestas
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From michael.bohlender at kdemail.net Wed Aug 14 15:52:55 2013
From: michael.bohlender at kdemail.net (Michael Bohlender)
Date: Wed, 14 Aug 2013 15:52:55 -0000
Subject: Review Request 112088: active-about: add new Contributors for PA 4
Message-ID: <20130814155255.31553.41552@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112088/
-----------------------------------------------------------
Review request for Plasma and Ruediger Gad.
Description
-------
Adds rcg and myself to the contributors.
I still wait for a reply from rcg. maybe he does not want to get listed, or his name is really Ruediger and not R?diger. :)
Diffs
-----
applications/about/package/contents/data/authors.html 1feb156
Diff: http://git.reviewboard.kde.org/r/112088/diff/
Testing
-------
Thanks,
Michael Bohlender
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Wed Aug 14 19:21:40 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Wed, 14 Aug 2013 19:21:40 +0000
Subject: plasma-framework cmake error after git pull --rebase
In-Reply-To:
References:
<4110686.Uf8LMm1XLv@deimos.diaunet>
Message-ID:
This is my runtime set up http://pastebin.com/raw.php?i=hu9BfWmV .Please
suggest what I am missing out.
And this is output of $env http://pastebin.com/raw.php?i=9wRZyPg3
Regards
On Wed, Aug 14, 2013 at 11:49 AM, Heena Mahour wrote:
> I used cmake -DCMAKE_INSTALL_PREFIX=$KF5 ..
>
>
>
> On Tue, Aug 13, 2013 at 7:36 PM, Marco Martin wrote:
>
>> On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote:
>> > Hi ,
>> > In order to make use of data engine tasks in plasma 2 .I did git pull
>> > --rebase in extra-cmake-module and kdelibs and rebuild it using runtime
>> > setup .Then I did the same with plasma-framework .But getting cmake
>> error
>> > on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn
>> > Please help me out :p (plasma framework takes time ;) )
>>
>> try to add -DCMAKE_PREFIX_PATH=$KF5 to the cmake command line, should
>> tell to
>> actually search for kdelibs under $KF5
>>
>> --
>> Marco Martin
>> _______________________________________________
>> Plasma-devel mailing list
>> Plasma-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/plasma-devel
>>
>
>
>
> --
> -Heena
> Season of kde'12 participant
> Google Summer of Code 2013
> Delhi College of Engineering(COE),India
> http://about.me/heena.mahour
> http://heenamahour.blogspot.in
>
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Wed Aug 14 20:22:12 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Wed, 14 Aug 2013 20:22:12 +0000
Subject: plasma-framework cmake error after git pull --rebase
In-Reply-To:
References:
<4110686.Uf8LMm1XLv@deimos.diaunet>
Message-ID:
Now I am able to build but on $plasma-shell I am getting this
http://pastebin.com/raw.php?i=ZeK1kquz ...
On Wed, Aug 14, 2013 at 7:21 PM, Heena Mahour wrote:
> This is my runtime set up http://pastebin.com/raw.php?i=hu9BfWmV .Please
> suggest what I am missing out.
> And this is output of $env http://pastebin.com/raw.php?i=9wRZyPg3
> Regards
>
>
> On Wed, Aug 14, 2013 at 11:49 AM, Heena Mahour wrote:
>
>> I used cmake -DCMAKE_INSTALL_PREFIX=$KF5 ..
>>
>>
>>
>> On Tue, Aug 13, 2013 at 7:36 PM, Marco Martin wrote:
>>
>>> On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote:
>>> > Hi ,
>>> > In order to make use of data engine tasks in plasma 2 .I did git pull
>>> > --rebase in extra-cmake-module and kdelibs and rebuild it using runtime
>>> > setup .Then I did the same with plasma-framework .But getting cmake
>>> error
>>> > on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn
>>> > Please help me out :p (plasma framework takes time ;) )
>>>
>>> try to add -DCMAKE_PREFIX_PATH=$KF5 to the cmake command line, should
>>> tell to
>>> actually search for kdelibs under $KF5
>>>
>>> --
>>> Marco Martin
>>> _______________________________________________
>>> Plasma-devel mailing list
>>> Plasma-devel at kde.org
>>> https://mail.kde.org/mailman/listinfo/plasma-devel
>>>
>>
>>
>>
>> --
>> -Heena
>> Season of kde'12 participant
>> Google Summer of Code 2013
>> Delhi College of Engineering(COE),India
>> http://about.me/heena.mahour
>> http://heenamahour.blogspot.in
>>
>>
>
>
> --
> -Heena
> Season of kde'12 participant
> Google Summer of Code 2013
> Delhi College of Engineering(COE),India
> http://about.me/heena.mahour
> http://heenamahour.blogspot.in
>
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From terietor at gmail.com Thu Aug 15 06:32:44 2013
From: terietor at gmail.com (Giorgos Tsiapaliokas)
Date: Thu, 15 Aug 2013 09:32:44 +0300
Subject: Compiling QT5 for KF5
In-Reply-To:
References:
<8100541.VleHKN6BFS@martin-thinkpad>
<1531398.4GBTIWfQFz@martin-thinkpad>
Message-ID:
On 14 August 2013 11:53, Bhushan Shah wrote:
>
> Wish I could get PKGBUILDs, Its available for 64 bit only. I asked Andrea
> Scarpino but no reply yet, May be different time zone! :-)
>
> Thanks!
>
https://aur.archlinux.org/packages/?O=0&K=kf5
--
Giorgos Tsiapaliokas (terietor)
terietor.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From bhush94 at gmail.com Thu Aug 15 06:55:41 2013
From: bhush94 at gmail.com (Bhushan Shah)
Date: Thu, 15 Aug 2013 12:25:41 +0530
Subject: Compiling QT5 for KF5
In-Reply-To:
References:
<8100541.VleHKN6BFS@martin-thinkpad>
<1531398.4GBTIWfQFz@martin-thinkpad>
Message-ID:
So, that means, KF5 *compiled* PKGs are available on repo, and PKGBUILDs to
build it are available on AUR? Right?
On Thu, Aug 15, 2013 at 12:02 PM, Giorgos Tsiapaliokas
wrote:
> https://aur.archlinux.org/packages/?O=0&K=kf5
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From sebas at kde.org Thu Aug 15 12:12:21 2013
From: sebas at kde.org (=?utf-8?q?Sebastian_K=C3=BCgler?=)
Date: Thu, 15 Aug 2013 12:12:21 -0000
Subject: Review Request 112088: active-about: add new Contributors for PA 4
In-Reply-To: <20130814155255.31553.41552@vidsolbach.de>
References: <20130814155255.31553.41552@vidsolbach.de>
Message-ID: <20130815121221.20793.90431@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112088/#review37838
-----------------------------------------------------------
Ship it!
One R(?|ue)diger has answered, you can commit it.
- Sebastian K?gler
On Aug. 14, 2013, 3:52 p.m., Michael Bohlender wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112088/
> -----------------------------------------------------------
>
> (Updated Aug. 14, 2013, 3:52 p.m.)
>
>
> Review request for Plasma and Ruediger Gad.
>
>
> Description
> -------
>
> Adds rcg and myself to the contributors.
> I still wait for a reply from rcg. maybe he does not want to get listed, or his name is really Ruediger and not R?diger. :)
>
>
> Diffs
> -----
>
> applications/about/package/contents/data/authors.html 1feb156
>
> Diff: http://git.reviewboard.kde.org/r/112088/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Michael Bohlender
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From kde at privat.broulik.de Thu Aug 15 14:51:40 2013
From: kde at privat.broulik.de (Kai Uwe Broulik)
Date: Thu, 15 Aug 2013 16:51:40 +0200
Subject: Actual width of checkbox element
Message-ID: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
Hi there,
I got a bug report [1] about that the popup width might get so small that the
PM checkbox doesn't fit entirely.
Is there a way to determin the actual width of the checkbox including its
label?
So I could do minimumWidth: Math.max( currentminimumwidth, checkboxwidth)
Or should I just use a fixed, say, 250px minimum width?
Best,
Kai Uwe
[1] https://bugs.kde.org/show_bug.cgi?id=323530
From dantti12 at gmail.com Thu Aug 15 15:11:43 2013
From: dantti12 at gmail.com (Daniel Nicoletti)
Date: Thu, 15 Aug 2013 12:11:43 -0300
Subject: Actual width of checkbox element
In-Reply-To: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
Message-ID:
This is the reason in Apper updater the checkbox
doesn't use the text property, instead I have a checkbox
with no text and a Text element next to it and eliding right.
I don't think you can do it other way as there is no
other properties exposed, but I might be wrong...
2013/8/15 Kai Uwe Broulik :
> Hi there,
>
> I got a bug report [1] about that the popup width might get so small that the
> PM checkbox doesn't fit entirely.
> Is there a way to determin the actual width of the checkbox including its
> label?
> So I could do minimumWidth: Math.max( currentminimumwidth, checkboxwidth)
> Or should I just use a fixed, say, 250px minimum width?
>
> Best,
> Kai Uwe
>
> [1] https://bugs.kde.org/show_bug.cgi?id=323530
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
--
Daniel Nicoletti
KDE Developer - http://dantti.wordpress.com
From notmart at gmail.com Thu Aug 15 15:13:18 2013
From: notmart at gmail.com (Marco Martin)
Date: Thu, 15 Aug 2013 17:13:18 +0200
Subject: Actual width of checkbox element
In-Reply-To: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
Message-ID: <201308151713.19164.notmart@gmail.com>
On Thursday 15 August 2013, Kai Uwe Broulik wrote:
> Hi there,
>
> I got a bug report [1] about that the popup width might get so small that
> the PM checkbox doesn't fit entirely.
> Is there a way to determin the actual width of the checkbox including its
> label?
> So I could do minimumWidth: Math.max( currentminimumwidth, checkboxwidth)
> Or should I just use a fixed, say, 250px minimum width?
its implicitWidth should return a proper value (if it doesn't means the
component should be fixed ;)
--
Marco Martin
From heena393 at gmail.com Thu Aug 15 15:32:18 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Thu, 15 Aug 2013 15:32:18 +0000
Subject: plasma-shell in plasma2
Message-ID:
Hey
I am getting http://picpaste.com/screen13-GNKwWhEy.png this on plasma-shell.
This is the output on terminal http://pastebin.com/raw.php?i=zL2fMK8c
Could you suggest how I may load plasmoid in plasma-shell ?
Regards
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Thu Aug 15 15:38:25 2013
From: notmart at gmail.com (Marco Martin)
Date: Thu, 15 Aug 2013 17:38:25 +0200
Subject: plasma-shell in plasma2
In-Reply-To:
References:
Message-ID: <201308151738.25492.notmart@gmail.com>
On Thursday 15 August 2013, Heena Mahour wrote:
> Hey
>
> I am getting http://picpaste.com/screen13-GNKwWhEy.png this on
> plasma-shell. This is the output on terminal
> http://pastebin.com/raw.php?i=zL2fMK8c Could you suggest how I may load
> plasmoid in plasma-shell ?
>
> Regards
Q_PLUGIN_PATH and QML2_IMPORT_PATH are correct?
--
Marco Martin
From heena393 at gmail.com Thu Aug 15 16:16:08 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Thu, 15 Aug 2013 16:16:08 +0000
Subject: plasma-shell in plasma2
In-Reply-To: <201308151738.25492.notmart@gmail.com>
References:
<201308151738.25492.notmart@gmail.com>
Message-ID:
export
QT_PLUGIN_PATH=$KF5/lib/plugins:$KF5/lib64/plugins:$KF5/lib/i386-linux-gnu/plugins:$QTDIR/plugins#:$QT_PLUGIN_PATH
export
QML2_IMPORT_PATH=$KF5/lib/qml:$KF5/lib64/qml:$KF5/lib/i386-linux-gnu/qml:$QTDIR/qml
If I do echo $KF5/lib/plugins then it is giving /opt/kf5/lib/plugins
How can I be sure that the paths are correct ?
On Thu, Aug 15, 2013 at 3:38 PM, Marco Martin wrote:
> On Thursday 15 August 2013, Heena Mahour wrote:
> > Hey
> >
> > I am getting http://picpaste.com/screen13-GNKwWhEy.png this on
> > plasma-shell. This is the output on terminal
> > http://pastebin.com/raw.php?i=zL2fMK8c Could you suggest how I may load
> > plasmoid in plasma-shell ?
> >
> > Regards
> Q_PLUGIN_PATH and QML2_IMPORT_PATH are correct?
>
> --
> Marco Martin
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Thu Aug 15 16:40:46 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Thu, 15 Aug 2013 16:40:46 +0000
Subject: plasma-shell in plasma2
In-Reply-To:
References:
<201308151738.25492.notmart@gmail.com>
Message-ID:
okayy on deleting ~/.config5/plasma-shellrc I got it .
On Thu, Aug 15, 2013 at 4:16 PM, Heena Mahour wrote:
> export
> QT_PLUGIN_PATH=$KF5/lib/plugins:$KF5/lib64/plugins:$KF5/lib/i386-linux-gnu/plugins:$QTDIR/plugins#:$QT_PLUGIN_PATH
> export
> QML2_IMPORT_PATH=$KF5/lib/qml:$KF5/lib64/qml:$KF5/lib/i386-linux-gnu/qml:$QTDIR/qml
> If I do echo $KF5/lib/plugins then it is giving /opt/kf5/lib/plugins
> How can I be sure that the paths are correct ?
>
>
>
> On Thu, Aug 15, 2013 at 3:38 PM, Marco Martin wrote:
>
>> On Thursday 15 August 2013, Heena Mahour wrote:
>> > Hey
>> >
>> > I am getting http://picpaste.com/screen13-GNKwWhEy.png this on
>> > plasma-shell. This is the output on terminal
>> > http://pastebin.com/raw.php?i=zL2fMK8c Could you suggest how I may load
>> > plasmoid in plasma-shell ?
>> >
>> > Regards
>> Q_PLUGIN_PATH and QML2_IMPORT_PATH are correct?
>>
>> --
>> Marco Martin
>> _______________________________________________
>> Plasma-devel mailing list
>> Plasma-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/plasma-devel
>>
>
>
>
> --
> -Heena
> Season of kde'12 participant
> Google Summer of Code 2013
> Delhi College of Engineering(COE),India
> http://about.me/heena.mahour
> http://heenamahour.blogspot.in
>
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From heena393 at gmail.com Thu Aug 15 17:18:38 2013
From: heena393 at gmail.com (Heena Mahour)
Date: Thu, 15 Aug 2013 17:18:38 +0000
Subject: plasma-shell in plasma2
In-Reply-To:
References:
<201308151738.25492.notmart@gmail.com>
Message-ID:
However , the plasmoids appear on the desktop (plasma 2 ) momentary then it
shows an output plasma-shell(31515)/default onTriggered: Saved coordinates
for ToolBox in config: 992x992
plasma-shell(31515)/default qt_assert: ASSERT: "oldActiveFocusItem" in file
items/qquickwindow.cpp, line 735
Aborted (core dumped)
on terminal and terminates ..
so unable to test it out .
On Thu, Aug 15, 2013 at 4:40 PM, Heena Mahour wrote:
> okayy on deleting ~/.config5/plasma-shellrc I got it .
>
>
> On Thu, Aug 15, 2013 at 4:16 PM, Heena Mahour wrote:
>
>> export
>> QT_PLUGIN_PATH=$KF5/lib/plugins:$KF5/lib64/plugins:$KF5/lib/i386-linux-gnu/plugins:$QTDIR/plugins#:$QT_PLUGIN_PATH
>> export
>> QML2_IMPORT_PATH=$KF5/lib/qml:$KF5/lib64/qml:$KF5/lib/i386-linux-gnu/qml:$QTDIR/qml
>> If I do echo $KF5/lib/plugins then it is giving /opt/kf5/lib/plugins
>> How can I be sure that the paths are correct ?
>>
>>
>>
>> On Thu, Aug 15, 2013 at 3:38 PM, Marco Martin wrote:
>>
>>> On Thursday 15 August 2013, Heena Mahour wrote:
>>> > Hey
>>> >
>>> > I am getting http://picpaste.com/screen13-GNKwWhEy.png this on
>>> > plasma-shell. This is the output on terminal
>>> > http://pastebin.com/raw.php?i=zL2fMK8c Could you suggest how I may
>>> load
>>> > plasmoid in plasma-shell ?
>>> >
>>> > Regards
>>> Q_PLUGIN_PATH and QML2_IMPORT_PATH are correct?
>>>
>>> --
>>> Marco Martin
>>> _______________________________________________
>>> Plasma-devel mailing list
>>> Plasma-devel at kde.org
>>> https://mail.kde.org/mailman/listinfo/plasma-devel
>>>
>>
>>
>>
>> --
>> -Heena
>> Season of kde'12 participant
>> Google Summer of Code 2013
>> Delhi College of Engineering(COE),India
>> http://about.me/heena.mahour
>> http://heenamahour.blogspot.in
>>
>>
>
>
> --
> -Heena
> Season of kde'12 participant
> Google Summer of Code 2013
> Delhi College of Engineering(COE),India
> http://about.me/heena.mahour
> http://heenamahour.blogspot.in
>
>
--
-Heena
Season of kde'12 participant
Google Summer of Code 2013
Delhi College of Engineering(COE),India
http://about.me/heena.mahour
http://heenamahour.blogspot.in
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From shantanu at kde.org Thu Aug 15 17:49:12 2013
From: shantanu at kde.org (Shantanu Tushar)
Date: Thu, 15 Aug 2013 17:49:12 -0000
Subject: Review Request 112105: [1.1] PMC fix for KDE 4.11 - prevent deletion
of popped pages from PageStack
Message-ID: <20130815174912.11615.76044@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112105/
-----------------------------------------------------------
Review request for Plasma.
Description
-------
In KDE 4.11, if you do not parent a page after popping it from PageStack, it will be deleted by QML's garbage collector. This creates erratic behavior such as stopping playback the moment the user presses back from media player.
This patch reparents the page to the root element (and hides it) to prevent the deletion.
Diffs
-----
shells/newshell/package/contents/ui/mediacenter.qml 28ae8e6
Diff: http://git.reviewboard.kde.org/r/112105/diff/
Testing
-------
Doesn't seem to affect any workflow. Things to look out are weird placements of the media elements (browser, player etc) due to the reparenting while going back/into some mode.
Thanks,
Shantanu Tushar
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Thu Aug 15 18:14:09 2013
From: notmart at gmail.com (Marco Martin)
Date: Thu, 15 Aug 2013 18:14:09 -0000
Subject: Review Request 112105: [1.1] PMC fix for KDE 4.11 - prevent
deletion of popped pages from PageStack
In-Reply-To: <20130815174912.11615.76044@vidsolbach.de>
References: <20130815174912.11615.76044@vidsolbach.de>
Message-ID: <20130815181409.16545.51251@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112105/#review37870
-----------------------------------------------------------
Ship it!
meh, i changed it because some reported leaks since the expected way is that the stack owns and deletes the page apparently (the other implementations of pagestack, like qtcontrols, ubuntu and meego do this, so is also a porting issue)
so yeah, will have to use this workaround... (if there isn't a better idea)
- Marco Martin
On Aug. 15, 2013, 5:49 p.m., Shantanu Tushar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112105/
> -----------------------------------------------------------
>
> (Updated Aug. 15, 2013, 5:49 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> In KDE 4.11, if you do not parent a page after popping it from PageStack, it will be deleted by QML's garbage collector. This creates erratic behavior such as stopping playback the moment the user presses back from media player.
> This patch reparents the page to the root element (and hides it) to prevent the deletion.
>
>
> Diffs
> -----
>
> shells/newshell/package/contents/ui/mediacenter.qml 28ae8e6
>
> Diff: http://git.reviewboard.kde.org/r/112105/diff/
>
>
> Testing
> -------
>
> Doesn't seem to affect any workflow. Things to look out are weird placements of the media elements (browser, player etc) due to the reparenting while going back/into some mode.
>
>
> Thanks,
>
> Shantanu Tushar
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From kde at privat.broulik.de Thu Aug 15 18:33:04 2013
From: kde at privat.broulik.de (Kai Uwe Broulik)
Date: Thu, 15 Aug 2013 20:33:04 +0200
Subject: Actual width of checkbox element
In-Reply-To: <201308151713.19164.notmart@gmail.com>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
<201308151713.19164.notmart@gmail.com>
Message-ID: <2133119.ov1jHK6aj1@kaiuwe-tragbar2>
Hi,
> its implicitWidth should return a proper value (if it doesn't means the
> component should be fixed ;)
implicitWidth seems to return 0 for the checkbox item.
(If I could've just kept the switch ? :P)
From ksinny at gmail.com Thu Aug 15 18:50:05 2013
From: ksinny at gmail.com (Sinny Kumari)
Date: Thu, 15 Aug 2013 18:50:05 -0000
Subject: Review Request 112105: [1.1] PMC fix for KDE 4.11 - prevent
deletion of popped pages from PageStack
In-Reply-To: <20130815174912.11615.76044@vidsolbach.de>
References: <20130815174912.11615.76044@vidsolbach.de>
Message-ID: <20130815185005.18456.66049@vidsolbach.de>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112105/#review37872
-----------------------------------------------------------
Ship it!
Thanks for noticing this issue on KDE 4.11. Works fine on KDE 4.10 too.
- Sinny Kumari
On Aug. 15, 2013, 5:49 p.m., Shantanu Tushar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112105/
> -----------------------------------------------------------
>
> (Updated Aug. 15, 2013, 5:49 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> In KDE 4.11, if you do not parent a page after popping it from PageStack, it will be deleted by QML's garbage collector. This creates erratic behavior such as stopping playback the moment the user presses back from media player.
> This patch reparents the page to the root element (and hides it) to prevent the deletion.
>
>
> Diffs
> -----
>
> shells/newshell/package/contents/ui/mediacenter.qml 28ae8e6
>
> Diff: http://git.reviewboard.kde.org/r/112105/diff/
>
>
> Testing
> -------
>
> Doesn't seem to affect any workflow. Things to look out are weird placements of the media elements (browser, player etc) due to the reparenting while going back/into some mode.
>
>
> Thanks,
>
> Shantanu Tushar
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
From notmart at gmail.com Thu Aug 15 19:52:03 2013
From: notmart at gmail.com (Marco Martin)
Date: Thu, 15 Aug 2013 21:52:03 +0200
Subject: Actual width of checkbox element
In-Reply-To: <2133119.ov1jHK6aj1@kaiuwe-tragbar2>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
<201308151713.19164.notmart@gmail.com> <2133119.ov1jHK6aj1@kaiuwe-tragbar2>
Message-ID: <201308152152.04296.notmart@gmail.com>
On Thursday 15 August 2013, Kai Uwe Broulik wrote:
> Hi,
>
> > its implicitWidth should return a proper value (if it doesn't means the
> > component should be fixed ;)
>
> implicitWidth seems to return 0 for the checkbox item.
then needs to be fixed...
patches welcome ;)
--
Marco Martin
From kde at privat.broulik.de Thu Aug 15 20:02:29 2013
From: kde at privat.broulik.de (Kai Uwe Broulik)
Date: Thu, 15 Aug 2013 22:02:29 +0200
Subject: Actual width of checkbox element
In-Reply-To: <201308152152.04296.notmart@gmail.com>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
<2133119.ov1jHK6aj1@kaiuwe-tragbar2> <201308152152.04296.notmart@gmail.com>
Message-ID: <18866242.x3D0qdbTJY@kaiuwe-tragbar2>
> patches welcome ;)
Hmm, when I try to manually set the implicitWidth or route it somewhereelse it
says "implicitWidth not available in QtQuick Item 1.0" (or so).
From notmart at gmail.com Thu Aug 15 20:04:54 2013
From: notmart at gmail.com (Marco Martin)
Date: Thu, 15 Aug 2013 22:04:54 +0200
Subject: Actual width of checkbox element
In-Reply-To: <18866242.x3D0qdbTJY@kaiuwe-tragbar2>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
<201308152152.04296.notmart@gmail.com> <18866242.x3D0qdbTJY@kaiuwe-tragbar2>
Message-ID: <201308152204.55238.notmart@gmail.com>
On Thursday 15 August 2013, Kai Uwe Broulik wrote:
> > patches welcome ;)
>
> Hmm, when I try to manually set the implicitWidth or route it somewhereelse
> it says "implicitWidth not available in QtQuick Item 1.0" (or so).
implicitWidth is available only since 1.1, so across the component the import
QtQuick 1.0 have to be changed in import QtQuick 1.1
--
Marco Martin
From kde at privat.broulik.de Thu Aug 15 20:29:36 2013
From: kde at privat.broulik.de (Kai Uwe Broulik)
Date: Thu, 15 Aug 2013 22:29:36 +0200
Subject: Actual width of checkbox element
In-Reply-To: <201308152204.55238.notmart@gmail.com>
References: <2372011.FJYrJkaRds@kaiuwe-tragbar2>
<18866242.x3D0qdbTJY@kaiuwe-tragbar2> <201308152204.55238.notmart@gmail.com>
Message-ID: <6857499.NQgcFn7EEn@kaiuwe-tragbar2>
Am Donnerstag, 15. August 2013, 22:04:54 schrieb Marco Martin:
> On Thursday 15 August 2013, Kai Uwe Broulik wrote:
> > > patches welcome ;)
> >
> > Hmm, when I try to manually set the implicitWidth or route it
> > somewhereelse
> > it says "implicitWidth not available in QtQuick Item 1.0" (or so).
>
> implicitWidth is available only since 1.1, so across the component the
> import QtQuick 1.0 have to be changed in import QtQuick 1.1
Oh, it was my fault :)
I tried to use the PowermanagementItem implicitWidth but the PMitem is not the
checkbox itself, so I had to set its implicitWidth to use the checkbox
implicitWidth and now all works without touching the plasma components \o/
From aacid at kde.org Thu Aug 15 22:03:45 2013
From: aacid at kde.org (Albert Astals Cid)
Date: Thu, 15 Aug 2013 22:03:45 -0000
Subject: Review Request 108325: LastNotificationPopup fix for closing not
to move + add move button + better layout for 3 action buttons
In-Reply-To: <20130405161633.27215.86702@vidsolbach.de>
References: <20130405161633.27215.86702@vidsolbach.de>
Message-ID: <20130815220345.17501.54172@vidsolbach.de>
> On April 5, 2013, 4:16 p.m., Albert Astals Cid wrote:
> > Can you please attach the diff properly and not give ship its to yourself?
> >
> > I'd suggest you discard this review that is already broken and start a new one where you attach the diff properly and you don't give you a ship it to yourself and then maybe others will have a look at it.
>
> Leszek Lesner wrote:
> If I would know hot to attach the diff properly I would have done it. It is just not recognizing the patch.
>
> Albert Astals Cid wrote:
> What error are you getting? How are you creating the patch?
>
> Leszek Lesner wrote:
> I got the error that this is not a patch or that the specified diff file is empty.
> I created the patch with dpkg-source --commit and removed the description lines above.
>
> Albert Astals Cid wrote:
> dpkg-source --commit seems like a bad way to create a git compatible diff.
>
> Just do "git diff" on the git repo.
>
> Leszek Lesner wrote:
> Hmm... I did not have any git repo there. I just downloaded the sources directly. Maybe that was the whole mistake.
> But it is some month already ago that I created that patch and it should be mostly outdated now that we have 4.10.2 out.
> All in all I am a little bit unhappy that I posted this patch on the bug tracker and no one really cared about it and then posting it here only on someones advice and it was still not reviewed or integrated.
> And I can't imagine that it is all my fault.
>
>
> Marco Martin wrote:
> you should download it with git, and send the patch against master branch, see http://techbase.kde.org/Getting_Started and http://www.youtube.com/watch?v=cqnNVmJocR4
I'm discarding this review, you gave a ship it to yourself and that was wrong. I gave you the pointers on how to do it right, i'm sorry if you find it hard.
- Albert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108325/#review30476
-----------------------------------------------------------
On Jan. 10, 2013, 3:18 p.m., Leszek Lesner wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108325/
> -----------------------------------------------------------
>
> (Updated Jan. 10, 2013, 3:18 p.m.)
>
>
> Review request for Plasma and Marco Martin.
>
>
> Description
> -------
>
> Fixes the LastNotificationsPopup moving around when clicking on a button and adds a move button to freely place it all over the desktop.
> Also this removes notifications jobs from the notification icon. (So clicking on X does not hide but really closes the notification and removes it)
> Also this fixes layout problems with 3 buttons displayed inside a notification (this caused the X button to overlap with the upper actionbutton).
>
> As my original diff was not recognized as normal diff(don't know what I did wrong there) I attached the patch directly to this thread.
>
>
> This addresses bug 311413.
> http://bugs.kde.org/show_bug.cgi?id=311413
>
>
> Diffs
> -----
>
>
> Diff: http://git.reviewboard.kde.org/r/108325/diff/
>
>
> Testing
> -------
>
> kdialog --passivepopup "bla"
> click on X, closes and removes job from notification Icon
> click and hold on move button moves the popup freely on the screen.
>
>
> File Attachments
> ----------------
>
> Patch
> http://git.reviewboard.kde.org/media/uploaded/files/2013/01/10/LastNotificationPopup_patch.diff
>
>
> Thanks,
>
> Leszek Lesner
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: