Re: Review Request 126621: [Task Manager] Add support for Unity Launcher API and Application Jobs

2016-01-17 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma, KDE Usability, Craig Drummond, Eike Hein, and Lukáš Tinkl. Changes --- Submitted with commit e284e9dc17051f22d05985e218fa44ddaba78de5 by Kai Uwe Broulik to branch master. Bugs: 343632 https://bugs.kde.org/show_bug.cgi?id=343632

Re: Review Request 126652: [Units] Return at least 1ms for durations

2016-01-17 Thread Kai Uwe Broulik
as submitted. Review request for Plasma. Changes --- Submitted with commit d9719f10374f4885a3ae3efd7e3b4f23e953e59d by Kai Uwe Broulik to branch master. Bugs: 357352 http://bugs.kde.org/show_bug.cgi?id=357352 Repository: plasma-framework Description --- Animators do not

Re: Review Request 124586: Dismiss logout dialog when clicking outside

2016-01-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124586/#review91199 --- Ping - Kai Uwe Broulik On Jan. 2, 2016, 12:54 nachm., Kai

Re: Review Request 126151: Port avatar gallery from kcm_useraccounts to user-manager

2016-01-17 Thread Kai Uwe Broulik
Broulik On Nov. 23, 2015, 9:50 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-17 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review91201 --- Ping - Kai Uwe Broulik On Jan. 7, 2016, 9:05 nachm., Kai

Review Request 126776: [Panel Containment] Hide applet if its status is Hidden

2016-01-17 Thread Kai Uwe Broulik
apses (Review 126777) when I only have one virtual desktop and re-appears if there are multiple. It also re-appears when I enter panel configuration. This should also be used for the quick chat applet. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing

Review Request 126777: [Pager] Use HiddenStatus instead of size hack

2016-01-17 Thread Kai Uwe Broulik
--- See Review 126776 Diffs - applets/pager/package/contents/ui/main.qml 0c367c6 applets/pager/plugin/pager.cpp ca31cb2 Diff: https://git.reviewboard.kde.org/r/126777/diff/ Testing --- Works Thanks, Kai Uwe Broulik ___ Plasma

Re: Review Request 126779: Fix for bug 358135: Media Player widget in expanded mode makes Amarok skip tracks

2016-01-18 Thread Kai Uwe Broulik
on.qml (line 152) <https://git.reviewboard.kde.org/r/126779/#comment62314> Space after if: if (…) { - Kai Uwe Broulik On Jan. 18, 2016, 10:29 vorm., Eduardo Lara wrote: > > --- > This is an automatically gener

Re: Review Request 126767: Breeze Cursor (default) Update

2016-01-18 Thread Kai Uwe Broulik
> On Jan. 16, 2016, 11:04 vorm., Kai Uwe Broulik wrote: > > - The busy cursor looks unlike anything we use elsewhere and eerily similar > > to what Android has nowadays. It looks rather hectic and that the two lines > > briefly connect looks awkward. > > - The "

Re: Review Request 124586: Dismiss logout dialog when clicking outside

2016-01-18 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma. Changes --- Submitted with commit 3dce39fd21eb36a1d912a299fad7fc3e2a4a7c72 by Kai Uwe Broulik to branch Plasma/5.5. Bugs: 357143 https://bugs.kde.org/show_bug.cgi?id=357143 Repository: plasma-workspace Description --- Doesn&#

Re: Review Request 126776: [Panel Containment] Hide applet if its status is Hidden

2016-01-18 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma. Changes --- Submitted with commit dc7c3c1a0f25ae4e6f9d66b9c8df300d0768c9f7 by Kai Uwe Broulik to branch master. Repository: plasma-desktop Description --- With the newly introduced HiddenStatus we now have a way to hide panel applets

Re: Review Request 126777: [Pager] Use HiddenStatus instead of size hack

2016-01-18 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma. Changes --- Submitted with commit eb3b3006b9e275688481be7eb57b798f73d12faf by Kai Uwe Broulik to branch master. Repository: plasma-desktop Description --- See Review 126776 Diffs - applets/pager/package/contents/ui/main.qml

Re: Review Request 126786: Quicklaunch: Add option to show title

2016-01-18 Thread Kai Uwe Broulik
applets/quicklaunch/package/contents/ui/main.qml (line 106) <https://git.reviewboard.kde.org/r/126786/#comment62379> wrapMode: Text.NoWrap also? - Kai Uwe Broulik On Jan. 17, 2016, 9:35 nachm., David Rosca wrote: > > ---

Re: Review Request 126786: Quicklaunch: Add option to show title

2016-01-18 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126786/#review91270 --- Ship it! Ship It! - Kai Uwe Broulik On Jan. 18, 2016

Re: Review Request 126801: Don't show drop applet placeholder on desktop in immutable state

2016-01-18 Thread Kai Uwe Broulik
. disable the drop area? We have the problem of Firefox not opening a window because we eat the drop event. Of course doing that won't solve the root cause but at least alleviate it. - Kai Uwe Broulik On Jan. 18, 2016, 6:47 nachm., David Rosca

Re: Review Request 126642: [Theme] Take Plasma Framework version into account in theme cache

2016-01-19 Thread Kai Uwe Broulik
changes to the theme SVG files without the version number having changed, we have a lot of people complaining about broken themes with the 5.4->5.5 transition and we'll get this over and over again until we find a solution for that. - Kai Uwe Broulik On Jan. 5, 2016, 6:11 nachm., Kai Uwe

Re: Review Request 126811: Emit contextualAppletsAboutToShow in systemtray

2016-01-19 Thread Kai Uwe Broulik
n the Applet*interface* not Applet. - Kai Uwe Broulik On Jan. 19, 2016, 12:25 nachm., David Edmundson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard

Re: Review Request 126811: Emit contextualAppletsAboutToShow in systemtray

2016-01-19 Thread Kai Uwe Broulik
> On Jan. 19, 2016, 8:41 nachm., Kai Uwe Broulik wrote: > > Doesn't compile, contextualActionsAboutToShow is in the Applet*interface* > > not Applet. Also, it should probably call it on the currently active applet, not the syste

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-19 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma, KDE Usability and Eike Hein. Changes --- Submitted with commit f9e87f911bb7823b1ae1aca16606cc9b50adf7e0 by Kai Uwe Broulik to branch master. Bugs: 352126 https://bugs.kde.org/show_bug.cgi?id=352126 Repository: plasma-desktop

Re: Review Request 125972: Don't list systray containment actions in systray context menu.

2016-01-19 Thread Kai Uwe Broulik
context menu entry for the clipboard plasmoid. It even shows an arrow but there's no menu. Might be the case when an applet's state is active but I have forcefully hidden it? - Kai Uwe Broulik On Jan. 19, 2016, 12:29 nachm., David Edmun

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-19 Thread Kai Uwe Broulik
enerated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/#review91354 ------- On Jan. 19, 2016, 9:47 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To

Re: KDE papercuts love

2016-01-20 Thread Kai Uwe Broulik
Hi,I suggested removing them a couple of times. +1Also since it's not really the Air from 4.x but has some adjustments to use theme colors in tray icons before we dumped it in favor of Breeze. This makes it look r

Re: Window thumbnail discussion

2016-01-21 Thread Kai Uwe Broulik
Hi,Yes, please! That's the crazy idea Thomas and I had a while ago. Looking forward to see it implemented. I would make it a bit more generic though, ie. not necessarily tied to just window thumbnails.We now ‎have

Re: Window thumbnail discussion

2016-01-21 Thread Kai Uwe Broulik
I misread the name, Window Metadata doesn't imply a restriction to thumbnails, so all good for future exploration I guess :)  Awesome stuff! ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126837: Load lockscreen wallpaper as fast as possible

2016-01-21 Thread Kai Uwe Broulik
) <https://git.reviewboard.kde.org/r/126837/#comment62494> Please use the active property of Loader rather than setting the source in various places. - Kai Uwe Broulik On Jan. 21, 2016, 3:04 nachm., Marco Martin wrote: > > -

Re: Review Request 126837: Load lockscreen wallpaper as fast as possible

2016-01-21 Thread Kai Uwe Broulik
> On Jan. 21, 2016, 4 nachm., Kai Uwe Broulik wrote: > > lookandfeel/contents/lockscreen/LockScreen.qml, line 102 > > <https://git.reviewboard.kde.org/r/126837/diff/1/?file=435073#file435073line102> > > > > Please use the active property of Loader rathe

Re: Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-23 Thread Kai Uwe Broulik
7;s also a reserved QML keyword. Also, you can access any role with a name clash by accessing it through "model", ie. model.index - Kai Uwe Broulik On Jan. 23, 2016, 1:14 nachm., Daniel Faust wrote: > > --- > This is an autom

Re: Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-23 Thread Kai Uwe Broulik
> On Jan. 23, 2016, 1:22 nachm., Kai Uwe Broulik wrote: > > id isn't particularly better either as that's also a reserved QML keyword. > > > > Also, you can access any role with a name clash by accessing it through > > "model", ie. model.index

Re: Review Request 126642: [Theme] Take Plasma Framework version into account in theme cache

2016-01-23 Thread Kai Uwe Broulik
org/r/126642/#review91402 --- On Jan. 5, 2016, 6:11 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > htt

Re: Review Request 126688: RFC: [Device Notifier] Provide inline feedback

2016-01-24 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and KDE Usability. Changes --- Submitted with commit 1ed7209f08aa69cec7847c606da8a2fc091dc603 by Kai Uwe Broulik to branch master. Repository: plasma-workspace Description --- Instead of showing a detached status bar with the device

Review Request 126878: [KRunner] Show single runner name

2016-01-24 Thread Kai Uwe Broulik
name label https://git.reviewboard.kde.org/media/uploaded/files/2016/01/24/d398ee71-78e5-458f-ad48-cf4fe828156f__runnername.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 126877: [Milou] Expose single runner name and icon

2016-01-24 Thread Kai Uwe Broulik
--- Works. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126300: Plasma Search KCM: display the runner descriptions

2016-01-24 Thread Kai Uwe Broulik
> On Jan. 24, 2016, 8:58 nachm., Jonathan Marten wrote: > > Ping anyone? Please if you fixed an issue raised mark is as fixed so the "open issues" count drops, making it easier to see what has changed. After it's zero and you already got a "Ship It" (see above, "Fix it", covered behind there's

Re: Review Request 126688: RFC: [Device Notifier] Provide inline feedback

2016-01-24 Thread Kai Uwe Broulik
enerated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126688/#review91550 --- On Jan. 24, 2016, 9:40 nachm., Kai Uwe Broulik wrote: > > --- > This is an automa

Re: Review Request 126748: [Plasma-NM] Don't emit connection deactivated message on suspending

2016-01-25 Thread Kai Uwe Broulik
-------------- On Jan. 15, 2016, 3:03 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126748/ > ---

Re: Review Request 126748: [Plasma-NM] Don't emit connection deactivated message on suspending

2016-01-26 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma, Jan Grulich and Lamarque Souza. Changes --- Submitted with commit d8edcc6ab77836127f3cef5833bcd9e8a88a6e9f by Kai Uwe Broulik to branch master. Repository: plasma-nm Description --- It's quite natural that the connection goes

Re: Review Request 126907: [klipper] Remove "Don't ask again" on clear clipboard history

2016-01-27 Thread Kai Uwe Broulik
> On Jan. 27, 2016, 3:55 nachm., Thomas Pfeiffer wrote: > > So, the situation is not that special: We ask for confirmation to err on > > the side of caution (which makes sense since there is no going back) but > > don't want to bother users with dialogs they don't care about. In fact, the > >

Re: Review Request 126878: [KRunner] Show single runner name

2016-01-28 Thread Kai Uwe Broulik
rated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126878/#review91645 --------------- On Jan. 24, 2016, 9:49 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatical

Re: Review Request 126878: [KRunner] Show single runner name

2016-01-29 Thread Kai Uwe Broulik
t's german, so label longer than usual also doesn't > help i guess) > > Kai Uwe Broulik wrote: > I explicitly made it so the runner window keeps its size but I can make > it grow accordingly. (is even easier code-wise) > > Marco Martin wrote: > what about us

Re: Review Request 126878: [KRunner] Show single runner name

2016-01-29 Thread Kai Uwe Broulik
t's german, so label longer than usual also doesn't > help i guess) > > Kai Uwe Broulik wrote: > I explicitly made it so the runner window keeps its size but I can make > it grow accordingly. (is even easier code-wise) > > Marco Martin wrote: > what about us

Re: Review Request 126551: [Service Runner] WIP: Add support for Jump List Actions

2016-01-30 Thread Kai Uwe Broulik
d.kde.org/media/uploaded/files/2015/12/28/9f3046f1-6acb-460d-a515-6abab6522dfe__krunnerjumplists.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126593: [Task Manager] Add Jump List actions to grouped tasks

2016-01-30 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126593/#review91807 --- Ping - Kai Uwe Broulik On Jan. 1, 2016, 6:41 nachm., Kai

Review Request 126929: [Quick Chat] Use HiddenStatus when no chats are present

2016-01-30 Thread Kai Uwe Broulik
fcb0c57 Diff: https://git.reviewboard.kde.org/r/126929/diff/ Testing --- Works like the Pager Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126929: [Quick Chat] Use HiddenStatus when no chats are present

2016-01-31 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma. Changes --- Submitted with commit 2a05e6efd280fcb3c902a82a1a16b3e71530fdb0 by Kai Uwe Broulik to branch master. Bugs: 298306 http://bugs.kde.org/show_bug.cgi?id=298306 Repository: ktp-desktop-applets Description --- This causes

Review Request 126938: [Widget Explorer] Show desktop on dragging widget

2016-01-31 Thread Kai Uwe Broulik
d the config dialog disappears immediately after it was shown. I guess this is because hide on window deactivate in Dialog checks for whether I tapped a containment, which I did, and then keeps the dialog open. Thanks, Kai Uwe Broulik ___ Plasma-

Review Request 126937: [KWindowSystemProxy] Add setter for showingDesktop

2016-01-31 Thread Kai Uwe Broulik
Description --- I don't know why this was omitted. Diffs - src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h 3e47f00 src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp a70ce90 Diff: https://git.reviewboard.kde.org/r/126937/diff/ Testing --- Thanks, Ka

Re: Review Request 126937: [KWindowSystemProxy] Add setter for showingDesktop

2016-01-31 Thread Kai Uwe Broulik
26937/diff/ Testing (updated) --- Works. I added it to the lockscreen qml in hopes of finding a privacy vulnerability but the greeter window stayed there no matter what. :) Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kd

Re: Review Request 126937: [KWindowSystemProxy] Add setter for showingDesktop

2016-02-01 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Martin Gräßlin. Changes --- Submitted with commit 8ea478dbcc3349dd421ac64fc5263801d6859185 by Kai Uwe Broulik to branch master. Repository: kdeclarative Description --- I don't know why this was omitted. Diffs -

Re: Review Request 126930: sddm theme: Add keyboard layout button

2016-02-01 Thread Kai Uwe Broulik
- 12) <https://git.reviewboard.kde.org/r/126930/#comment62693> You can do that in the property directly: property var layouts: [ { shortName: "us", longName: "English" } ...

Re: Review Request 126143: [ksmserver] Fall back to session runner if UserSwitcher.qml is broken

2016-02-01 Thread Kai Uwe Broulik
dialog.cpp 26d6710 Diff: https://git.reviewboard.kde.org/r/126143/diff/ Testing --- Broke my UserSwitcher.qml file, got KRunner instead. Fixed it, got the proper QML UI again. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-dev

Re: Review Request 125793: [Task Manager] Add proper close button to tooltip

2016-02-01 Thread Kai Uwe Broulik
--- Works. File Attachments New close button https://git.reviewboard.kde.org/media/uploaded/files/2015/10/25/0fea55c5-5a9b-4321-bec5-34ec3bacf239__closebuttonproper.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma

Re: Review Request 126942: LockScreen: Don't show layout variant in keyboard layout button

2016-02-01 Thread Kai Uwe Broulik
"de(nodeadkeys)" and "de(neo)" - Kai Uwe Broulik On Feb. 1, 2016, 11:08 vorm., David Rosca wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 126942: LockScreen: Don't show layout variant in keyboard layout button

2016-02-01 Thread Kai Uwe Broulik
hat's because kscreenlocker_greet is a QGuiApplication and QToolTip used by QtQuick.Controls requires QApplication - Kai Uwe Broulik On Feb. 1, 2016, 11:14 vorm., David Rosca wrote: > > --- > This is an automatically generated e-mai

Re: Review Request 126938: [Widget Explorer] Show desktop on dragging widget

2016-02-01 Thread Kai Uwe Broulik
to the DesktopView? - Kai Uwe Broulik On Jan. 31, 2016, 2:51 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 126944: LockScreen: Use layout display name in KeyboardLayoutButton

2016-02-01 Thread Kai Uwe Broulik
pp (line 92) <https://git.reviewboard.kde.org/r/126944/#comment62700> You can use a lambda - Kai Uwe Broulik On Feb. 1, 2016, 12:58 nachm., David Rosca wrote: > > --- > This is an automatically generate

Re: Review Request 126944: LockScreen: Use layout display name in KeyboardLayoutButton

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126944/#review91852 --- Ship it! Ship It! - Kai Uwe Broulik On Feb. 1, 2016, 1

Re: Review Request 126943: Add getLayoutDisplayName to keyboard layouts DBus API

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126943/#review91851 --- Ship it! Ship It! - Kai Uwe Broulik On Feb. 1, 2016

Re: Review Request 126938: [Widget Explorer] Show desktop on dragging widget

2016-02-01 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma, KDE Usability and Martin Gräßlin. Changes --- Submitted with commit 1aab04cec1c479d68171216b877d1b91a7af5adc by Kai Uwe Broulik to branch master. Bugs: 327530 https://bugs.kde.org/show_bug.cgi?id=327530 Repository: plasma-desktop

Re: Review Request 126593: [Task Manager] Add Jump List actions to grouped tasks

2016-02-01 Thread Kai Uwe Broulik
nager::BasicMenu which we could use. Unfortunately QAction doesn't support "sections" so the "Recent Documents" heading won't work with that. - Kai Uwe Broulik On Jan. 1, 2016, 6:41 nachm., Kai Uwe Broulik wrote: > > ---

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126949/#review91876 --- +1 - Kai Uwe Broulik On Feb. 1, 2016, 7:30 nachm., Martin

Review Request 126953: Remove Theme Details KCM

2016-02-01 Thread Kai Uwe Broulik
-4fef-8dc1-872588e967b5__simplifiedthemesettings.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 126947: [Shell] Make Applet/ContainmentConfig transient for DesktopView

2016-02-01 Thread Kai Uwe Broulik
w.cpp bc40083 Diff: https://git.reviewboard.kde.org/r/126947/diff/ Testing --- Dragged widget from Widget Explorer and successfully configured it. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/ma

Review Request 126954: [Notifications] Fix Job finished persistency

2016-02-01 Thread Kai Uwe Broulik
/ Testing --- Works now. KNotificationDBusTest also works fine. Plasma Undo notifications are no longer persistent but this is unrelated to this patch. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https

Re: Review Request 126954: [Notifications] Fix Job finished persistency

2016-02-01 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Martin Klapetek. Changes --- Submitted with commit bec916168c269df8807678c61aeb551cc3aac10e by Kai Uwe Broulik to branch master. Repository: plasma-workspace Description --- This was an oversight when porting to call the dataengine

Re: Review Request 126952: Set layout in kickoff config properly

2016-02-01 Thread Kai Uwe Broulik
28) <https://git.reviewboard.kde.org/r/126952/#comment62717> Why not make this the ColumnLayout? Should give you sensible minimum sizing of the dialog and stuff for free? - Kai Uwe Broulik On Feb. 1, 2016, 7:52 nachm., David Edmundson

Review Request 126956: [Desktop Containment] Remove custom tooltip delegate

2016-02-01 Thread Kai Uwe Broulik
ts/ui/FolderItemToolTipDelegate.qml a8b6f44 containments/desktop/package/contents/ui/FolderViewLayer.qml 1d470eb Diff: https://git.reviewboard.kde.org/r/126956/diff/ Testing --- Works. No more glitchy tooltips. (I wish the thumbnails in Folder tooltips were bigger, though) Thanks, Kai Uwe Br

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread Kai Uwe Broulik
> On Feb. 1, 2016, 7:39 nachm., Boudhayan Gupta wrote: > > +1 > > > > Ideally the default look-n-feel package should include only two themes - > > the default one and a high-contrast counterpart. In fact all other themes > > should reside elsewhere. > > > > If we still want to "officially" pr

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread Kai Uwe Broulik
> On Feb. 1, 2016, 8:12 nachm., andreas kainz wrote: > > Hi, I know I say I will keep oxygen-icons5 up to date, and I hope I get the > > time I need the plasma oxygen theme too to give the user the one single > > click feature (look & feel package). Is there an space where you can move > > "un

Review Request 126957: [FolderView] Show title in tooltip

2016-02-01 Thread Kai Uwe Broulik
ettings. File Attachments Screenshot https://git.reviewboard.kde.org/media/uploaded/files/2016/02/01/63234dfa-36e4-4daf-99ec-014ebb663c2c__Screenshot_20160201_231414.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread Kai Uwe Broulik
files/2016/02/01/a065dfcf-ca75-4d50-81aa-4d725245344e__Screenshot_20160201_234605.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126957: [FolderView] Show title in tooltip

2016-02-01 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Eike Hein. Changes --- Submitted with commit 954a4bc30b84b46d5cdfd9d5809dcd56fdff7dfd by Kai Uwe Broulik to branch master. Repository: plasma-desktop Description --- Show the title you would get in the popup also in the tooltip

Re: Review Request 126672: Fix most of Clazy warnings in plasma-framework

2016-02-01 Thread Kai Uwe Broulik
/#comment62711> QLatin1Char('.') ? src/plasma/scripting/scriptengine.cpp (line 67) <https://git.reviewboard.kde.org/r/126672/#comment62712> This vector is already const, so it cannot detach src/scriptengines/qml/plasmoid/containmentinterface.cpp (line 484) <https:/

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-01 Thread Kai Uwe Broulik
ts Screenshot https://git.reviewboard.kde.org/media/uploaded/files/2016/02/01/a065dfcf-ca75-4d50-81aa-4d725245344e__Screenshot_20160201_234605.png Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 126947: [Shell] Make Applet/ContainmentConfig transient for DesktopView

2016-02-01 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Martin Gräßlin. Changes --- Submitted with commit 467565d4556b0481779e3aa0cbce47ed9a919318 by Kai Uwe Broulik to branch master. Repository: plasma-workspace Description --- Prevents them from being hidden by show desktop. They&#x

Re: Review Request 126950: Fix kicker userExtraRunners

2016-02-01 Thread Kai Uwe Broulik
) <https://git.reviewboard.kde.org/r/126950/#comment62718> Array.prototype.push.apply(runners, plasmoid.configuration.extraRunners) is faster, sigh, JavaScript... https://jsperf.com/array-prototype-push-apply-vs-concat/20 - Kai Uwe Broulik On Feb. 1, 2016, 7:52 nachm.,

Re: Review Request 126951: Clone useExtraRunners config option from kicker to kickoff

2016-02-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126951/#review91897 --- Ship it! Ship It! - Kai Uwe Broulik On Feb. 1, 2016, 7

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-02 Thread Kai Uwe Broulik
and update to kf5 with Kubuntu 16.04 > > LTS. It would be bad to support oxygen for an additional LTS cycle, but for > > the switch it would be nice to give the user the posibility for an soft > > switch. > > andreas kainz wrote: > in addition a lot of dev's

Re: Review Request 126956: [Desktop Containment] Remove custom tooltip delegate

2016-02-02 Thread Kai Uwe Broulik
--- On Feb. 1, 2016, 9:23 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126956/ > -

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-02 Thread Kai Uwe Broulik
board.kde.org/r/126960/#review91926 --- On Feb. 1, 2016, 11:12 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 126962: Animate lock screen initialization

2016-02-02 Thread Kai Uwe Broulik
) <https://git.reviewboard.kde.org/r/126962/#comment62753> OpacityAnimator ? - Kai Uwe Broulik On Feb. 2, 2016, 3:10 nachm., Aleix Pol Gonzalez wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: KActivities split, 2nd phase

2016-02-02 Thread Kai Uwe Broulik
‎> ‎if plasma provided wallpapers via d-bus, it would not be a problem We should add something like this nonetheless as parsing the config files broke more than once already. Also there's requests for setting a wallpaper via DBus. Unfortunately due to Plasma's modular architecture wrt wallpapers

Re: Review Request 126968: Morphingpopups effect, to animate tooltips

2016-02-03 Thread Kai Uwe Broulik
359 - 360) <https://git.reviewboard.kde.org/r/126968/#comment62775> Braces pls Also below scripting/scriptedeffect.cpp (lines 363 - 365) <https://git.reviewboard.kde.org/r/126968/#comment62774> const the list and use range-for - K

Re: Review Request 124397: Add today button to calendar

2016-02-04 Thread Kai Uwe Broulik
:) - Kai Uwe Broulik On Feb. 4, 2016, 8:30 nachm., Bernhard Friedreich wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
0/#review91907 --- On Feb. 1, 2016, 11:12 nachm., Kai Uwe Broulik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.re

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
files/2016/02/04/d80b6161-3da3-4669-ba7c-19f62edbf542__Screenshot_20160205_001739.png How about this? #2 https://git.reviewboard.kde.org/media/uploaded/files/2016/02/04/73f2ada9-48cd-4b22-8ef2-5d37f2238442__Screenshot_20160205_001754.png Thanks, Kai Uwe Broulik ___ Plasma

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-04 Thread Kai Uwe Broulik
optional and turned off by default and were only re-introduced after frequent user complaints. - Kai Uwe --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126960/#review92071 ------------

Re: Review Request 126993: drkonqi : fix build errors and build nongui

2016-02-05 Thread Kai Uwe Broulik
deprecated. Not sure about the QFileDialog stuff. - Kai Uwe Broulik On Feb. 5, 2016, 1:21 nachm., René J.V. Bertin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 419 - Still Failing!

2016-02-06 Thread Kai Uwe Broulik
Marco, Your weather dataengine pulls in Qt XML but doesn't find it. Had the same build error locally.   Originalnachricht   Von: no-re...@kde.org Gesendet: Samstag, 6. Februar 2016 11:44 An: sgcl...@kde.org; plasma-devel@kde.org Antwort an: no-re...@kde.org Betreff: Jenkins-kde-ci: plasma-worksp

Re: Review Request 126811: Emit contextualAppletsAboutToShow in systemtray

2016-02-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126811/#review92126 --- Ship it! Ship It! - Kai Uwe Broulik On Feb. 6, 2016, 2

Re: Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

2016-02-07 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and KDE Usability. Changes --- Submitted with commit 4ebf130ee66a912a5dac360441a8c0060eab247e by Kai Uwe Broulik to branch master. Bugs: 336124, 348362 and 358536 http://bugs.kde.org/show_bug.cgi?id=336124 http://bugs.kde.org

Re: Review Request 126953: Remove Theme Details KCM

2016-02-07 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma, KDE Usability and Andrew Lake. Changes --- Submitted with commit 35bfaf2978ce4e24c07238b0230dd48b2ac6b566 by Kai Uwe Broulik to branch master. Repository: plasma-desktop Description --- This removes the theme editor thing in the

Re: Review Request 126994: Drop usage of Qt4Support

2016-02-07 Thread Kai Uwe Broulik
/kcategorizeditemsviewmodels.cpp (line 73) <https://git.reviewboard.kde.org/r/126994/#comment62851> This comment is obsolete now - Kai Uwe Broulik On Feb. 5, 2016, 8:16 nachm., Aleix Pol Gonzalez wrote: > > --- > This is an automatically g

Re: Review Request 126962: Animate lock screen initialization

2016-02-08 Thread Kai Uwe Broulik
> On Feb. 8, 2016, 9:29 vorm., Kai Uwe Broulik wrote: > > lookandfeel/contents/lockscreen/LockScreen.qml, line 52 > > <https://git.reviewboard.kde.org/r/126962/diff/2/?file=443681#file443681line52> > > > > onStatusChanged: { > > if (status === Load

Re: Review Request 126962: Animate lock screen initialization

2016-02-08 Thread Kai Uwe Broulik
) <https://git.reviewboard.kde.org/r/126962/#comment62864> onStatusChanged: { if (status === Loader.Ready) ? - Kai Uwe Broulik On Feb. 8, 2016, 3:24 vorm., Aleix Pol Gonzalez wrote: > > --- > This is a

Re: Review Request 126962: Animate lock screen initialization

2016-02-08 Thread Kai Uwe Broulik
> On Feb. 8, 2016, 9:29 vorm., Kai Uwe Broulik wrote: > > lookandfeel/contents/lockscreen/LockScreen.qml, line 52 > > <https://git.reviewboard.kde.org/r/126962/diff/2/?file=443681#file443681line52> > > > > onStatusChanged: { > > if (status === Lo

Re: ActionButton mouse area issue

2016-02-09 Thread Kai Uwe Broulik
‎> Well, in any case, it has to be possible to permanently scroll the last item > of a list above the action button, regardless of any property setting. > Otherwise it would simply be broken, no matter how we change the action > button > behavior (as it would always be partially covered by the b

Re: Review Request 126956: [Desktop Containment] Remove custom tooltip delegate

2016-02-10 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Eike Hein. Changes --- Submitted with commit d290ac0d0e071fa69cf07f1c3b95d140ecbfbf97 by Kai Uwe Broulik to branch master. Repository: plasma-desktop Description --- Fixes the tooltips being broken for me (not properly resizing

Re: Review Request 126878: [KRunner] Show single runner name

2016-02-11 Thread Kai Uwe Broulik
t's german, so label longer than usual also doesn't > help i guess) > > Kai Uwe Broulik wrote: > I explicitly made it so the runner window keeps its size but I can make > it grow accordingly. (is even easier code-wise) > > Marco Martin wrote: > what about us

Re: Review Request 126878: [KRunner] Show single runner name

2016-02-11 Thread Kai Uwe Broulik
t's german, so label longer than usual also doesn't > help i guess) > > Kai Uwe Broulik wrote: > I explicitly made it so the runner window keeps its size but I can make > it grow accordingly. (is even easier code-wise) > > Marco Martin wrote: > what about us

Re: Review Request 126877: [Milou] Expose single runner name and icon

2016-02-11 Thread Kai Uwe Broulik
marked as submitted. Review request for Plasma and Vishesh Handa. Changes --- Submitted with commit efcdab0f7238762bfdabfd3ba3569676eb8890ba by Kai Uwe Broulik to branch master. Repository: milou Description --- This exposes the name and icon of the single runner that is currently

<    1   2   3   4   5   6   7   8   9   10   >