[Plasma Workspace Wallpapers] [Bug 346251] New: Default wallpaper cannot be found in desktop settings

2015-04-16 Thread Vit Pelcak
https://bugs.kde.org/show_bug.cgi?id=346251 Bug ID: 346251 Summary: Default wallpaper cannot be found in desktop settings Product: Plasma Workspace Wallpapers Version: 5.2.95 Platform: Other OS: Linux Status:

[Breeze] [Bug 340137] Implement support for window groups (window tabs) in Breeze

2015-04-16 Thread Rodrigo Fernandes
https://bugs.kde.org/show_bug.cgi?id=340137 Rodrigo Fernandes rfernan...@gmx.com changed: What|Removed |Added Version|5.1.0 |5.2.95 --- Comment #3

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/#review78990 --- applets/quickshare/plasmoid/contents/ui/main.qml (line 98)

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
On April 15, 2015, 11:03 p.m., Aleix Pol Gonzalez wrote: Interesting. Do you have any screenshot about what this share button looks like? Also I'm unsure about what it should look like, QuickShare overrides the value in the clipboard, is that what you want as well? Or should we

[Plasma Workspace Wallpapers] [Bug 346251] Default wallpaper cannot be found in desktop settings

2015-04-16 Thread Marco Martin
https://bugs.kde.org/show_bug.cgi?id=346251 Marco Martin notm...@gmail.com changed: What|Removed |Added CC||notm...@gmail.com --- Comment

Re: Review Request 123372: Update the date in the calendar/tooltip when user changes clock timezone

2015-04-16 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123372/ --- (Updated April 16, 2015, 9:59 a.m.) Status -- This change has been

[Breeze] [Bug 340137] Implement support for window groups (window tabs) in Breeze

2015-04-16 Thread Phil Schaf
https://bugs.kde.org/show_bug.cgi?id=340137 Phil Schaf flying-sh...@web.de changed: What|Removed |Added CC||flying-sh...@web.de Ever

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/#review79036 --- applets/quickshare/plasmoid/contents/ui/main.qml (line 228)

[Plasma Workspace Wallpapers] [Bug 346252] New: Default wallpaper has jagged edges

2015-04-16 Thread Vit Pelcak
https://bugs.kde.org/show_bug.cgi?id=346252 Bug ID: 346252 Summary: Default wallpaper has jagged edges Product: Plasma Workspace Wallpapers Version: 5.2.95 Platform: openSUSE RPMs OS: Linux Status: UNCONFIRMED

[Plasma Workspace Wallpapers] [Bug 346251] Default wallpaper cannot be found in desktop settings

2015-04-16 Thread Jonathan Riddell
https://bugs.kde.org/show_bug.cgi?id=346251 Jonathan Riddell j...@jriddell.org changed: What|Removed |Added CC||j...@jriddell.org ---

[Plasma Workspace Wallpapers] [Bug 346252] Default wallpaper has jagged edges

2015-04-16 Thread Marco Martin
https://bugs.kde.org/show_bug.cgi?id=346252 Marco Martin notm...@gmail.com changed: What|Removed |Added Resolution|--- |WAITINGFORINFO

[Plasma Workspace Wallpapers] [Bug 346252] Default wallpaper has jagged edges

2015-04-16 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=346252 David Edmundson k...@davidedmundson.co.uk changed: What|Removed |Added CC|

Build failed in Jenkins: plasma-workspace_stable_qt5 #105

2015-04-16 Thread KDE CI System
See http://build.kde.org/job/plasma-workspace_stable_qt5/105/changes Changes: [mklapetek] Update the date in the calendar/tooltip when user changes clock timezone -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Aleix Pol Gonzalez
On April 16, 2015, 12:13 p.m., David Edmundson wrote: applets/quickshare/plugin/contenttracker.cpp, line 84 https://git.reviewboard.kde.org/r/123375/diff/1/?file=361201#file361201line84 normalise the signatures, const shouldn't be there.

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
On April 16, 2015, 10:19 a.m., Aleix Pol Gonzalez wrote: applets/quickshare/plasmoid/contents/ui/main.qml, line 228 https://git.reviewboard.kde.org/r/123375/diff/1/?file=361198#file361198line228 I'm wondering whether we should just always show the document-share icon...?

Re: Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Martin Klapetek
On April 16, 2015, 1:43 p.m., Kai Uwe Broulik wrote: I think we should always pulse, and if available, use the needs attention icon (but don't cycle between normal and attention icon) Martin Klapetek wrote: If you don't cycle it's quite easy to miss, so I think there'd be very

Re: Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123381/ --- (Updated April 16, 2015, 1:23 p.m.) Review request for Plasma. Changes

Re: Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Marco Martin
On April 16, 2015, 11:42 a.m., Kai Uwe Broulik wrote: What about just wrapping it in a ScrollView rather than plain flickable so you get scrollbars that make it obvious there's more? i would be fine with that as well - Marco ---

Re: Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Aleix Pol Gonzalez
On April 16, 2015, 1:42 p.m., Kai Uwe Broulik wrote: What about just wrapping it in a ScrollView rather than plain flickable so you get scrollbars that make it obvious there's more? Marco Martin wrote: i would be fine with that as well Sebastian Kügler wrote: +1 for

Re: Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123381/ --- (Updated April 16, 2015, 1:12 p.m.) Review request for Plasma. Changes

Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123382/ --- Review request for Plasma. Bugs: 341165

Re: Review Request 123280: Kickoff - Properly register recent applications right after startup

2015-04-16 Thread Anthony Vital
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123280/ --- (Updated avr. 16, 2015, 1:09 après-midi) Review request for Plasma.

Re: Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123381/#review79045 --- I think we should always pulse, and if available, use the

Re: Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123382/#review79044 --- What about just wrapping it in a ScrollView rather than plain

Re: Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123382/#review79043 --- applets/systemtray/package/contents/ui/main.qml (line 70)

Re: Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Martin Klapetek
On April 16, 2015, 1:43 p.m., Kai Uwe Broulik wrote: I think we should always pulse, and if available, use the needs attention icon (but don't cycle between normal and attention icon) If you don't cycle it's quite easy to miss, so I think there'd be very little point in just switching

Re: Review Request 123382: System tray: Show more than 8 entries.

2015-04-16 Thread Sebastian Kügler
On April 16, 2015, 11:42 a.m., Kai Uwe Broulik wrote: What about just wrapping it in a ScrollView rather than plain flickable so you get scrollbars that make it obvious there's more? Marco Martin wrote: i would be fine with that as well +1 for scrollview. - Sebastian

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
On April 16, 2015, 10:13 a.m., David Edmundson wrote: applets/quickshare/plasmoid/contents/ui/main.qml, line 98 https://git.reviewboard.kde.org/r/123375/diff/1/?file=361198#file361198line98 content tracker has a mime type property sent over DBus, use that? yeah, seems it's not

Review Request 123381: Fallback to AttentionIcon for SNI when animations are disabled

2015-04-16 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123381/ --- Review request for Plasma. Repository: plasma-workspace Description

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/ --- (Updated April 16, 2015, 11:19 a.m.) Review request for Plasma and Aleix

[Breeze] [Bug 340137] Implement support for window groups (window tabs) in Breeze

2015-04-16 Thread Martin Gräßlin
https://bugs.kde.org/show_bug.cgi?id=340137 --- Comment #4 from Martin Gräßlin mgraess...@kde.org --- I'm sorry to say, but window tabbing is not yet implemented for kdecoration2 :-( -- You are receiving this mail because: You are the assignee for the bug.

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/#review79061 --- Ship it! Ship It! - Aleix Pol Gonzalez On April 16, 2015,

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/ --- (Updated April 16, 2015, 4 p.m.) Status -- This change has been

Build failed in Jenkins: plasma-workspace_stable_qt5 #106

2015-04-16 Thread KDE CI System
See http://build.kde.org/job/plasma-workspace_stable_qt5/106/changes Changes: [mklapetek] [notifications] Make sure the timer is restarted after the popup moves -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building

Re: Review Request 123375: give quickshare the SLC capability

2015-04-16 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123375/ --- (Updated April 16, 2015, 3:42 p.m.) Review request for Plasma and Aleix

Re: Review Request 123377: Fix i18n catalog connections in comic applet

2015-04-16 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123377/ --- (Updated April 16, 2015, 3:54 p.m.) Status -- This change has been

Re: Review Request 123280: Kickoff - Properly register recent applications right after startup

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

Re: Review Request 123280: Kickoff - Properly register recent applications right after startup

2015-04-16 Thread Anthony Vital
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123280/ --- (Updated avr. 16, 2015, 4:26 après-midi) Status -- This change has

Build failed in Jenkins: plasma-desktop_stable_qt5 #111

2015-04-16 Thread KDE CI System
See http://build.kde.org/job/plasma-desktop_stable_qt5/111/changes Changes: [anthony.vital] Kickoff - Properly register recent applications right after startup -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building

Review Request 123386: Fix i18n catalog connection of a string in shortcuts config

2015-04-16 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123386/ --- Review request for Plasma. Repository: plasma-desktop Description

Re: Review Request 123386: Fix i18n catalog connection of a string in shortcuts config

2015-04-16 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123386/#review79067 --- Ship it! I don't think there's another way - David

Re: Review Request 123351: Display sddm loginprompt only on the primary monitor

2015-04-16 Thread David Kahles
On April 15, 2015, 10:53 p.m., David Kahles wrote: I just noticed another issue: when pressing the shutdown/reboot button, the login bar is shifted to the left and the shutdown/reboot bar comes in from the right. If you have a monitor left of the primary monitor, the loginbar is

Re: Review Request 123351: Display sddm loginprompt only on the primary monitor

2015-04-16 Thread David Kahles
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123351/ --- (Updated April 16, 2015, 3:59 p.m.) Review request for Plasma and David

Re: Review Request 123386: Fix i18n catalog connection of a string in shortcuts config

2015-04-16 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123386/ --- (Updated April 16, 2015, 5:26 p.m.) Status -- This change has been

Build failed in Jenkins: plasma-desktop_stable_qt5 #112

2015-04-16 Thread KDE CI System
See http://build.kde.org/job/plasma-desktop_stable_qt5/112/changes Changes: [lasse.liehu] Fix i18n catalog connection of a string in Shortcuts config -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on

Build failed in Jenkins: plasma-desktop_stable_qt5 #113

2015-04-16 Thread KDE CI System
See http://build.kde.org/job/plasma-desktop_stable_qt5/113/changes Changes: [lasse.liehu] Fix i18n in desktop containment -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER

Re: Review Request 123299: Kickoff - Watch changes in the recent documents directory

2015-04-16 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123299/#review79079 --- Ship it! Minor changes needed, otherwise it's good to go in.

Re: Review Request 123299: Kickoff - Watch changes in the recent documents directory

2015-04-16 Thread Sebastian Kügler
On Thursday, April 16, 2015 21:10:02 Anthony Vital wrote: On avr. 16, 2015, 8:46 après-midi, Sebastian Kügler wrote: applets/kickoff/core/recentlyusedmodel.cpp, line 113 https://git.reviewboard.kde.org/r/123299/diff/1/?file=360493#file360493 line113 Why this change? Because

Re: Review Request 123387: Fix i18n in desktop containment

2015-04-16 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123387/#review79076 --- Ship it! Ship It! - Lukáš Tinkl On Dub. 16, 2015, 8:58

Re: Review Request 123387: Fix i18n in desktop containment

2015-04-16 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123387/ --- (Updated April 16, 2015, 8:41 p.m.) Status -- This change has been

Re: Review Request 123387: Fix i18n in desktop containment

2015-04-16 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123387/#review79077 --- Ship it! Ship It! - Sebastian Kügler On April 16, 2015,

Review Request 123387: Fix i18n in desktop containment

2015-04-16 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123387/ --- Review request for Plasma. Repository: plasma-desktop Description

Re: Review Request 123299: Kickoff - Watch changes in the recent documents directory

2015-04-16 Thread Anthony Vital
On avr. 16, 2015, 8:46 après-midi, Sebastian Kügler wrote: applets/kickoff/core/recentlyusedmodel.cpp, line 113 https://git.reviewboard.kde.org/r/123299/diff/1/?file=360493#file360493line113 Why this change? Because documentUrl doesn't exist anymore, so I didn't want to leave it

Re: Review Request 123299: Kickoff - Watch changes in the recent documents directory

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