Re: Review Request 122679: Show Desktop feature: refurbished

2015-04-07 Thread Thomas Lübking
On März 30, 2015, 6:02 vorm., Martin Gräßlin wrote: effects/CMakeLists.txt, line 133 https://git.reviewboard.kde.org/r/122679/diff/2/?file=357029#file357029line133 the name looks a little bit mhhh ;-) Thomas Lübking wrote: When thinking about a name that does not sound like

Re: PIM Sprint this Weekend

2015-04-07 Thread Martin Gräßlin
On Tuesday 07 April 2015 10:48:07 Vishesh Handa wrote: Hey guys I'm travelling to Toulouse for the PIM Sprint this weekend. If there is anything specific anyone would like to me to discuss w.r.t Plasma, please let me know. please push in a direction that we get events in the calendar back.

Re: Plasma 5.3 changelog

2015-04-07 Thread Martin Gräßlin
On Monday 06 April 2015 21:07:17 Eike Hein wrote: Hiyo, we're freezing this week, so it's time to gather the changes for the promo folks again. I'll start with the noteworthy stuff in my components: ScreenLocker: * XInput2 devices are grabbed: increases security for setups with multiple

Re: Review Request 123275: Image Wallpaper: Restore index after removing a wallpaper

2015-04-07 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123275/#review78598 --- wallpapers/image/imagepackage/contents/ui/config.qml (line

PIM Sprint this Weekend

2015-04-07 Thread Vishesh Handa
Hey guys I'm travelling to Toulouse for the PIM Sprint this weekend. If there is anything specific anyone would like to me to discuss w.r.t Plasma, please let me know. -- Vishesh Handa ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Review Request 123276: Plasma-framework: Fix coding style, width should be renamed to height.

2015-04-07 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123276/#review78597 --- Ship it! Ship It! - Marco Martin On April 6, 2015, 2:07

Re: Plasma 5.3 changelog

2015-04-07 Thread Marco Martin
On Monday 06 April 2015, Eike Hein wrote: Hiyo, we're freezing this week, so it's time to gather the changes for the promo folks again. I'll start with the noteworthy stuff in my components: stuff i touched announcement-worty: kdeplasma-addons: * comic applet ported to plasma 5

[Powerdevil] [Bug 345940] New: plasmoid looks a bit unorganised

2015-04-07 Thread Michi
https://bugs.kde.org/show_bug.cgi?id=345940 Bug ID: 345940 Summary: plasmoid looks a bit unorganised Product: Powerdevil Version: unspecified Platform: Other OS: Linux Status: UNCONFIRMED Severity:

[Powerdevil] [Bug 345940] plasmoid looks a bit unorganised

2015-04-07 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=345940 David Edmundson k...@davidedmundson.co.uk changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123271/#review78604 --- What when I place it on the desktop? I'd like it to fill the

Re: PIM Sprint this Weekend

2015-04-07 Thread Mark Gaiser
On Tue, Apr 7, 2015 at 10:51 AM, Martin Gräßlin mgraess...@kde.org wrote: On Tuesday 07 April 2015 10:48:07 Vishesh Handa wrote: Hey guys I'm travelling to Toulouse for the PIM Sprint this weekend. If there is anything specific anyone would like to me to discuss w.r.t Plasma, please

Re: PIM Sprint this Weekend

2015-04-07 Thread Martin Klapetek
On Tue, Apr 7, 2015 at 12:35 PM, Mark Gaiser mark...@gmail.com wrote: From a technical point of view the events can all be fetched in the QML Calendar Components (which i made together with some pim folks). That part is there (read only events that is) and is just waiting till pimlibs are

Review Request 123281: Avoid triggering bug 340691

2015-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123281/ --- Review request for Plasma, David Edmundson, Eike Hein, and Marco Martin.

Re: PIM Sprint this Weekend

2015-04-07 Thread Vishesh Handa
On Tue, Apr 7, 2015 at 10:51 AM, Martin Gräßlin mgraess...@kde.org wrote: (boud's blog post is a nice reminder that libraries should think about their users) His blog is one big rant. I love it. He's completely right. -- Vishesh Handa ___

Minutes Monday Plasma Hangout

2015-04-07 Thread Marco Martin
Hangout of 7 April 2015 Antonis: * undo feature for wallpaper removal * bugfixing wallpaper configuration * release of plasmate repo (will be plasma-sdk) * will tag master and make a branch out of it, then merge frameworks into master David: * fixing a bug in xddm (problem in gnome-keyring) *

Re: PIM Sprint this Weekend

2015-04-07 Thread Martin Steigerwald
Am Dienstag, 7. April 2015, 12:15:05 schrieb Vishesh Handa: On Tue, Apr 7, 2015 at 10:51 AM, Martin Gräßlin mgraess...@kde.org wrote: (boud's blog post is a nice reminder that libraries should think about their users) His blog is one big rant. I love it. He's completely right. I

[Powerdevil] [Bug 345940] Move Enable Power Management checkbox to the top as other applets

2015-04-07 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=345940 Martin Klapetek mklape...@kde.org changed: What|Removed |Added Resolution|INVALID |--- Ever confirmed|0

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Martin Klapetek
On April 7, 2015, 11:43 a.m., Kai Uwe Broulik wrote: What when I place it on the desktop? I'd like it to fill the entire room there. If I want a massive Kickoff button on my desktop, that's up to me :) +1; it should really be iff vertical == true - Martin

Re: Review Request 123281: Avoid triggering bug 340691

2015-04-07 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123281/#review78614 --- Ship it! Ship It! - Marco Martin On April 7, 2015, 11:13

Re: Plasma 5.3 changelog

2015-04-07 Thread David Edmundson
* better high dpi support in various places (could be worth screenshots about that in the announcement or better waiting applications like dolphin are finally released?) I think we should save the high DPI stuff till 5.4 when it's user facing.

Re: PIM Sprint this Weekend

2015-04-07 Thread Aleix Pol
On Tue, Apr 7, 2015 at 10:48 AM, Vishesh Handa m...@vhanda.in wrote: Hey guys I'm travelling to Toulouse for the PIM Sprint this weekend. If there is anything specific anyone would like to me to discuss w.r.t Plasma, please let me know. Supporting libaccounts (so kaccounts, former

[Powerdevil] [Bug 345940] Move Enable Power Management checkbox to the top as other applets

2015-04-07 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=345940 Kai Uwe Broulik k...@privat.broulik.de changed: What|Removed |Added CC||k...@privat.broulik.de

Re: Review Request 123281: Avoid triggering bug 340691

2015-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123281/ --- (Updated April 7, 2015, 11:46 a.m.) Status -- This change has been

Re: Plasma 5.3 changelog

2015-04-07 Thread Bhushan Shah
On Tue, Apr 7, 2015 at 12:37 AM, Eike Hein h...@kde.org wrote: Hiyo, we're freezing this week, so it's time to gather the changes for the promo folks again. I'll start with the noteworthy stuff in my components: Plasma 5.3 will have technology preview of plasma-mediacenter, Promo people may

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Marco Martin
On April 7, 2015, 12:59 p.m., Sebastian Kügler wrote: So if the panel is horizontal, and high, people get a huge icon. If it's vertical and wide, the icon is limited and centered? That's pretty inconsistent. I agree that a huge button may not be what the user wants, but then what's

Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123283/ --- Review request for Plasma and Lukáš Tinkl. Repository: plasma-workspace

Re: Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread Matthias Klumpp
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123283/#review78626 --- :D That's an awesome feature. I guess there's an interesting

Re: Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123283/#review78625 --- Ship it! Ship It! - Martin Gräßlin On April 7, 2015, 4:25

Review Request 123282: The energy info KCM

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123282/ --- Review request for Plasma. Repository: kinfocenter Description ---

Re: Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread Matthias Klumpp
On April 7, 2015, 2:31 nachm., Matthias Klumpp wrote: :D That's an awesome feature. I guess there's an interesting story behind this, when it was implemented (for debugging?). Just to be clear, that's no objection to the patch - please kill this thing ;-) - Matthias

Re: Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123283/#review78627 --- Ship it! Remember to report it in the changelog. ;) - Aleix

Re: Review Request 123282: The energy info KCM

2015-04-07 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123282/#review78620 --- Ship it! cool, modulo just few small issues CMakeLists.txt

Re: Review Request 123282: The energy info KCM

2015-04-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123282/#review78621 --- CMakeLists.txt (line 34)

Re: PIM Sprint this Weekend

2015-04-07 Thread Vishesh Handa
On Tue, Apr 7, 2015 at 12:44 PM, Martin Steigerwald mar...@lichtvoll.de wrote: I wonder about the mail indexing and search. Its still not available in Plasma 5, isn´t it? (Still on 4.14.6/7 and Akonadi 1.13). I'm assuming you're talking about KRunner? We can search through the email index

Re: Review Request 123275: Image Wallpaper: Restore index after removing a wallpaper

2015-04-07 Thread Antonis Tsiapaliokas
On April 7, 2015, 8:31 a.m., Marco Martin wrote: wallpapers/image/imagepackage/contents/ui/config.qml, line 296 https://git.reviewboard.kde.org/r/123275/diff/1/?file=360286#file360286line296 hmm, not understanding exactly why the index has to be decremented by count.

Re: Review Request 123276: Plasma-framework: Fix coding style, width should be renamed to height.

2015-04-07 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123276/ --- (Updated April 7, 2015, 2:49 p.m.) Status -- This change has been

Re: Review Request 123282: The energy info KCM

2015-04-07 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123282/#review78622 --- Modules/energy/kcm.cpp (line 65)

Re: Plasma 5.3 changelog

2015-04-07 Thread David Rosca
On Mon, Apr 6, 2015 at 9:07 PM, Eike Hein h...@kde.org wrote: Hiyo, we're freezing this week, so it's time to gather the changes for the promo folks again. I'll start with the noteworthy stuff in my components: User-visible changes in Bluedevil: * Bluedevil was ported to BluezQt. This

Review Request 123284: Create breeze shadows unconditionally

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123284/ --- Review request for Plasma and Hugo Pereira Da Costa. Repository: breeze

Re: Review Request 123284: Create breeze shadows unconditionally

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123284/ --- (Updated April 7, 2015, 3:46 p.m.) Status -- This change has been

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread David Edmundson
On April 7, 2015, 3:28 p.m., Kai Uwe Broulik wrote: applets/timer/package/contents/ui/configAppearance.qml, line 43 https://git.reviewboard.kde.org/r/123274/diff/4/?file=360398#file360398line43 HIG says Checkboxes should always be positive, ie. Show seconds and then enable it by

Re: Review Request 123283: Remove interesting ksmserver easter egg

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123283/ --- (Updated April 7, 2015, 4:56 p.m.) Status -- This change has been

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/ --- (Updated April 7, 2015, 3:19 nachm.) Review request for Plasma.

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
On April 7, 2015, 3:28 nachm., Kai Uwe Broulik wrote: applets/timer/package/contents/ui/configAppearance.qml, line 29 https://git.reviewboard.kde.org/r/123274/diff/4/?file=360398#file360398line29 What is that invisible (flat and no title) GroupBox for? copycat is to be blamed (took

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
On April 7, 2015, 3:28 nachm., Kai Uwe Broulik wrote: applets/timer/package/contents/ui/configAppearance.qml, line 43 https://git.reviewboard.kde.org/r/123274/diff/4/?file=360398#file360398line43 HIG says Checkboxes should always be positive, ie. Show seconds and then enable it

Re: Review Request 123284: Create breeze shadows unconditionally

2015-04-07 Thread David Edmundson
On April 7, 2015, 3:38 p.m., Hugo Pereira Da Costa wrote: Sounds good. Bye bye fancy Xcb code. Same goes for oxygen I guess :) well, you still have xcb code just below to actaully upload the shadows. Now you have that awkward situation where you say goodbye to someone and then have to

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/ --- (Updated April 7, 2015, 3:16 nachm.) Review request for Plasma.

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/#review78631 --- applets/timer/package/contents/ui/configAppearance.qml (line

Re: Review Request 123282: The energy info KCM

2015-04-07 Thread Kai Uwe Broulik
On April 7, 2015, 1:50 nachm., Lukáš Tinkl wrote: Modules/energy/package/contents/ui/main.qml, line 97 https://git.reviewboard.kde.org/r/123282/diff/1/?file=360362#file360362line97 What about multiple batteries? That's just so it initially shows the first battery. When you have

Re: Review Request 121208: Port the KConfigXtEditor to KDevPlatform

2015-04-07 Thread Giorgos Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121208/ --- (Updated April 7, 2015, 7:08 p.m.) Review request for Plasma.

Re: Review Request 121208: Port the KConfigXtEditor to KDevPlatform

2015-04-07 Thread Giorgos Tsiapaliokas
On March 25, 2015, 1:59 a.m., Aleix Pol Gonzalez wrote: Is it all in now? Maybe an updated screenshot would help? Anyway, I trust your judgement if you want to ship this. The KConfigXt doesn't exist anymore in the toolviews. I have added an image in which I do 1. Click in the

Re: Review Request 123282: The energy info KCM

2015-04-07 Thread Kai Uwe Broulik
On April 7, 2015, 1:45 nachm., Aleix Pol Gonzalez wrote: kcmcontainer.cpp, line 62 https://git.reviewboard.kde.org/r/123282/diff/1/?file=360371#file360371line62 unrelated? Ah yeah, that is to make the headings in KInfoCenter match the new KTitleWidget style, should go in

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/#review78632 --- A bunch of pedantic comments below

Re: Review Request 123284: Create breeze shadows unconditionally

2015-04-07 Thread Hugo Pereira Da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123284/#review78634 --- Ship it! Sounds good. Bye bye fancy Xcb code. Same goes for

Re: Review Request 123289: [kde4breze] Fix some KConfig usage (wrt bug 340691)

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123289/#review78645 --- Ship it! Ship It! - David Edmundson On April 7, 2015,

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread David Edmundson
On April 7, 2015, 3:28 p.m., Kai Uwe Broulik wrote: applets/timer/package/contents/ui/configAppearance.qml, line 43 https://git.reviewboard.kde.org/r/123274/diff/4/?file=360398#file360398line43 HIG says Checkboxes should always be positive, ie. Show seconds and then enable it by

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
On April 7, 2015, 3:28 nachm., Kai Uwe Broulik wrote: applets/timer/package/contents/ui/configAppearance.qml, line 43 https://git.reviewboard.kde.org/r/123274/diff/4/?file=360398#file360398line43 HIG says Checkboxes should always be positive, ie. Show seconds and then enable it

Re: Review Request 123288: Fix some more KConfig usage (wrt bug 340691)

2015-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123288/ --- (Updated April 7, 2015, 6:50 p.m.) Status -- This change has been

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/ --- (Updated April 7, 2015, 5:20 nachm.) Review request for Plasma.

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/#review78643 --- Ship it! Ship It! - David Edmundson On April 7, 2015,

Review Request 123289: [kde4breze] Fix some KConfig usage (wrt bug 340691)

2015-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123289/ --- Review request for Plasma, David Edmundson and Marco Martin. Bugs:

Re: Plasma 5.3 changelog

2015-04-07 Thread Kai Uwe Broulik
Hi, a lot of fancy new stuff in the power management department, most notably: - Activity config returned, allowing to configure power management settings differently for certain activities - Laptop will not suspend when closing the lid while an external monitor is connected (cinema mode, by

Re: Review Request 123288: Fix some more KConfig usage (wrt bug 340691)

2015-04-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123288/#review78644 --- Ship it! Ship It! - David Edmundson On April 7, 2015,

[plasmashell] [Bug 345940] Move Enable Power Management checkbox to the top as other applets

2015-04-07 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=345940 Kai Uwe Broulik k...@privat.broulik.de changed: What|Removed |Added Product|Powerdevil |plasmashell

Re: Review Request 123289: [kde4breze] Fix some KConfig usage (wrt bug 340691)

2015-04-07 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123289/ --- (Updated April 7, 2015, 7:16 p.m.) Status -- This change has been

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Mathias Kraus
On April 7, 2015, 9:43 vorm., Kai Uwe Broulik wrote: applets/kickoff/package/contents/ui/CompactRepresentation.qml, line 28 https://git.reviewboard.kde.org/r/123271/diff/1/?file=360272#file360272line28 readonly property The user could change the edge and then the property would

Re: Review Request 122679: Show Desktop feature: refurbished

2015-04-07 Thread Thomas Lübking
On März 30, 2015, 6:02 vorm., Martin Gräßlin wrote: effects/CMakeLists.txt, line 133 https://git.reviewboard.kde.org/r/122679/diff/2/?file=357029#file357029line133 the name looks a little bit mhhh ;-) Thomas Lübking wrote: When thinking about a name that does not sound like

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Mathias Kraus
On April 7, 2015, 9:43 vorm., Kai Uwe Broulik wrote: What when I place it on the desktop? I'd like it to fill the entire room there. If I want a massive Kickoff button on my desktop, that's up to me :) Martin Klapetek wrote: +1; it should really be iff vertical == true This patch

Re: Review Request 123274: Timer Plasmoid: Added config ui for hiding/showing seconds and title, and cmd execution

2015-04-07 Thread Bernhard Friedreich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/ --- (Updated April 7, 2015, 9:08 p.m.) Status -- This change has been

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Mathias Kraus
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123271/ --- (Updated April 7, 2015, 9:56 nachm.) Review request for Plasma.

Re: PIM Sprint this Weekend

2015-04-07 Thread Vishesh Handa
On Wed, Apr 8, 2015 at 12:15 AM, Martin Steigerwald mar...@lichtvoll.de wrote: Wow, okay. I thought you removed mail indexing from Baloo as Akonadi developers wanted to integrate it with Akonadi directly and I thought that work wasn´t yet completed. I mean mail search over Alt-F2, but also

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Mathias Kraus
On April 7, 2015, 12:59 nachm., Sebastian Kügler wrote: So if the panel is horizontal, and high, people get a huge icon. If it's vertical and wide, the icon is limited and centered? That's pretty inconsistent. I agree that a huge button may not be what the user wants, but then

Re: PIM Sprint this Weekend

2015-04-07 Thread Martin Steigerwald
Am Dienstag, 7. April 2015, 16:37:45 schrieb Vishesh Handa: On Tue, Apr 7, 2015 at 12:44 PM, Martin Steigerwald mar...@lichtvoll.de wrote: I wonder about the mail indexing and search. Its still not available in Plasma 5, isn´t it? (Still on 4.14.6/7 and Akonadi 1.13). I'm assuming

Re: Review Request 122679: Show Desktop feature: refurbished

2015-04-07 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122679/ --- (Updated April 7, 2015, 10:30 p.m.) Status -- This change has been

Re: Review Request 123288: Fix some more KConfig usage (wrt bug 340691)

2015-04-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123288/#review78653 --- Please make sure to use the bug field in reviewboard and

Re: Review Request 123290: Port task manager away from mouse event listener

2015-04-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123290/#review78654 --- Bug 339482 - Kai Uwe Broulik On April 8, 2015, 4:04 vorm.,

Re: Review Request 123271: improve kickoff representation on vertical panels

2015-04-07 Thread Kai Uwe Broulik
On April 7, 2015, 9:43 vorm., Kai Uwe Broulik wrote: What when I place it on the desktop? I'd like it to fill the entire room there. If I want a massive Kickoff button on my desktop, that's up to me :) Martin Klapetek wrote: +1; it should really be iff vertical == true Mathias

Review Request 123290: Port task manager away from mouse event listener

2015-04-07 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123290/ --- Review request for Plasma and Eike Hein. Repository: plasma-desktop

Re: Review Request 123290: Port task manager away from mouse event listener

2015-04-07 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123290/ --- (Updated April 8, 2015, 5:57 a.m.) Review request for Plasma and Eike