Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126870/#review91521 --- >Continous usage, read as suspend/resume/use and so on,

[Differential] [Closed] D851: Port kded plugin to json metadata

2016-01-24 Thread heikobecker (Heiko Becker)
This revision was automatically updated to reflect the committed changes. Closed by commit rKWRITED5fd327369bc6: Port kded plugin to json metadata (authored by heikobecker). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D851?vs=2062=2081#toc REPOSITORY rKWRITED KWriteD CHANGES SINCE

Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126870/ --- Review request for KDE Frameworks, Plasma and David Edmundson. Bugs:

[Breeze] [Bug 358271] Some comboboxes are missing shadows and are hard to click

2016-01-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=358271 Hugo Pereira Da Costa changed: What|Removed |Added Assignee|plasma-devel@kde.org

Re: Window thumbnail discussion

2016-01-24 Thread Martin Graesslin
On Thursday, January 21, 2016 12:06:48 PM CET Martin Gräßlin wrote: > I plan to write something down in the wiki next week, which should be a > better starting point for discussions then. Created a wiki page: https://community.kde.org/KWin/Window_Metadata Please feel free to extend/fix as needed

Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread David Edmundson
> On Jan. 24, 2016, 2:43 p.m., David Edmundson wrote: > > >Continous usage, read as suspend/resume/use and so on, cause *cool* > > >tooltip animation transition between item to decrease it permormance > > > > Why does QPropertyAnimation decrease in performance after suspend/restore? > > > >

Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread Anthony Fieroni
> On Ян. 24, 2016, 4:43 след обяд, David Edmundson wrote: > > >Continous usage, read as suspend/resume/use and so on, cause *cool* > > >tooltip animation transition between item to decrease it permormance > > > > Why does QPropertyAnimation decrease in performance after suspend/restore? > > >

Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread Anthony Fieroni
> On Ян. 24, 2016, 4:43 след обяд, David Edmundson wrote: > > src/declarativeimports/core/tooltipdialog.cpp, line 184 > > > > > > You now have a a boolean and a potential null pointer both indicating > > the

Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126870/ --- (Updated Ян. 24, 2016, 5:20 след обяд) Review request for KDE

Re: Review Request 126841: Use PlasmaCore.IconItem in Kicker, Kickoff and Folder View

2016-01-24 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126841/#review91528 --- Ship it! Ship It! - Eike Hein On Jan. 22, 2016, 11:28

Re: Review Request 126870: Tooltip animation transition in the panel decrease it performance after continous usage

2016-01-24 Thread Anthony Fieroni
> On Ян. 24, 2016, 4:43 след обяд, David Edmundson wrote: > > >Continous usage, read as suspend/resume/use and so on, cause *cool* > > >tooltip animation transition between item to decrease it permormance > > > > Why does QPropertyAnimation decrease in performance after suspend/restore? > > >

Review Request 126878: [KRunner] Show single runner name

2016-01-24 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126878/ --- Review request for Plasma, KDE Usability and Vishesh Handa. Repository:

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

2016-01-24 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126877/ --- Review request for Plasma and Vishesh Handa. Repository: milou

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

Review Request 126879: [digital-clock] Use iso-codes for country names

2016-01-24 Thread Pino Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126879/ --- Review request for Localization and Translation (l10n), Plasma and John

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

2016-01-24 Thread Heiko Tietze
> On Jan. 24, 2016, 8:58 p.m., Jonathan Marten wrote: > > Ping anyone? > > Kai Uwe Broulik wrote: > 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

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

2016-01-24 Thread Heiko Tietze
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126300/#review91549 --- Ship it! Ship It! - Heiko Tietze On Jan. 10, 2016,

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

2016-01-24 Thread Antonis Geralis
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126688/#review91550 --- Hello! There is a typo in file ksolidnotify.cpp:166, also

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

2016-01-24 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126688/ --- (Updated Jan. 25, 2016, 12:40 a.m.) Status -- This change has been

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

2016-01-24 Thread Jonathan Marten
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126300/#review91537 --- Ping anyone? - Jonathan Marten On Jan. 10, 2016, 4:33

Re: Review Request 126879: [digital-clock] Use iso-codes for country names

2016-01-24 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126879/#review91552 --- +1 applets/digital-clock/plugin/timezonesi18n.cpp (line

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

2016-01-24 Thread Kai Uwe Broulik
> On Jan. 24, 2016, 10:26 nachm., Antonis Geralis wrote: > > Hello! There is a typo in file ksolidnotify.cpp:166, also shown in third > > screenshot. Well spotted. Fixed, thanks. - Kai Uwe --- This is an automatically generated

Re: Review Request 126879: [digital-clock] Use iso-codes for country names

2016-01-24 Thread Pino Toscano
> On Gen. 24, 2016, 10:39 p.m., David Edmundson wrote: > > applets/digital-clock/plugin/timezonesi18n.cpp, line 517 > > > > > > may as well keep the comment from the i18n? Not sure what you mean. The point here

Re: Review Request 126879: [digital-clock] Use iso-codes for country names

2016-01-24 Thread David Edmundson
> On Jan. 24, 2016, 10:39 p.m., David Edmundson wrote: > > applets/digital-clock/plugin/timezonesi18n.cpp, line 517 > > > > > > may as well keep the comment from the i18n? > > Pino Toscano wrote: > Not sure