Re: Minutes Monday Plasma Hangout

On Mon, Jul 28, 2014 at 5:56 PM, Marco Martin wrote: > On Monday 28 July 2014, Marco Martin wrote: > > On Monday 28 July 2014, Sebastian Kügler wrote: > > > - Started mechanism to load stuff from Look and Feel packages > > > > > > would like comment on https://git.reviewboard.kde.org/r/119451/

Re: Minutes Monday Plasma Hangout

On Mon, Jul 28, 2014 at 9:36 PM, Marco Martin wrote: > On Monday 28 July 2014, Marco Martin wrote: > > > [Icons] > > > Theme=~/.share/plasma/lookandfeel/foo/icons > > > > comments on that? > > furthermore, how would you prefer to use it? > > as a standalone library? static in plasma-workspace stu

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

> On July 28, 2014, 7:03 p.m., Mark Gaiser wrote: > > If there is one hardware combination that works best on linux, it's OpenGL > > + Intel. > > I find it hard to believe that GLES is better and in all cases. That must > > be a driver/mesa bug that should be investigated and fixed. > > > > If

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

> On July 28, 2014, 6:25 p.m., Eike Hein wrote: > > Can you recompile plasma-desktop with OpenGL ES and test if the drag pixmap > > for Folder icons works? > > > > I wrote the code in a way that ought to work on both desktop GL and ES, but > > it's worth testing. I have it running with OpenGL

Re: qml controls and documentation

On Mon, Jul 28, 2014 at 8:07 PM, Marco Martin wrote: > On Monday 28 July 2014, Aleix Pol wrote: > > > > does somebody knows if is possible with the markup used now for > > > extracting documentation to still have comments to document all the > > > properties, even those *not* defined in the qml f

Re: Minutes Monday Plasma Hangout

On Monday 28 July 2014, Marco Martin wrote: > > [Icons] > > Theme=~/.share/plasma/lookandfeel/foo/icons > > comments on that? > furthermore, how would you prefer to use it? > as a standalone library? static in plasma-workspace stuff? in > libplasmaquick? other? next on that, the lookandfeel packa

Re: Review Request 119523: port TextField to QtControls

> On July 28, 2014, 6:54 p.m., David Edmundson wrote: > > src/declarativeimports/plasmacomponents/qml/TextField.qml, line 366 > > > > > > This is going to overlap with the text I think. > > It might be worth

Re: Review Request 119523: port TextField to QtControls

> On July 28, 2014, 7:09 p.m., Kai Uwe Broulik wrote: > > src/declarativeimports/plasmacomponents/qml/styles/TextFieldStyle.qml, line > > 52 > > > > > > Can't you account for the clear button width here? probabl

Re: Review Request 119523: port TextField to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119523/#review63367 --- src/declarativeimports/plasmacomponents/qml/styles/TextFieldS

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63366 --- If there is one hardware combination that works best on linux,

Re: Review Request 119523: port TextField to QtControls

On July 28, 2014, 6:54 p.m., Marco Martin wrote: > > Which methods do we need the inner control for? I saw positionAtPos and > > positionToRectangle, any others? > > Marco Martin wrote: > those two and passwordCharacter and missing properties are: passwordCharacter errorHighlight - Davi

Re: Review Request 119523: port TextField to QtControls

> On July 28, 2014, 6:54 p.m., David Edmundson wrote: > > src/declarativeimports/plasmacomponents/qml/TextField.qml, line 366 > > > > > > This is going to overlap with the text I think. > > It might be worth

Re: Review Request 119523: port TextField to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119523/#review63363 --- Even if we fix the methods it's too close to 5.1 to merge righ

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63362 --- Can you recompile plasma-desktop with OpenGL ES and test if th

Re: Review Request 119524: Force OpenGLES if on Qt 5.4

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/#review63360 --- I still don't have 5.4 to test for now.. but, are you sure thi

Review Request 119523: port TextField to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119523/ --- Review request for KDE Frameworks and Plasma. Repository: plasma-framewor

Review Request 119524: Force OpenGLES if on Qt 5.4

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/ --- Review request for Plasma. Repository: plasma-workspace Description ---

Re: qml controls and documentation

On Monday 28 July 2014, Aleix Pol wrote: > > does somebody knows if is possible with the markup used now for > > extracting documentation to still have comments to document all the > > properties, even those *not* defined in the qml file in question? > > It should probably look more or less like

Re: Review Request 119515: Support per-activity favourites

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119515/ --- (Updated July 28, 2014, 5:50 p.m.) Status -- This change has been ma

Re: Review Request 119522: readlink fails whens the folder does not exist.

> On July 28, 2014, 11:14 p.m., Aleix Pol Gonzalez wrote: > > So does it still work if it's a link, after this patch? Yep, works when my ~/.config is a symlink. - Rohan --- This is an automatically generated e-mail. To reply, visit: htt

Re: Review Request 119465: Have separate texture hashes for each window

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119465/ --- (Updated July 28, 2014, 5:48 p.m.) Status -- This change has been ma

Re: Review Request 119515: Support per-activity favourites

> On July 28, 2014, 5:31 p.m., Thomas Braxton wrote: > > applets/kickoff/package/contents/ui/ContextMenu.qml, line 62 > > > > > > addToGlobalFavorites sounds better Agreed. - Ivan

Re: Review Request 119522: readlink fails whens the folder does not exist.

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119522/#review63351 --- So does it still work if it's a link, after this patch? - Ale

Re: Review Request 119515: Support per-activity favourites

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119515/#review63349 --- applets/kickoff/package/contents/ui/ContextMenu.qml

Re: Review Request 119515: Support per-activity favourites

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119515/#review63347 --- Ship it! for kickoff, for what i'm concerned is good to go -

Review Request 119522: readlink fails whens the folder does not exist.

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119522/ --- Review request for Plasma and Àlex Fiestas. Repository: plasma-workspace

Re: Review Request 119515: Support per-activity favourites

> On July 28, 2014, 12:03 p.m., Marco Martin wrote: > > +1 from me, would like toknow from Eike if this provides all is needed for > > use cases outside kickoff > > Eike Hein wrote: > Kicker doesn't use Kickoff's models, i.e. activity support has to be > implemented in Kicker's own favorit

Re: Review Request 119464: port ToolButton to QtControls

> On July 28, 2014, 4:40 p.m., David Edmundson wrote: > > >The ugly thing here is that QtControls ToolButton doesn't have a flat > > >property, it should probably be upstreamed, at least for compatibility > > >with the qwidgets ToolButton > > > > I can't see a flat property on the widget's too

Re: Review Request 119464: port ToolButton to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119464/#review63340 --- >The ugly thing here is that QtControls ToolButton doesn't hav

Running Plasma on Qt 5.4

Hi, Just in case somebody has the magic idea of trying Qt 5.4, bear in mind that it's currently not working (or, well, working very bad). Here's a patch if you really feel like trying it [1] which seems to work. Cheers! Aleix [1] https://codereview.qt-project.org/#/c/90811/ __

Re: qml controls and documentation

On Mon, Jul 28, 2014 at 6:11 PM, Marco Martin wrote: > Hi all, > > right now the plasma qml components are documented not bad, like: > > http://api.kde.org/frameworks-api/frameworks5-apidocs/plasma-framework/html/classorg_1_1kde_1_1plasma_1_1components_1_1TextField.html > > however, porting them

qml controls and documentation

Hi all, right now the plasma qml components are documented not bad, like: http://api.kde.org/frameworks-api/frameworks5-apidocs/plasma-framework/html/classorg_1_1kde_1_1plasma_1_1components_1_1TextField.html however, porting them to QtControls, has the sideeffect of pretty much killing the docum

Re: Minutes Monday Plasma Hangout

On Monday 28 July 2014, Marco Martin wrote: > On Monday 28 July 2014, Sebastian Kügler wrote: > > - Started mechanism to load stuff from Look and Feel packages > > > > would like comment on https://git.reviewboard.kde.org/r/119451/ > > Based on the discussion with David in the hangout, i would

Re: Review Request 119464: port ToolButton to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119464/ --- (Updated July 28, 2014, 3:50 p.m.) Status -- This change has been ma

Re: Review Request 119464: port ToolButton to QtControls

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119464/#review63337 --- Ship it! Ship It! - David Edmundson On July 25, 2014, 4:52

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

On July 19, 2014, 12:17 a.m., Vadim Zhukov wrote: > > (As a general note, for build system related stuff like this you can also > > try including the "buildsystem" group, which can be more responsive at > > times) > > > > > The ibus-panel can't build on OpenBSD because some required definition

Re: port plasma to imx6 device (cubox-i)

On Mon, Jul 21, 2014 at 2:32 PM, Malte Grosse wrote: > Dear plasma team, > > ive seen a review about the new plasma 5. > i really like the design - it looks really nice. > my question is - if anybody is interested in porting the new plasma to the > cubox-i? > the cubox i is a small arm device (fr

Re: Using more of our CI, Coverage and cppcheck

On Sun, Jul 27, 2014 at 11:59 PM, Àlex Fiestas wrote: > Hi there! > > During this weekend Ben and I have enabled the code-coverage in plasma- > framework, my personal idea is to enable it in most projects. > > We should not go loco with code coverage, we stand where we stand and > adding > test n

Re: Review Request 119515: Support per-activity favourites

> On July 28, 2014, 12:03 p.m., Marco Martin wrote: > > +1 from me, would like toknow from Eike if this provides all is needed for > > use cases outside kickoff Kicker doesn't use Kickoff's models, i.e. activity support has to be implemented in Kicker's own favorites model. - Eike

Re: Minutes Monday Plasma Hangout

On Monday 28 July 2014, Sebastian Kügler wrote: > - Started mechanism to load stuff from Look and Feel packages > would like comment on https://git.reviewboard.kde.org/r/119451/ Based on the discussion with David in the hangout, i would still do: - the look and feel package provides qml files fo

Re: Review Request 119515: Support per-activity favourites

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119515/#review63321 --- +1 from me, would like toknow from Eike if this provides all i

Review Request 119515: Support per-activity favourites

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119515/ --- Review request for Plasma, Eike Hein, Marco Martin, and Sebastian Kügler.

Re: Review Request 119513: reintroduce containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, args)

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119513/ --- (Updated July 28, 2014, 11:46 a.m.) Status -- This change has been m

Re: Review Request 119513: reintroduce containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, args)

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119513/#review63318 --- Ship it! Ship It! - Sebastian Kügler On July 28, 2014, 11:

Review Request 119513: reintroduce containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, args)

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119513/ --- Review request for KDE Frameworks and Plasma. Repository: plasma-framewor

Minutes Monday Plasma Hangout

Minutes Plasma Hangout week 31, 28th July 2014 Present: Antonis, Marco, David, Ivan, Jonathan, Vishesh, Sebastian, Aleix, Aleix: - worked on texture sharing in FrameSvg Antonis: - Worked on widgetexplorer in Active shell - Worked on main containment for Active - both need more work David: - Wo

Re: Review Request 119465: Have separate texture hashes for each window

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119465/#review63313 --- src/declarativeimports/core/framesvgitem.cpp

systemtray initial configuration

Hello, I'm trying to alter default configuration of systemtray applet. I want to remove device notifier ("notifier") from there, but keep "org.kde.notifications" and others. The problem is that whenever I touch systemtray configuration (even simple reloadConfig call), org.kde.notifications widget

port plasma to imx6 device (cubox-i)

Dear plasma team, ive seen a review about the new plasma 5. i really like the design - it looks really nice. my question is - if anybody is interested in porting the new plasma to the cubox-i? the cubox i is a small arm device (freescale imx6) http://www.solid-run.com/wiki/CuBox-i_Hardware the com