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

2015-11-17 Thread Sebastian Kügler via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355447 Sebastian Kügler changed: What|Removed |Added CC||se...@kde.org

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

2015-11-17 Thread Sebastian Kügler
On Monday, November 16, 2015 04:16:26 PM Johan Ouwerkerk wrote: > > Are we just disabling it and then be done? > > That is not what this change is about. The change rectifies an inconsistency > between the top level cmake configure stuff which checks the build > environment for dependencies, and

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

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/ --- (Updated Nov. 17, 2015, 1:46 p.m.) Status -- This change has been

Re: [kde-promo] 5.5 announcement

2015-11-17 Thread Martin Klapetek
On Tue, Nov 17, 2015 at 8:02 AM, Sebastian Kügler wrote: > > IMO, the first and most prominent image should reflect the default more > closely. For the other images, customization is fine, but for the first > one, > we need the recognizability of the default setup. > But then we

Re: Review Request 126093: kcm_fonts: Don't touch any Xft options when anti-aliasing is set to system settings

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

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

2015-11-17 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126083/#review88474 --- Ship it! Ship It! - Bhushan Shah On Nov. 16, 2015, 5:13

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

2015-11-17 Thread Martin Walch via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355447 --- Comment #3 from Martin Walch --- Created attachment 95560 --> https://bugs.kde.org/attachment.cgi?id=95560=edit Video showing the broken cursor in action (In reply to Martin Gräßlin from comment #2) > Which distribution are

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

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126089/#review88477 --- There's a ColorButton in kquickcontrols. Also what's up with

Review Request 126098: Install session file for plasma on wayland

2015-11-17 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126098/ --- Review request for Plasma and Martin Gräßlin. Repository:

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Lamarque Souza
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126096/ --- (Updated Nov. 17, 2015, 4:31 p.m.) Review request for Plasma, Solid and

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126096/#review88493 --- Ship it! Ship It! - Daniel Vrátil On Nov. 17, 2015, 5:31

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Lamarque Souza
> On Nov. 17, 2015, 4:08 p.m., Martin Gräßlin wrote: > > backends/xrandr/xrandrscreen.cpp, line 67 > > > > > > Suggestion QScopedPointer > > Daniel Vrátil wrote: > Yes

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126096/#review88491 --- backends/xrandr/xrandrscreen.cpp (line 66)

Re: Review Request 126098: Install session file for plasma on wayland

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126098/#review88490 --- Looks good to me - Martin Gräßlin On Nov. 17, 2015, 4:55

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Daniel Vrátil
> On Nov. 17, 2015, 5:08 p.m., Martin Gräßlin wrote: > > backends/xrandr/xrandrscreen.cpp, line 67 > > > > > > Suggestion QScopedPointer Yes please, but use XCB::ScopedPointer

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Daniel Vrátil
> On Nov. 17, 2015, 5:08 p.m., Martin Gräßlin wrote: > > backends/xrandr/xrandrscreen.cpp, line 67 > > > > > > Suggestion QScopedPointer > > Daniel Vrátil wrote: > Yes

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Lamarque Souza
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126096/ --- (Updated Nov. 17, 2015, 4:41 p.m.) Status -- This change has been

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

2015-11-17 Thread andreas kainz
> On Nov. 17, 2015, 9:42 vorm., Kai Uwe Broulik wrote: > > Heh, I got quoted by various magazines saying we've matured and are no > > longer obsessed with the K in everything :D > > > > Overall I like it. > > - shine-heart: kinda looks like it has a disease rather than shining > > -

Re: Review Request 126096: Fix memory leak in kscreen backend of libkscreen

2015-11-17 Thread Lamarque Souza
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126096/ --- (Updated Nov. 17, 2015, 4:26 p.m.) Review request for Plasma, Solid and

Re: Review Request 126097: [server] Fix possible crash after deleting an output(device)interface

2015-11-17 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126097/#review88489 --- Ship it! looks good to me. - Sebastian Kügler On Nov. 17,

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

2015-11-17 Thread Andrea Del Sarto
> On Nov. 17, 2015, 9:42 a.m., Kai Uwe Broulik wrote: > > Heh, I got quoted by various magazines saying we've matured and are no > > longer obsessed with the K in everything :D > > > > Overall I like it. > > - shine-heart: kinda looks like it has a disease rather than shining > > - face-smile:

Review Request 126097: [server] Fix possible crash after deleting an output(device)interface

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126097/ --- Review request for Plasma and Sebastian Kügler. Repository: kwayland

Re: Review Request 126097: [server] Fix possible crash after deleting an output(device)interface

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126097/ --- (Updated Nov. 17, 2015, 3:37 p.m.) Status -- This change has been

Re: Review Request 126093: kcm_fonts: Don't touch any Xft options when anti-aliasing is set to system settings

2015-11-17 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126093/ --- (Updated Nov. 17, 2015, 1:55 p.m.) Status -- This change has been

Review Request 126095: [backends/hwcomposer] Announce support for DPMS

2015-11-17 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126095/ --- Review request for Plasma and Kai Uwe Broulik. Repository: kwin

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

2015-11-17 Thread Marco Martin
> On Nov. 17, 2015, 1:24 p.m., Kai Uwe Broulik wrote: > > themeexplorer/package/contents/ui/ColorEditor.qml, line 95 > > > > > > Doesn't "Dialog" do that automatically? kinda... only if i let him do the buttons,

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

2015-11-17 Thread Andrea Del Sarto
> On Nov. 16, 2015, 11:03 p.m., Jonathan Riddell wrote: > > Loving the icons! > > > > (not so keen on the random K name, I'd just go for Konqi as a name) > > andreas kainz wrote: > I'm for konqui meets Katie then we can make a love story. will ASK Andrea > to join the discussion. thanks

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

2015-11-17 Thread Marco Martin
> On Nov. 17, 2015, 1:15 p.m., Sebastian Kügler wrote: > > File Attachment: snapshot18.png - snapshot18.png > > > > > > spacing (also on the right and at the bottom) not sure.. on my system i set the window decoration to be no side

Re: RFC: Disable DrKonqi

2015-11-17 Thread Matthias Klumpp
2015-11-17 8:52 GMT+01:00 Martin Graesslin : > 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.

Re: [kde-promo] 5.5 announcement

2015-11-17 Thread Martin Klapetek
On Tue, Nov 17, 2015 at 11:31 AM, Łukasz Sawicki wrote: > > Make sure that the items on the desktop are properly resized and there is > no scrollbars visible (desktop settings - wallpaper and folder view > plasmoid). This is quite a common complaint from the non KDE users. >

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

2015-11-17 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126089/#review88435 --- Overall, good stuff. Some minor comments, and my review

Re: Review Request 126098: Install session file for plasma on wayland

2015-11-17 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126098/#review88497 --- This is super cool and deserves a blog post with bells and

Re: Review Request 126101: allow loading backends in-process

2015-11-17 Thread Sebastian Kügler
> On Nov. 17, 2015, 9:55 p.m., Daniel Vrátil wrote: > > Thanks a lot for the patch! I like the general approach and I just have two > > objections regarding the implementation: > > > > 1) I don't like the split of the ConfigOperations for In- and > > Out-Of-Process cases. Please just

Re: Review Request 125705: Use LANG for month names in calendar applet

2015-11-17 Thread Martin Klapetek
> On Oct. 28, 2015, 9:44 p.m., Martin Klapetek wrote: > > So...any word on this? > > Albert Astals Cid wrote: > Re-reading your comment, if i understand it correctly, you claim it is a > bug in Qt that it does obey LC_DATE for month names when according to you > LC_DATE is only for the

Re: Review Request 125864: Add back the Agenda part of the Calendar

2015-11-17 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125864/ --- (Updated Nov. 18, 2015, 1:28 a.m.) Status -- This change has been

Re: Review Request 126099: [kicker] Call user switcher instead of unconditionally starting a new session

2015-11-17 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126099/#review88503 --- Ship it! Ship It! - Eike Hein On Nov. 17, 2015, 7:09

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

2015-11-17 Thread Sebastian Kügler
> 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

Review Request 126101: allow loading backends in-process

2015-11-17 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126101/ --- Review request for Plasma, Solid, Aleix Pol Gonzalez, Daniel Vrátil, and

Re: Review Request 126101: allow loading backends in-process

2015-11-17 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126101/ --- (Updated Nov. 18, 2015, 2:36 a.m.) Review request for Plasma, Solid,

Re: Review Request 126101: allow loading backends in-process

2015-11-17 Thread Sebastian Kügler
> On Nov. 17, 2015, 9:55 p.m., Daniel Vrátil wrote: > > src/backendmanager_p.h, line 130 > > > > > > Why is this even a QHash? We don't want to run multiple backends at > > once, or do we? Wouldn't just holding

Re: Review Request 125862: Add simple calendar events plugin based on KHolidays

2015-11-17 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125862/ --- (Updated Nov. 18, 2015, 1:33 a.m.) Review request for Plasma. Changes

Re: Review Request 125465: kcm_keyboard: Use udev device notifier when xcb-xinput is not available

2015-11-17 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125465/#review88464 --- If there are no objections, I will push this before beta

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

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126092/#review88462 --- Heh, I got quoted by various magazines saying we've matured

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

2015-11-17 Thread Boudhayan Gupta
> On Nov. 17, 2015, 12:51 p.m., Martin Gräßlin wrote: > > should the file be added to ksnapshot? Already done, installs to both {CMAKE_INSTALL_PREFIX}/share/khotkeys and {CMAKE_INSTALL_PREFIX}/share/apps/khotkeys (for kde4 compatibility) - Boudhayan

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

2015-11-17 Thread andreas kainz
> On Nov. 16, 2015, 11:03 nachm., Jonathan Riddell wrote: > > Loving the icons! > > > > (not so keen on the random K name, I'd just go for Konqi as a name) I'm for konqui meets Katie then we can make a love story. will ASK Andrea to join the discussion. - andreas

Re: Review Request 126036: Update wallpaper package with plasma 5.5 wallpaper contest

2015-11-17 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126036/ --- (Updated Nov. 17, 2015, 8:35 vorm.) Status -- This change has been

Review Request 126093: kcm_fonts: Don't touch any Xft options when anti-aliasing is set to system settings

2015-11-17 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126093/ --- Review request for Plasma. Repository: plasma-desktop Description

Review Request 126099: [kicker] Call user switcher instead of unconditionally starting a new session

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126099/ --- Review request for Plasma and Eike Hein. Repository: plasma-desktop

Review Request 126100: [KRunner] Forward KRunner switchUser to KSMServer

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126100/ --- Review request for Plasma and Vishesh Handa. Repository:

Re: Review Request 126101: allow loading backends in-process

2015-11-17 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126101/#review88502 --- Thanks a lot for the patch! I like the general approach and I

Re: Review Request 126099: [kicker] Call user switcher instead of unconditionally starting a new session

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126099/ --- (Updated Nov. 17, 2015, 9:18 p.m.) Status -- This change has been

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

2015-11-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125991/ --- (Updated Nov. 17, 2015, 9:37 nachm.) Review request for Plasma and KDE