---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118342/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118343/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118341/#review58541
---
Ship it!
Ship It!
- Martin Klapetek
On May 27, 2014, 7:43
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58542
---
Could you attach a screenshot of the Compositing KCM with the
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/
---
(Updated May 27, 2014, 9:25 a.m.)
Review request for Plasma, Hugo
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58545
---
File Attachment: compositing_kcm - bustle1.png
Heya
Something that just occured to me is that it appears we currently have
no autostart compatibility with kdelibs4 applications. Namely when an
application installs an autostart desktop file in 4.x they'd end up in
$PREFIX/share/autostart which by default does not appear to be
searched by
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118346/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/
---
(Updated May 27, 2014, 9:42 a.m.)
Review request for Plasma, Andrew
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118346/#review58547
---
at the moment this cannot be enabled yet. I have already
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118342/#review58548
---
Ship it!
Ship It!
- Aleix Pol Gonzalez
On May 27, 2014,
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118343/#review58552
---
Ship it!
Ship It!
- Sebastian Kügler
On May 27, 2014,
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
On Monday, May 26, 2014 17:47:35 Teo Mrnjavac wrote:
On Tuesday, May 20, 2014 15:16:25 Teo Mrnjavac wrote:
On Tuesday, May 20, 2014 14:55:16 David Edmundson wrote:
On Tue, May 20, 2014 at 2:28 PM, Teo Mrnjavac t...@kde.org wrote:
Another important issue is that currently the only tested
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118254/
---
(Updated May 27, 2014, 11:27 a.m.)
Status
--
This change has been
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118341/#review58556
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118342/#review58557
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118342/
---
(Updated May 27, 2014, 11:38 a.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118341/
---
(Updated May 27, 2014, 11:38 a.m.)
Status
--
This change has been
On May 27, 2014, 12:08 p.m., Martin Gräßlin wrote:
at the moment this cannot be enabled yet. I have already prepared a similar
change and backed away from it, due to no way to figure out whether prison
is compiled against Qt4 or Qt5.
Further more on Debian based systems the
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118346/
---
(Updated May 27, 2014, 1:45 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118347/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118348/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118351/
---
Review request for Plasma.
Repository: plasma-workspace
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58565
---
Makes sense, do we know why they used to be -1?
- Aleix Pol
On May 27, 2014, 1:11 p.m., Aleix Pol Gonzalez wrote:
Makes sense, do we know why they used to be -1?
If negative it would go into combinedLayoutSpacing which we don't implement.
Which would in turn use the (now deprecated) defaultLayoutSpacing
- David
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118347/#review58567
---
Unfortunately this way we cannot have a plural aware suffix.
On May 27, 2014, 3:52 p.m., Martin Klapetek wrote:
Unfortunately this way we cannot have a plural aware suffix.
What if you do this from code like setSuffix(i18np(signular, plural,
value)); ?
Oh but that's not a binding so it won't auto update...too much time in QML xD
Nevermind.
On May 27, 2014, 3:52 p.m., Martin Klapetek wrote:
Unfortunately this way we cannot have a plural aware suffix.
What if you do this from code like setSuffix(i18np(signular, plural,
value)); ?
Martin Klapetek wrote:
Oh but that's not a binding so it won't auto update...too
On Tue, May 27, 2014 at 3:37 PM, Martin Gräßlin mgraess...@kde.org wrote:
Hi all,
I'm currently considering whether we should change the window icon used in
KWin's killer helper (the dialog which gets shown when trying to close a
hung
window).
Attached are three variants.
1.
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58575
---
Ship it!
Ship It!
- Aleix Pol Gonzalez
On May 27, 2014,
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118351/#review58577
---
Ship it!
Ship It!
- Aleix Pol Gonzalez
On May 27, 2014,
bomb, cashew - or ask nuno ;-)
the kwin icon looks dull and has no meaning to the user.
i don't like using the hung app icon - the icon shows up in the taskbar as
well and creates a wrong idea about the source of the dialog (it feels odd
that a process would detect that it does no more respond
On Tuesday 27 May 2014 16:21:11 Aleix Pol wrote:
On Tue, May 27, 2014 at 3:37 PM, Martin Gräßlin mgraess...@kde.org wrote:
Hi all,
I'm currently considering whether we should change the window icon used in
KWin's killer helper (the dialog which gets shown when trying to close a
hung
See http://build.kde.org/job/plasma-workspace_master_qt5/288/changes
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
On Tue, May 27, 2014 at 4:55 PM, Martin Gräßlin mgraess...@kde.org wrote:
On Tuesday 27 May 2014 16:21:11 Aleix Pol wrote:
On Tue, May 27, 2014 at 3:37 PM, Martin Gräßlin mgraess...@kde.org
wrote:
Hi all,
I'm currently considering whether we should change the window icon
used in
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118357/
---
Review request for Plasma.
Repository: plasma-workspace
Description
See http://build.kde.org/job/plasma-workspace_master_qt5/289/changes
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58586
---
Ship it!
Ship It!
- Andrew Lake
On May 27, 2014, 9:42
On May 27, 2014, 4:12 p.m., Andrew Lake wrote:
Ship It!
In case anyone is wondering why this arbitrary fellow said Ship It!, I just
wanted to provide an explicit thumbs up that this matches the 4px spacing basis
grid recommended in the HIG
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118357/#review58588
---
I agree it makes sense, +1. Not adding ship it because the
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118348/
---
(Updated May 27, 2014, 4:48 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118348/#review58591
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118351/#review58590
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118351/
---
(Updated May 27, 2014, 4:48 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118351/#review58592
---
klipper/klipper.cpp
On May 27, 2014, 7:10 p.m., Ömer Fadıl Usta wrote:
klipper/klipper.cpp, line 888
https://git.reviewboard.kde.org/r/118351/diff/1/?file=275367#file275367line888
is this a forgotten KDialog or on purpose ?
it's on purpose. This is currently dead code (see the #ifdef HAVE_PRISON),
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/105312/
---
(Updated May 27, 2014, 2:27 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118361/
---
Review request for Plasma.
Repository: plasma-framework
Description
On May 27, 2014, 4:12 p.m., Andrew Lake wrote:
Ship It!
Andrew Lake wrote:
In case anyone is wondering why this arbitrary fellow said Ship It!, I
just wanted to provide an explicit thumbs up that this matches the 4px
spacing basis grid recommended in the HIG
On May 27, 2014, 4:12 p.m., Andrew Lake wrote:
Ship It!
Andrew Lake wrote:
In case anyone is wondering why this arbitrary fellow said Ship It!, I
just wanted to provide an explicit thumbs up that this matches the 4px
spacing basis grid recommended in the HIG
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118361/#review58609
---
src/plasmaquick/dialog.cpp
On May 20, 2014, 3:08 p.m., Alexander Potashev wrote:
You probably need to check in CMakeLists.txt that version of
shared-mime-info is at least 0.40.
Aurélien Gâteau wrote:
shared-mime-info is only required at run time so I am not sure it makes
sense to check at build time.
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117954/#review58611
---
Ship it!
keditfiletype/mimetypedata.h
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118361/#review58613
---
That's frickin' clever!
We can remove a lot of hacks thanks
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/
---
(Updated May 27, 2014, 10:38 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118337/#review58614
---
This review has been submitted with commit
On May 27, 2014, 4:12 p.m., Andrew Lake wrote:
Ship It!
Andrew Lake wrote:
In case anyone is wondering why this arbitrary fellow said Ship It!, I
just wanted to provide an explicit thumbs up that this matches the 4px
spacing basis grid recommended in the HIG
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118367/
---
Review request for Plasma.
Bugs: 335427
63 matches
Mail list logo