Re: Review Request 119025: Actually pass IBUS_DEFINITIONS when compiling ibus-panel

2014-07-18 Thread Raphael Kubo da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119025/#review62669 --- cmake/FindIBus.cmake

Re: Review Request 119355: Change use of MacroPushRequiredVars to CMakePushCheckState

2014-07-18 Thread Tobias Berner
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119355/ --- (Updated July 18, 2014, 8:55 p.m.) Status -- This change has been ma

Re: Review Request 119358: Fix position of ToolBox on second screen

2014-07-18 Thread Dan Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119358/ --- (Updated July 18, 2014, 6:50 p.m.) Status -- This change has been ma

Re: Review Request 119358: Fix position of ToolBox on second screen

2014-07-18 Thread David Edmundson
> On July 18, 2014, 4:42 p.m., David Edmundson wrote: > > Ship It! in the 5.0 branch too please - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119358/#review62660 --

Re: Review Request 119358: Fix position of ToolBox on second screen

2014-07-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119358/#review62660 --- Ship it! Ship It! - David Edmundson On July 18, 2014, 4:35

Review Request 119358: Fix position of ToolBox on second screen

2014-07-18 Thread Dan Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119358/ --- Review request for Plasma. Repository: plasma-workspace Description ---

Re: Review Request 119357: Replace SpringAnimation with NumberAnimation with EaseOutIn

2014-07-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119357/#review62659 --- Ship it! Looks good to me. - Aleix Pol Gonzalez On July 18

Re: Review Request 119357: Replace SpringAnimation with NumberAnimation with EaseOutIn

2014-07-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119357/ --- (Updated July 18, 2014, 4:21 p.m.) Review request for Plasma and Bhushan

Review Request 119357: Replace SpringAnimation with NumberAnimation with EaseOutIn

2014-07-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119357/ --- Review request for Plasma and Bhushan Shah. Repository: plasma-workspace

Google made a game in QML, the TextureManager for QML's Image component might be interesting

2014-07-18 Thread Mark Gaiser
Hi, I don't know exactly how they use it, but they needed it for a QML based game they call VoltAir. You can find the code here [1]. The issues they found while making the game can be found here [2]. Quite a few issues actually! The "TextureManager" classes [3, 4] I'm reporting this because i'v

Re: Review Request 119336: Convert FrameSvg to 9 textures: different approach

2014-07-18 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119336/#review62657 --- src/declarativeimports/core/framesvgitem.cpp

Re: [kde-community] Jitsi Meet installation for KDE?

2014-07-18 Thread Tomaz Canabrava
Seems to me a lovely idea. On Fri, Jul 18, 2014 at 10:36 AM, Thomas Pfeiffer wrote: > On Sunday 06 July 2014 00:08:38 Michael Bohlender wrote: >> Hi everyone, >> video conferencing seems to gain more and more importance within KDE. The >> Plasma team has their weekly "Monday Hangout", the VDG reg

Re: [kde-community] Jitsi Meet installation for KDE?

2014-07-18 Thread Thomas Pfeiffer
On Sunday 06 July 2014 00:08:38 Michael Bohlender wrote: > Hi everyone, > video conferencing seems to gain more and more importance within KDE. The > Plasma team has their weekly "Monday Hangout", the VDG regularly video-chats > as well, and I'm sure we're not the only ones. > Currently, Google Han

Re: Review Request 119330: Convert FrameSvgItem to use 9 tiles instead of a big texture.

2014-07-18 Thread Aleix Pol Gonzalez
> On July 18, 2014, 11:58 a.m., Marco Martin wrote: > > framesvgitem part starting to look good! > > I would still like to have all of framesvg private again tough Can you ellaborate on what don't you like about the approach? We're not installing headers, so we don't need to maintain ABI there.

Re: Review Request 119355: Change use of MacroPushRequiredVars to CMakePushCheckState

2014-07-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119355/#review62654 --- Ship it! Ship It! - Aleix Pol Gonzalez On July 18, 2014, 1

Re: Wayland planning meeting

2014-07-18 Thread Pier Luigi
2014-07-16 15:49 GMT+02:00 Sebastian Kügler : > Hey, > > During the planning meeting for 5.1, it became evident that we need a planning > meeting for our Wayland efforts. Here's a doodle, that, if you will work on > Wayland support or can help otherwise, you'll want to fill in: > > http://doodle.co

Re: Review Request 119355: Change use of MacroPushRequiredVars to CMakePushCheckState

2014-07-18 Thread Raphael Kubo da Costa
> On July 18, 2014, 2:57 p.m., Aleix Pol Gonzalez wrote: > > Is it available in 2.8.13? Otherwise we'd have to bump the dependency > > requirement to 3.0. It's actually been available since 2.8.6: http://cmake.org/gitweb?p=cmake.git;a=commit;h=1325260a664577a1356f3034ed5f110d45f6a6d8 - Rapha

Re: Review Request 119330: Convert FrameSvgItem to use 9 tiles instead of a big texture.

2014-07-18 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119330/#review62651 --- framesvgitem part starting to look good! I would still like to

Re: Review Request 119336: Convert FrameSvg to 9 textures: different approach

2014-07-18 Thread Marco Martin
> On July 18, 2014, 10:54 a.m., David Edmundson wrote: > > src/declarativeimports/core/framesvgitem.cpp, line 121 > > > > > > 1) This leaks. > > if you remove a node from a parent you have to delete it yourse

Re: Review Request 119355: Change use of MacroPushRequiredVars to CMakePushCheckState

2014-07-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119355/#review62649 --- Is it available in 2.8.13? Otherwise we'd have to bump the dep

Review Request 119355: Change use of MacroPushRequiredVars to CMakePushCheckState

2014-07-18 Thread Tobias Berner
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119355/ --- Review request for Plasma and Raphael Kubo da Costa. Repository: plasma-w

Re: Review Request 119330: Convert FrameSvgItem to use 9 tiles instead of a big texture.

2014-07-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119330/ --- (Updated July 18, 2014, 10:54 a.m.) Review request for Plasma. Changes

Re: Review Request 119336: Convert FrameSvg to 9 textures: different approach

2014-07-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119336/#review62619 --- src/declarativeimports/core/framesvgitem.cpp

Re: Review Request 119352: missing fclose - Resource leaking

2014-07-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119352/#review62640 --- Ship it! Good catch! - Aleix Pol Gonzalez On July 18, 2014

Re: 4.x transition blockage - workspace libs

2014-07-18 Thread Harald Sitter
On Thu, Jul 17, 2014 at 1:37 PM, Michael Palimaka wrote: >> I just discused this with mgraesslin on IRC and he's fine with adding >> a compatibility build flag to 4.x that makes it only install the >> necessary libraries to avoid file conflicts with plasma-workspace 5. >> >> Does that sounds suit

Re: Re: How to delay loading of a kded module

2014-07-18 Thread Martin Gräßlin
On Friday 18 July 2014 10:42:26 Jan Grulich wrote: > On Friday 18 of July 2014 10:37 Kai Uwe Broulik wrote: > > Hi, > > > > For notifications you could wait for org.freedesktop.Notifications to > > become available. We have a similar issue in PowerDevil where the "your > > battery is low" notifica

Re: How to delay loading of a kded module

2014-07-18 Thread Jan Grulich
On Friday 18 of July 2014 10:37 Kai Uwe Broulik wrote: > Hi, > > For notifications you could wait for org.freedesktop.Notifications to become > available. We have a similar issue in PowerDevil where the "your battery is > low" notification also spawns an ugly popup ontop of ksplash. > Yep, that

Re: How to delay loading of a kded module

2014-07-18 Thread Kai Uwe Broulik
Hi, For notifications you could wait for org.freedesktop.Notifications to become available. We have a similar issue in PowerDevil where the "your battery is low" notification also spawns an ugly popup ontop of ksplash. I guess same for org.kde.kwalletd? Cheers Am 18.07.2014 09:06 schrieb Jan

Plasma 5

2014-07-18 Thread Rob Merritt
Hi! Well, I just want to start by saying that I really am thoroughly blown away by the direction that KDE has taken. I use a variety of OS's for my day to day needs, but for my main laptop I use OS X. I have been waiting a long time to find a reason to move this machine to Linux - and I think I

Re: 4.x transition blockage - workspace libs

2014-07-18 Thread Bernd Steinhauser
On 09/07/14 16:32, Harald Sitter wrote: - libprocessui4 (used by kdevelop and sentinella) [additionally requires libprocesscore4] - libkworkspace4 (used by kget, ktorrent, ktux and apper) In order to build/run the mentioned applications one needs to have kde-workspace built and installed which t

How to delay loading of a kded module

2014-07-18 Thread Jan Grulich
Hi all, I would need help with delayed loading of our kded module due two problems when it's loaded too early: 1) When you are automatically connected to a network you will get an old notifications (the ugly one putted on top of the screen) because KDE notifications are not running at that tim

Review Request 119352: missing fclose - Resource leaking

2014-07-18 Thread Ömer Fadıl Usta
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119352/ --- Review request for Plasma. Repository: plasma-workspace Description ---