Re: Review Request 126013: fix: kacess should not be built if xkb is not present

> On Nov. 10, 2015, 1:26 a.m., David Edmundson wrote: > > Ship It! > > Johan Ouwerkerk wrote: > Forgot to mention that the patch can be pulled from the fix-cmake-xkb-dep > branch at: g...@github.com:cmacq2/plasma-desktop.git > > Johan Ouwerkerk wrote: > Please note: I do not have

Re: Review Request 126083: [hwcomposer] Don't waitVsync if we haven't activated vsync yet

> On Nov. 16, 2015, 12:49 p.m., Bhushan Shah wrote: > > It includes another random change? > > Marco Martin wrote: > if (!m_hasVsync) ? no, these are two commits. The first one is the change in waitVSync, the other one the backlight handling - Martin

Re: Review Request 125951: [calendar] Move the plugins handling to a separate class

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125951/ --- (Updated Nov. 16, 2015, 4:55 p.m.) Status -- This change has been

Re: Review Request 125966: Allow adding config categories dynamically

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125966/ --- (Updated Nov. 16, 2015, 4:55 p.m.) Status -- This change has been

Re: Review Request 125817: Add plugin system for Calendar events

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125817/ --- (Updated Nov. 16, 2015, 4:55 p.m.) Status -- This change has been

Re: RFC: Disable DrKonqi

On Mon, Nov 16, 2015 at 5:50 AM, Sebastian Kügler wrote: > > Perhaps only enable drkonqi for debug builds? That's the kind of backtrace > that should be useful, otherwise the bt would be useless anyway... > > I'm not so sure about all this, I do think that backtraces are valuable

Re: RFC: Disable DrKonqi

On Monday 16 November 2015, Sebastian Kügler wrote: > Perhaps only enable drkonqi for debug builds? That's the kind of backtrace > that should be useful, otherwise the bt would be useless anyway... > > I'm not so sure about all this, I do think that backtraces are valuable to > us, and I also

Re: Review Request 126083: [hwcomposer] Don't waitVsync if we haven't activated vsync yet

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/#review88416 --- +1 - Marco Martin On Nov. 16, 2015, 11:43 a.m., Martin

Re: Review Request 126061: [Menu] Popup above parent if would exceed screen boundaries

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126061/#review88414 --- Ship it! src/declarativeimports/plasmacomponents/qmenu.cpp

Re: Review Request 126018: Use the plural form for "Notification" category

> On Nov. 10, 2015, 10:57 p.m., Martin Klapetek wrote: > > > should I change also X-KDE-System-Settings-Category=notification to > > > notifications > > > > I don't think so, it would require changing everywhere. Unless there is > > nothing actually using it, in which case, yes. Where should

Re: Review Request 125966: Allow adding config categories dynamically

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125966/#review88433 --- Ship it! Ship It! - David Edmundson On Nov. 5, 2015, 8:06

Re: Minutes Monday Plasma Hangout

On Mon, Nov 16, 2015 at 5:43 PM, Sebastian Kügler wrote: > bshash: > - pushed screenlocker split to make it wayland-ready Uhoh, I meant screenlocker integrated into kwin_wayland -- Bhushan Shah http://bhush9.github.io IRC Nick : bshah on Freenode

Re: RFC: Disable DrKonqi

On Monday, November 16, 2015 12:11:09 PM CET Luca Beltrame wrote: > In data lunedì 16 novembre 2015 10:50:01 CET, Sebastian Kügler ha scritto: > > us, and I also think that it would be cheating to just hide the drkonqi > > window, but on the other hand, if it improves the user experience, that's >

Minutes Monday Plasma Hangout

Present: bshah, Sho, jensreu, kbroulik, notmart, mgraesslin, ozark, Sho, sebas, andreas_k, Riddell Date: 16 Nov, 2015 Maintainer Juggling: - Eike now maintains Kickoff - Andreas now maintains Oxygen icons - Bhushan now maintains Plasma mobile images bshash: - pushed screenlocker split to

Re: Review Request 126083: [hwcomposer] Don't waitVsync if we haven't activated vsync yet

> On Nov. 16, 2015, 11:49 a.m., Bhushan Shah wrote: > > It includes another random change? if (!m_hasVsync) ? - Marco --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/#review88413

Re: Review Request 126061: [Menu] Popup above parent if would exceed screen boundaries

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126061/#review88417 --- doesn't that just move the problem to buttons that are at the

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125991/#review88421 --- otherwise, looks good to me. I'll let others weigh in as

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125991/#review88427 --- Note to self: Disable drag area when applet is pending

Review Request 126089: Metadata and color edit for plasma theme explorer

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126089/ --- Review request for Plasma. Repository: plasma-sdk Description ---

Re: Minutes Monday Plasma Hangout

On 11/16/2015 01:13 PM, Sebastian Kügler wrote: > Sho: > - favorite migration not entirely figured out That's about the planned KActivities-backed favorites stuff (which is skipping 5.5 now); migrating faves from the old config format to the new one for the backend swap in 5.5 is taken care of

Review Request 126091: Remove printscreen.khotkeys from KHotkeys package

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126091/ --- Review request for KDE Frameworks, Plasma and David Edmundson.

Re: Review Request 126091: Remove printscreen.khotkeys from KHotkeys package

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126091/#review88438 --- >assuming, that is, distros push out the updates together

5.5 Errata

https://community.kde.org/Plasma/5.5_Errata Please check over the bugs copied over from 5.4 errata Plasma bugs marked critical are here, please check over if they should be marked as not critical or close as won't fix or fixed goo.gl/XnwSev Please add any other bugs or missing features to the

Re: Review Request 126091: Remove printscreen.khotkeys from KHotkeys package

> On Nov. 16, 2015, 10:46 p.m., Hrvoje Senjan wrote: > > >assuming, that is, distros push out the updates together > > > > don't assume this ;-) at least send a mail to kde-distro-packagers about > > the sideeffects if those 3 are pushed asynchronously 'course, I'll post there as well as add

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 p.m., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

Re: RFC: Disable DrKonqi

Just FYI, on a systemd based system, the coredump is already automatically stored in journal and can be accessed easily by coredumpctl. Personally I use drkonqi a lot to fix issues, it's more visible for me to notice the problem (as developer). I just wonder whether drkonqi provides some extra

Re: Review Request 126061: [Menu] Popup above parent if would exceed screen boundaries

> On Nov. 16, 2015, 12:12 nachm., David Edmundson wrote: > > doesn't that just move the problem to buttons that are at the top of the > > screen? Imho it's far less likely for someone to open a menu deliberately *above* a button. - Kai Uwe

Re: Review Request 126092: Konqui based Emotion pack named eKspression

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126092/ --- (Updated Nov. 16, 2015, 10:38 p.m.) Review request for Plasma. Changes

Re: [kde-promo] 5.5 announcement

On Monday, November 16, 2015 20:51:25 Jonathan Riddell wrote: > Questions: > > "The Breeze [Plasma Widgets?] have been updated to make them more consistent." > > is that the QtWidget theme or the Plasma widget theme or both? The Plasma theme (I'm not aware of sweeping changes in the QWidget

Re: [kde-promo] 5.5 announcement

Ken and Hugo do a lot of polishing for 5.5 as fare as I know. maybe you can send them an mail and ask for feedback https://forum.kde.org/viewtopic.php?f=285=127663 cheers 2015-11-16 22:53 GMT+01:00 Sebastian Kügler : > On Monday, November 16, 2015 20:51:25 Jonathan Riddell

Re: Review Request 124585: Add pretty user switcher

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124585/#review88450 --- If no one objects by tomorrow evening, I'll merge this. -

Review Request 126092: Konqui based Emotion pack named eKspression

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126092/ --- Review request for Plasma. Repository: kemoticons Description ---

Re: Review Request 126074: [Widget Explorer] Allow filtering for uninstallable widgets

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126074/ --- (Updated Nov. 16, 2015, 8:09 p.m.) Status -- This change has been

5.5 announcement

Here's a first go at the 5.5 announcement https://www.kde.org/announcements/plasma-5.4.95.php martin k should have screenshots later today Questions: "The Breeze [Plasma Widgets?] have been updated to make them more consistent." is that the QtWidget theme or the Plasma widget theme or both?

[Breeze] [Bug 355447] Mouse cursor theme partially fails on first login with new user account

https://bugs.kde.org/show_bug.cgi?id=355447 --- Comment #1 from Martin Walch --- Created attachment 95537 --> https://bugs.kde.org/attachment.cgi?id=95537=edit Screenshot of a kmines window with the wrong cursor in the lower right corner. A screenshot of kmines when

Re: Review Request 126063: [DragArea] Support specifying delegateImage as QString

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126063/ --- (Updated Nov. 16, 2015, 8:13 p.m.) Status -- This change has been

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 4:37 p.m., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 p.m., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 nachm., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

Re: Review Request 126010: [digital-clock] Split the Calendar config stuff to it's own category + load the plugins configs

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126010/ --- (Updated Nov. 16, 2015, 9:03 p.m.) Status -- This change has been

[Breeze] [Bug 355447] New: Mouse cursor theme partially fails on first login with new user account

https://bugs.kde.org/show_bug.cgi?id=355447 Bug ID: 355447 Summary: Mouse cursor theme partially fails on first login with new user account Product: Breeze Version: 5.4.3 Platform: Other OS: Linux

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 p.m., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

build.kde.org issues

Looking over build.kde.org issues before the Beta on Thursday there's 1 build failure and 3 test failures, is anyone able to take a look? https://build.kde.org/view/Plasma5/ khotkeys: I can't recreate this failure locally so it could be an issue with the CI

Re: Review Request 125990: [Widget Explorer] Fix applet uninstall

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125990/ --- (Updated Nov. 16, 2015, 8:31 p.m.) Status -- This change has been

Re: 5.5 announcement

Hi, some thoughts (need better wording obviously :)) > The Plasma Widget explorer now supports a two column view with new widget > icons for Breeze, Breeze Dark and Oxygen Also, if I can make it on time it'll have improved support for user-installed applets (better uninstall experience with

Re: Review Request 126092: Konqui based Emotion pack named eKspression

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126092/#review88452 --- Loving the icons! (not so keen on the random K name, I'd

Re: Review Request 126018: Use the plural form for "Notification" category

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126018/ --- (Updated Nov. 16, 2015, 11:51 p.m.) Status -- This change has been

Re: Review Request 126079: Keep drag and drop in range on both sides

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126079/ --- (Updated Nov. 16, 2015, 11:33 a.m.) Status -- This change has been

Review Request 126083: [hwcomposer] Don't waitVsync if we haven't activated vsync yet

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/ --- Review request for Plasma. Repository: kwin Description --- Would

Re: Review Request 126083: [hwcomposer] Don't waitVsync if we haven't activated vsync yet

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/#review88413 --- It includes another random change? - Bhushan Shah On Nov.

Re: Review Request 126092: Konqui based Emotion pack named eKspression

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126092/#review88453 --- The name is a bit strange (I think "Konqi" would work just

Jenkins-kde-ci: khotkeys master kf5-qt5 » Linux,gcc - Build # 22 - Fixed!

GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/khotkeys%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/22/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 17 Nov 2015 05:50:41 + Build duration: 5 min 45 sec CHANGE SET No changes JUNIT RESULTS Name:

Re: build.kde.org issues

On Tue, Nov 17, 2015 at 12:57 AM, Jonathan Riddell wrote: > khotkeys: I can't recreate this failure locally so it could be an > issue with the CI > > https://build.kde.org/job/khotkeys%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/21/console > >

Jenkins-kde-ci: khotkeys master kf5-qt5 » Linux,gcc - Build # 22 - Fixed!

GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/khotkeys%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/22/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 17 Nov 2015 05:50:41 + Build duration: 5 min 45 sec CHANGE SET No changes JUNIT RESULTS Name:

Re: Review Request 126080: [xembed-sni-proxy] Fix quit if we fail to claim

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126080/ --- (Updated Ноя. 16, 2015, 11:43 д.п.) Review request for Plasma.

Re: Review Request 125990: [Widget Explorer] Fix applet uninstall

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125990/#review88405 --- Ship it! Ship It! - Marco Martin On Nov. 15, 2015, 3:06

Re: Review Request 126074: [Widget Explorer] Allow filtering for uninstallable widgets

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126074/#review88404 --- Ship it! Ship It! - Marco Martin On Nov. 15, 2015, 3:07

Re: Review Request 126080: [xembed-sni-proxy] Fix quit if we fail to claim

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126080/#review88408 --- Ship it! - David Edmundson On Nov. 16, 2015, 8:43 a.m.,

Re: Review Request 126013: fix: kacess should not be built if xkb is not present

> On Nov. 10, 2015, 1:26 a.m., David Edmundson wrote: > > Ship It! > > Johan Ouwerkerk wrote: > Forgot to mention that the patch can be pulled from the fix-cmake-xkb-dep > branch at: g...@github.com:cmacq2/plasma-desktop.git > > Johan Ouwerkerk wrote: > Please note: I do not have

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 nachm., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

Re: RFC: Disable DrKonqi

Hi, You do get an annoying "app is no longer working" window with (sometimes) options to wait longer or send a report. Let's have a look at what others do: - iOS: just dumps you to the home screen - Windows RT: dumps you to the home screen‎, its guidelines explicitly even say: do not annoy the

Re: Review Request 125991: [Widget Explorer] Allow uninstalling user-installed applets

> On Nov. 8, 2015, 3:37 p.m., Thomas Pfeiffer wrote: > > Showing it always does make sense as long as it affects only a few > > Plasmoids, yes. > > About the undo: I'm wondering if it might make sense - for consistency's > > sake - to do the undo mechanism similar to undoing the removal of a

RFC: Disable DrKonqi

Hi all, as some might know I got my first Android tablet recently and made an observation when using it: the quality is extremely bad! I have constant application crashers. I experience significantly more crashers in the half an hour usage of Android each day, than on the 10 hours of KDE

Re: Review Request 126013: fix: kacess should not be built if xkb is not present

> On Nov. 10, 2015, 2:26 a.m., David Edmundson wrote: > > Ship It! > > Johan Ouwerkerk wrote: > Forgot to mention that the patch can be pulled from the fix-cmake-xkb-dep > branch at: g...@github.com:cmacq2/plasma-desktop.git > > Johan Ouwerkerk wrote: > Please note: I do not have

Re: Review Request 126079: Keep drag and drop in range on both sides

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126079/#review88412 --- Ship it! Ship It! - Sebastian Kügler On Nov. 15, 2015,

Re: RFC: Disable DrKonqi

In data lunedì 16 novembre 2015 10:50:01 CET, Sebastian Kügler ha scritto: > us, and I also think that it would be cheating to just hide the drkonqi > window, but on the other hand, if it improves the user experience, that's > good, no? With my forum admin hat on: what I'm supposed to tell a

Re: RFC: Disable DrKonqi

On Monday, November 16, 2015 11:26:27 AM Kai Uwe Broulik wrote: > You do get an annoying "app is no longer working" window with (sometimes) > options to wait longer or send a report. > > Let's have a look at what others do: > - iOS: just dumps you to the home screen > - Windows RT: dumps you to

Re: RFC: plasmoid templates in plasma-sdk

On Friday, November 13, 2015 02:42:30 PM Marco Martin wrote: > On Friday 13 November 2015 14:05:50 Aleix Pol wrote: > > > I'm planning to move it to plasma-sdk (together with a new one for an > > > hybrid QML/C++) that it gets a bit less forgotten than the > > > long-forgotten kapptemplate > > >

Re: Review Request 126022: [ksmserver] Add a dedicated --no-lockscreen command line option

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126022/ --- (Updated Nov. 16, 2015, 8:14 a.m.) Status -- This change has been

Re: RFC: Disable DrKonqi

On Monday, November 16, 2015 11:58:45 AM CET Weng Xuetian wrote: > Just FYI, on a systemd based system, the coredump is already > automatically stored in journal and can be accessed easily by > coredumpctl. Good to know. That could make my life easier :-) > > Personally I use drkonqi a lot to

[Breeze] [Bug 355447] Mouse cursor theme partially fails on first login with new user account

https://bugs.kde.org/show_bug.cgi?id=355447 Martin Gräßlin changed: What|Removed |Added CC||mgraess...@kde.org ---

Re: Review Request 126091: Remove printscreen.khotkeys from KHotkeys package

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126091/#review88456 --- should the file be added to ksnapshot? - Martin Gräßlin On