Re: qml controls and documentation

2014-08-01 Thread Aurélien Gâteau
On Mon, Jul 28, 2014, at 15:26, Aleix Pol wrote: > 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 al

Re: Review Request 118665: Avoid invalid read in ShadowHelper destructor

2014-06-19 Thread Aurélien Gâteau
hell5 desktop" does not print any error about invalid reads in ~ShadowHelper anymore. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 118662: fix plugin install target for phonon platform plugin

2014-06-18 Thread Aurélien Gâteau
now. - Aurélien Gâteau On June 11, 2014, 10:51 a.m., Harald Sitter wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 118665: Avoid invalid read in ShadowHelper destructor

2014-06-13 Thread Aurélien Gâteau
sktop" does not print any error about invalid reads in ~ShadowHelper anymore. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 118665: Avoid invalid read in ShadowHelper destructor

2014-06-11 Thread Aurélien Gâteau
changes are much more invasive. Diffs - kstyle/oxygenstyle.cpp 972e4c8 Diff: https://git.reviewboard.kde.org/r/118665/diff/ Testing --- Running "valgrind kcmshell5 desktop" does not print any error about invalid reads in ~ShadowHelper anymore. Thanks, Aurél

Re: Review Request 117954: Port keditfiletype away from kde4support

2014-05-28 Thread Aurélien Gâteau
97cd3c1 Diff: https://git.reviewboard.kde.org/r/117954/diff/ Testing --- Program starts and looks like it behaves correctly. Unit-test is failing, but was already failing before :/ Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel

Re: Review Request 117954: Port keditfiletype away from kde4support

2014-05-23 Thread Aurélien Gâteau
> On May 20, 2014, 5: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 &

Re: Review Request 117954: Port keditfiletype away from kde4support

2014-05-23 Thread Aurélien Gâteau
05e425a keditfiletype/typeslistitem.cpp 97cd3c1 Diff: https://git.reviewboard.kde.org/r/117954/diff/ Testing --- Program starts and looks like it behaves correctly. Unit-test is failing, but was already failing before :/ Thanks, Aurélien Gâteau

Re: Review Request 117954: Port keditfiletype away from kde4support

2014-05-22 Thread Aurélien Gâteau
54/#review58199 --- On May 20, 2014, 4:56 p.m., Aurélien Gâteau wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.r

Re: Review Request 117954: Port keditfiletype away from kde4support

2014-05-20 Thread Aurélien Gâteau
://git.reviewboard.kde.org/r/117954/diff/ Testing --- Program starts and looks like it behaves correctly. Unit-test is failing, but was already failing before :/ Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https

Review Request 117954: Port keditfiletype away from kde4support

2014-05-02 Thread Aurélien Gâteau
behaves correctly. Unit-test is failing, but was already failing before :/ Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 117884: rename binaries for co-installability

2014-05-02 Thread Aurélien Gâteau
getsAddons to KIO According to LXR KMimeTypeChooser is only used in KIO, so this change would work. The problem is this would break source compatibility, and now that we are close to the release date I don't think Kevin or David would allow that. - Aurélien Gâteau On April 29, 2014, 6:16

Re: Review Request 117464: [kglobalaccel] Remove notification support

2014-04-10 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117464/#review55327 --- Ship it! Ship It! - Aurélien Gâteau On April 10, 2014, 8

Re: Review Request 117464: [kglobalaccel] Remove notification support

2014-04-10 Thread Aurélien Gâteau
> On April 10, 2014, 9:17 a.m., Aurélien Gâteau wrote: > > Looks good, but patch does not apply here. It fails with: > > > > error: kglobalaccel/CMakeLists.txt: does not exist in index > > error: kglobalaccel/globalshortcutsregistry.cpp: does not exist in in

Re: Review Request 117464: [kglobalaccel] Remove notification support

2014-04-10 Thread Aurélien Gâteau
: does not exist in index - Aurélien Gâteau On April 10, 2014, 8:20 a.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: "Plasma Next + Apps" view on build.kde.org

2014-03-19 Thread Aurélien Gâteau
On Wed, Mar 19, 2014, at 3:21, Martin Gräßlin wrote: > On Wednesday 19 March 2014 03:07:27 Aurélien Gâteau wrote: > > Just a quick note to let you know I created a new view on build.kde.org > > which groups the jobs from Plasma Next and the available KF5 > > applications.

"Plasma Next + Apps" view on build.kde.org

2014-03-19 Thread Aurélien Gâteau
Just a quick note to let you know I created a new view on build.kde.org which groups the jobs from Plasma Next and the available KF5 applications. You can find it here: http://build.kde.org/view/Plasma%20Next%20+%20Apps/ I hope it helps you tracking the state of our code. Feel free to extend it w

Re: Review Request 116714: Use CMake config file to find dbusmenu-qt5

2014-03-12 Thread Aurélien Gâteau
/ Testing --- Rebuilt kde-workspace, dbusmenu-qt5 is found. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 116714: Use CMake config file to find dbusmenu-qt5

2014-03-12 Thread Aurélien Gâteau
nerated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116714/#review52657 --- On March 11, 2014, 3:12 p.m., Aurélien Gâteau wrote: > > --- > This is an automatically generated e-mail. To

Review Request 116714: Use CMake config file to find dbusmenu-qt5

2014-03-11 Thread Aurélien Gâteau
/generic/dataengines/CMakeLists.txt 187c24f appmenu/CMakeLists.txt 7e166e7 cmake/modules/FindDBusMenuQt5.cmake 7d43489 CMakeLists.txt 271e7d5 Diff: https://git.reviewboard.kde.org/r/116714/diff/ Testing --- Rebuilt kde-workspace, dbusmenu-qt5 is found. Thanks, Aurélien Gâteau

Re: Review Request 116599: Set minimum version of Wayland to 1.2

2014-03-05 Thread Aurélien Gâteau
://git.reviewboard.kde.org/r/116598/ is applied. Diffs - CMakeLists.txt 41923b6 Diff: https://git.reviewboard.kde.org/r/116599/diff/ Testing --- Fixes the build for me (Kubuntu Saucy, libwayland-client 1.1, no libwayland-egl) Thanks, Aurélien Gâteau

Review Request 116599: Set minimum version of Wayland to 1.2

2014-03-04 Thread Aurélien Gâteau
://git.reviewboard.kde.org/r/116599/diff/ Testing --- Fixes the build for me (Kubuntu Saucy, libwayland-client 1.1, no libwayland-egl) Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 114932: Use QExplicitlySharedDataPointer as the replacement of KSharedPtr

2014-01-14 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114932/#review47363 --- Ship it! OK, go for it then. - Aurélien Gâteau On Jan. 9

Re: Review Request 114932: Use QExplicitlySharedDataPointer as the replacement of KSharedPtr

2014-01-13 Thread Aurélien Gâteau
. I didn't know about QSharedData::ref() (I blame the lack of doc). I am curious about the performance issue you mention, can you expand on it? - Aurélien Gâteau On Jan. 9, 2014, 8:14 p.m., Alexander Richardson wrote: > > --- &

Re: Review Request 114912: KSharedPtr => QWeakPointer and QSharedPointer

2014-01-09 Thread Aurélien Gâteau
(). Diffs - CMakeLists.txt 8b60011 src/plasma/CMakeLists.txt aced599 src/plasma/private/svg_p.h 5fdebf3 src/plasma/svg.cpp 6542b34 Diff: https://git.reviewboard.kde.org/r/114912/diff/ Testing --- Ran a plasma-shell session, did not notice any crash. Thanks, Aurélien

Re: Review Request 114912: KSharedPtr => QWeakPointer and QSharedPointer

2014-01-08 Thread Aurélien Gâteau
/CMakeLists.txt aced599 src/plasma/private/svg_p.h 5fdebf3 src/plasma/svg.cpp 6542b34 Diff: https://git.reviewboard.kde.org/r/114912/diff/ Testing (updated) --- Ran a plasma-shell session, did not notice any crash. Thanks, Aurélien Gâteau

Review Request 114912: KSharedPtr => QWeakPointer and QSharedPointer

2014-01-08 Thread Aurélien Gâteau
/private/svg_p.h 5fdebf3 src/plasma/svg.cpp 6542b34 Diff: https://git.reviewboard.kde.org/r/114912/diff/ Testing --- Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Framework licenses (was: Refocusing the tech preview)

2013-12-23 Thread Aurélien Gâteau
Le lundi 23 décembre 2013 18:30:11 Kevin Ottens a écrit : > > Missing COPYING.LIB files are more annoying, though. Right now they are > > missing in kf5umbrella, kmediaplayer and plasma-framework (but I assume > > plasma-framework is not part from the tp1?) I am going to add them this > > afternoon

Re: Review Request 113433: Use FindDBusMenuQt5.cmake

2013-10-30 Thread Aurélien Gâteau
is detected and the statusnotifieritem dataengine builds. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 113433: Use FindDBusMenuQt5.cmake

2013-10-25 Thread Aurélien Gâteau
/generic/dataengines/CMakeLists.txt 509e24e plasma/generic/dataengines/statusnotifieritem/CMakeLists.txt 14ee673 Diff: http://git.reviewboard.kde.org/r/113433/diff/ Testing --- Qt5 version of dbusmenu-qt is detected and the statusnotifieritem dataengine builds. Thanks, Aurélien Gâteau

Re: Review Request 113086: KDeclarative cleanup

2013-10-04 Thread Aurélien Gâteau
<http://git.reviewboard.kde.org/r/113086/#comment30235> This is not exactly the same: if I am not mistaken, qWarning() will add extra spaces and maybe quotes around the '\n'. - Aurélien Gâteau On Oct. 3, 2013, 10:02 p.m., Al

Re: Fwd: Reminder: use KF5::foo instead of ${foo_LIBRARIES} in CMakeLists

2013-09-25 Thread Aurélien Gâteau
On Wednesday 25 September 2013 11:22:57 Sebastian Kügler wrote: > CMake-gods, can you confirm the below? (It's inconsistent with my > understanding, and how we've done it in the past months, I'd like to have a > specialist opinion before going around and changing every single > CMakeLists.txt in Pl

Re: [homerun] Repeated Entries

2013-07-23 Thread Aurélien Gâteau
Le jeudi 18 juillet 2013 16:12:32 Sebastian Kügler a écrit : > On Wednesday, July 17, 2013 17:36:43 Jacky Alcine wrote: > > Hey Plasma developers, Jacky here. > > > > I'm curious about how Homerun goes about rendering entries. I've noticed > > constantly that entries shown in "Applications" would

Re: [RFC] Disable DrKonqi for KWin in stable releases

2013-04-30 Thread Aurélien Gâteau
Le Tuesday 30 April 2013 13:37:51 Martin Gräßlin a écrit : > The nice side effect would be that it would look like improved quality as > normaly a user wouldn't notice a KWin crash. It's a short flicker during > which the window decorations got recrated, but that's it. Has there been a > crash if

Review Request 110208: Fix memleak: make ~DialogShadows delete DialogShadows::Private

2013-04-26 Thread Aurélien Gâteau
--- Ran Plasma, without crash, tooltips still work. Ran upcoming, Plasma::Dialog-based, Colibri: the change makes Valgrind a bit happier. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma

Review Request 109946: Fix crash in SortFilterModel::mapRowFromSource()

2013-04-10 Thread Aurélien Gâteau
. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Potential new component for Plasma QML components: ShadowEffect

2013-04-04 Thread Aurélien Gâteau
> On Thursday 04 April 2013, Aurélien Gâteau wrote: >> Hi, >> >> I would like to propose a new component for Plasma QML components: >> ShadowEffect. >> >> This component has been created for Homerun and is used to show a shadow >> behind texts whe

Potential new component for Plasma QML components: ShadowEffect

2013-04-04 Thread Aurélien Gâteau
Hi, I would like to propose a new component for Plasma QML components: ShadowEffect. This component has been created for Homerun and is used to show a shadow behind texts when Homerun is used as a containment. It is similar to the QGraphicsDropShadowEffect which comes with Qt, but provides a mor

Preparing release of Homerun 0.2.0

2013-02-23 Thread Aurélien Gâteau
Hi, I am planning a release of Homerun 0.2.0 on Tuesday, February 26th. That's quite soon, but I need to get it out in time for Kubuntu Raring feature-freeze, which is on March 7th (and to make things a bit more challenging I am leaving for vacations on the first week of March, so next week real

Re: Homerun and single-runner runners

2013-02-13 Thread Aurélien Gâteau
Le Wednesday 13 February 2013 15:02:33 Aaron J. Seigo a écrit : > > 2. The way single-runners are queried is not very elegant. It looks like > > > this: > this probably should be provided by RunnerManager and RunnerSyntax. sth > like: > > QString RunnerSyntax::asQuery(const QString &userInput = Q

Re: Homerun and single-runner runners

2013-02-13 Thread Aurélien Gâteau
Le Wednesday 13 February 2013 14:10:40 Marco Martin a écrit : > On Wednesday 13 February 2013, Aurélien Gâteau wrote: > > >> 3. I noticed some runners advertise single-runner mode but do not > > >> define a > > >> default syntax. Is it a bug or is th

Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 13:50:08 +0100, Mark wrote: I'm just going to assume you're making a joke here. In case you're not: bad idea! Do you have arguments to back this bold claim? Sure. I've opened this topic to improve the pager for default usage: - Show it even when you have just one VD to f

Re: Re: Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 13:44:31 +0100, Martin Gräßlin wrote: On Wednesday 13 February 2013 06:43:47 Aurélien Gâteau wrote: On Wed, 13 Feb 2013 12:23:41 +0100, Martin Gräßlin wrote: > On Wednesday 13 February 2013 06:17:45 Aurélien Gâteau wrote: >> Since the virtual desktop kcm is alre

Re: Homerun and single-runner runners

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 12:45:33 +0100, Marco Martin wrote: if (query.contains(placeHolder)) { return query.replace(placeHolder, term); } else { return query + ' ' + term; } } Is there a better way to do this? uhm... RunnerManager::launchQuery(const QString &term, cons

Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 13:11:13 +0100, Mark wrote: On Wed, Feb 13, 2013 at 12:17 PM, Aurélien Gâteau wrote: On Wed, 13 Feb 2013 11:40:15 +0100, Marco Martin wrote: On Wednesday 13 February 2013, Mark wrote: That would probably be the best way i guess. But then we still have an empty taskbar

Re: Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 12:23:41 +0100, Martin Gräßlin wrote: On Wednesday 13 February 2013 06:17:45 Aurélien Gâteau wrote: Since the virtual desktop kcm is already shown from within the pager applet configuration dialog, maybe it can simply be removed from systemsettings? erm no, I'm

Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
On Wed, 13 Feb 2013 11:40:15 +0100, Marco Martin wrote: On Wednesday 13 February 2013, Mark wrote: That would probably be the best way i guess. But then we still have an empty taskbar area occupied by the pager during the 4.10 cycle. I can hardly imagine that to be the intention? It's just a

Review Request 108938: Make "Kate session" runner support "single runner" mode

2013-02-13 Thread Aurélien Gâteau
8938/diff/ Testing --- Tested with upcoming single-runner support in Homerun, as mentioned on plasma-devel mailing-list. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
Le Wednesday 13 February 2013 11:04:06 Martin Gräßlin a écrit : > On Wednesday 13 February 2013 10:58:09 Mark wrote: > > On Wed, Feb 13, 2013 at 10:38 AM, Aurélien Gâteau wrote: > > > Le Tuesday 12 February 2013 23:55:39 Mark a écrit : [snip] > > >> Why are y

Homerun and single-runner runners

2013-02-13 Thread Aurélien Gâteau
Hi, I have been working lately on adding better support for single-runner runners to Homerun. The code is currently available in the "wip/single-runner" branch of Homerun. This code exposes all runners which advertise single-runner mode as sources. When a single-runner is used, Homerun uses th

Re: QML Placces design discussion

2013-02-13 Thread Aurélien Gâteau
Le Tuesday 12 February 2013 23:55:39 Mark a écrit : > On Tue, Feb 12, 2013 at 8:06 PM, Shaun Reich wrote: > > On Tue, Feb 12, 2013 at 10:40 AM, Martin Gräßlin wrote: > >> Technical implementation aside: I think that makes absolute sense: > >> * one vd by default > >> * do not show pager for one

Review Request 108922: Fix off-by-one error in RunnerModel

2013-02-12 Thread Aurélien Gâteau
/diff/ Testing --- Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 107600: Keep placeholder text visible when empty and focused

2013-02-06 Thread Aurélien Gâteau
nhappy about having different behaviours between > > the "same" Qt and KDE widgets. > > Aurélien Gâteau wrote: > Makes sense. I am planning on providing a patch for Qt anyway. In the > long run we should even be able to replace KLineEdit::clickMessage > impleme

Re: Review Request 108700: Make sure Menu.status is set to Closed when the menu closes itself

2013-02-05 Thread Aurélien Gâteau
b as the first word. Would "markAsClosed()" work for you? - Aurélien --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108700/#review26519 ------

Review Request 108700: Make sure Menu.status is set to Closed when the menu closes itself

2013-02-01 Thread Aurélien Gâteau
/diff/ Testing --- Run attached testmenu.qml program. Without the patch, the text says "Opened" even after closing the menu. File Attachments Test program http://git.reviewboard.kde.org/media/uploaded/files/2013/02/01/testmenu.qml Thanks, Aurél

Re: Screen Edge handling in 4.11

2013-01-21 Thread Aurélien Gâteau
Le Monday 21 January 2013 12:37:04 Marco Martin a écrit : > On Monday 21 January 2013, Martin Graesslin wrote: > > > I take it this system will be implemented in a library. Do you plan for > > > a > > > fallback plan, in case the system is using another window manager? For > > > example when runnin

Re: Screen Edge handling in 4.11

2013-01-21 Thread Aurélien Gâteau
Le Monday 21 January 2013 11:03:53 Martin Gräßlin a écrit : > Hi workspace devs, > > I just finished a rewrite of the Screen Edge handling in KWin [1] and now I > want to tackle one of the long standing issues: hidden panel activation. For > those who do not know the plans first designed years ago

Re: Review Request 108422: Add configuration page for QML-based containments

2013-01-15 Thread Aurélien Gâteau
.kde.org/r/108422/diff/ Testing --- Tested with the text-shadows branch of kde:homerun. Also used and configured other non-QML containment like Folder View to ensure the change does not introduce regressions. Thanks, Aurélien Gâteau ___ Plasma-dev

Review Request 108423: Call Applet::configChanged() in Containment::configChanged()

2013-01-15 Thread Aurélien Gâteau
Testing --- Tested with the text-shadows branch of kde:homerun, settings are correctly applied when the "Apply" button is clicked. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mai

Review Request 108422: Add configuration page for QML-based containments

2013-01-15 Thread Aurélien Gâteau
regressions. Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Follow up on doxyqml

2012-12-23 Thread Aurélien Gâteau
Le Thursday 20 December 2012 19:09:39 Aurélien Gâteau a écrit : > Hi, > > Doxyqml, the Doxygen input filter for QML, has been deployed on api.kde.org. > I then converted the doc of most QML components of kde- > runtime/plasma/declarativeimport to use Doxygen (You may have notice

Follow up on doxyqml

2012-12-20 Thread Aurélien Gâteau
Hi, Doxyqml, the Doxygen input filter for QML, has been deployed on api.kde.org. I then converted the doc of most QML components of kde- runtime/plasma/declarativeimport to use Doxygen (You may have noticed a series of commits with messages like "Doxygenize Foo"). The only components I haven't

Re: Review Request: Keep placeholder text visible when empty and focused

2012-12-14 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107600/#review23416 --- On Dec. 12, 2012, 5:11 p.m., Aurélien Gâteau wrote: > > -

Re: Review Request: Keep placeholder text visible when empty and focused

2012-12-12 Thread Aurélien Gâteau
, as can be seen in attached screenshot. Screenshots --- TextField with placeholder from Widget Gallery http://git.reviewboard.kde.org/r/107600/s/872/ Thanks, Aurélien Gâteau ___ Plasma-devel mailing list Plasma-devel@kde.org https

Re: Review Request: Keep placeholder text visible when empty and focused

2012-12-10 Thread Aurélien Gâteau
I am going to file merge requests for those as well. - Aurélien --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107600/#review23194 -----

Review Request: Keep placeholder text visible when empty and focused

2012-12-05 Thread Aurélien Gâteau
/107600/diff/ Testing --- Ran widgetgallery with the patch code, as can be seen in attached screenshot. Screenshots --- TextField with placeholder from Widget Gallery http://git.reviewboard.kde.org/r/107600/s/872/ Thanks, Aurélien Gâteau

Re: Homerun sources and Plasma runners

2012-11-26 Thread Aurélien Gâteau
Le jeudi 22 novembre 2012 13:29:06 Aaron J. Seigo a écrit : > > Homerun AbstractSource class is not a model. It inherits from QObject and > > provide virtual methods to create models. A source must implement at > > least: > > "QAbstractItemModel *createModelFromConfigGroup(const KConfigGroup > > &

Re: Doxygen and QML on api.kde.org

2012-11-22 Thread Aurélien Gâteau
Le mercredi 21 novembre 2012 12:11:20 Allen Winter a écrit : > You should announce this on the doxygen-users mailing list > Or I can do that if you want. It probably makes more sense if I am the one to do it, so that I can answer questions. > > Will let you know when I'm finished installing on a

Re: Homerun sources and Plasma runners

2012-11-22 Thread Aurélien Gâteau
Le mercredi 21 novembre 2012 20:52:30 Marco Martin a écrit : > On Wednesday 21 November 2012, Aurélien Gâteau wrote: > > Trying to map the way sources work to runners, instead of having a runner > > calling RunnerContext::addMatches(), a runner would have a createModel() > >

Doxygen and QML on api.kde.org

2012-11-21 Thread Aurélien Gâteau
Hi Allen, I wrote a Doxygen input filter for QML, making it possible to document QML classes just like we document C++ classes. You can find it here: http://agateau.com/projects/doxyqml/ Plasma developers are interested in using this to document Plasma QML components. Do you think we can get d

Re: Homerun sources and Plasma runners

2012-11-21 Thread Aurélien Gâteau
Le mardi 20 novembre 2012 20:16:17 Aaron J. Seigo a écrit : > On Tuesday, November 20, 2012 18:12:47 Aurélien Gâteau wrote: > > First, I am convinced using Qt models is the right API design for Homerun. > > I > nobody is saying "don't use Qt models in homerun". My

Re: Homerun sources and Plasma runners

2012-11-20 Thread Aurélien Gâteau
Spent yesterday evening trying to think about this topic from all edges. First, I am convinced using Qt models is the right API design for Homerun. I have a very hard time giving up on them for the sake of not breaking source compatibility, to the point where I think it is better to do a clean b

Re: Homerun sources and Plasma runners

2012-11-19 Thread Aurélien Gâteau
Le samedi 17 novembre 2012 12:56:13 Aaron J. Seigo a écrit : > On Friday, November 16, 2012 17:04:04 Aurélien Gâteau wrote: > > A preliminary subject is API compatibility: I don't think it is possible > > to extend the current AbstractRunner interface in a BC way to meet our

Re: Homerun sources and Plasma runners

2012-11-19 Thread Aurélien Gâteau
Le lundi 19 novembre 2012 14:58:40 Aurélien Gâteau a écrit : > Le vendredi 16 novembre 2012 18:16:12 Marco Martin a écrit : > > > Sources features missing in Runners: > > > - model-based => more QML friendly > > > > I still think that wrapped in a model or

Re: Homerun sources and Plasma runners

2012-11-19 Thread Aurélien Gâteau
Le vendredi 16 novembre 2012 18:16:12 Marco Martin a écrit : > > Sources features missing in Runners: > > - model-based => more QML friendly > > I still think that wrapped in a model or not is a detail. > still don't see what advantages would bring the api being a model, rather a > model being a c

Homerun sources and Plasma runners

2012-11-16 Thread Aurélien Gâteau
Hi all, Yesterday afternoon, Aaron, Alex and I discussed Homerun sources, Plasma runners (and a few other topics). We agreed it would be better to have Homerun use Runners instead of proposing its own plugin API and eliminate as much as possible of the forked code. # Closing the gap between Runn

Re: Introducing Homerun

2012-11-15 Thread Aurélien Gâteau
Le jeudi 15 novembre 2012 10:19:11 Marco Martin a écrit : > On Wednesday 14 November 2012, Aurélien Gâteau wrote: > > > I see that listview has a section property that would be exactly what is > > > needed, while gridview doesn't, meh > > > > Agreed,

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mercredi 14 novembre 2012 16:31:19 Aaron J. Seigo a écrit : > On Wednesday, November 14, 2012 15:05:09 Aurélien Gâteau wrote: > > Le mercredi 14 novembre 2012 12:33:55 Aaron J. Seigo a écrit : > > > On Wednesday, November 14, 2012 11:30:58 Aurélien Gâteau wrote: > >

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mercredi 14 novembre 2012 15:26:03 Marco Martin a écrit : > On Wednesday 14 November 2012, Aurélien Gâteau wrote: > > > if you need to be able to group by runners, add that to the existing > > > implementation. how does the model in homerun facilitate grouping by > &

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mardi 13 novembre 2012 16:59:59 Alex Fiestas a écrit : > I see that at the moment it has the same kickoff sections which makes sense > for an initial release. > > Have you thought on any alternative? I would like the "Files" tab to become something which shows all recent document activity. >

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mercredi 14 novembre 2012 12:53:11 Marco Martin a écrit : > On Wednesday 14 November 2012, Aurélien Gâteau wrote: > > > krunnermodel, a Qt model that comes with a QML component plugin (so you > > > can just import and use it directly) is in > > > kde-runtime/plasma

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mercredi 14 novembre 2012 12:33:55 Aaron J. Seigo a écrit : > On Wednesday, November 14, 2012 11:30:58 Aurélien Gâteau wrote: > > Le mardi 13 novembre 2012 11:23:00 Aaron J. Seigo a écrit : > > > krunnermodel, a Qt model that comes with a QML component plugin (so you > >

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mardi 13 novembre 2012 17:58:24 Oliver Henshaw a écrit : > This discussion reminded me of > https://blog.mozilla.org/faaborg/2009/10/13/browsing-your-personal-web/which > is about search & browse interfaces. Hope it's food for thought for anyone > who hasn't seen it before - it certainly struck

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le lundi 12 novembre 2012 17:21:17 Nuno Pinheiro a écrit : > Don't take my criticism the wrong way. No worry :) [snip] > With all of that said allot of people use their desktop in deeply different > ways than what I do, so :) probably allot of people will like it so ;) > take what I say th

Re: Introducing Homerun

2012-11-14 Thread Aurélien Gâteau
Le mardi 13 novembre 2012 11:23:00 Aaron J. Seigo a écrit : > > > > One of the main differences between Homerun sources and Plasma runners > > > > is > > > > that sources can provide default content > > > > > > as can runners (for many releases now). a Runner can define its default > > > syntax, a

Re: Introducing Homerun

2012-11-12 Thread Aurélien Gâteau
Le dimanche 11 novembre 2012 12:02:58 Aaron J. Seigo a écrit : > On Friday, November 9, 2012 23:09:35 Aurélien Gâteau wrote: > > As you can see, it is a fullscreen launcher with content organized in > > tabs. > > i find it to be a very interesting approach, and one that

Introducing Homerun

2012-11-09 Thread Aurélien Gâteau
Hi all, Some of you have already heard about Homerun, but it has never been properly announced on this mailing list. Time to fix this. # What is Homerun? Homerun is an alternative launcher for KDE Plasma, built using QML Plasma Components. It has been developed by Shaun Reich and I during the

Re: QML style guide

2012-10-30 Thread Aurélien Gâteau
Le mardi 30 octobre 2012 18:16:26 Aurélien Gâteau a écrit : > The only part I don't really like is putting left curly brace on its own > line for functions because: > > - having different conventions for event-handlers and function code blocks > look inconsistent > &

Re: QML style guide

2012-10-30 Thread Aurélien Gâteau
Le lundi 29 octobre 2012 21:42:18 Aaron J. Seigo a écrit : > hi.. > > we have a style guide for our C++ code: the kdelibs style, which was mostly > borrowed from Qt. unfortunately for us there was no QML guide we could > similarly abscond with. > > and it shows. > > summary -> i've started putti

Re: Doxyqml, supporting type

2012-10-23 Thread Aurélien Gâteau
Le mardi 23 octobre 2012 00:23:25 Aleix Pol a écrit : > I think you are great Aurélien :D *blushes* :) > > More seriously though, I think that a patch on some current code would > be really helpful. Sure. When this is deployed, I want to go through the current documentation and split it accordi

Doxyqml, supporting type

2012-10-22 Thread Aurélien Gâteau
Hi, I have been working a bit more on Doxyqml, the Doxygen input filter for QML [1]. It now uses a more robust parser, is growing a test-suite and can override types when needed, allowing you to specify types for function parameters, return values and properties. Here is an excerpt of the READ

Re: Doxygen for QML components

2012-10-17 Thread Aurélien Gâteau
Le mercredi 17 octobre 2012 16:20:26 Aleix Pol a écrit : > > The crude, done in 10 minutes one: > > Define a custom comment to set on the property alias to document the type. > > > > Something like this: > > property alias text: label.text // doxyqml: type=string > > > > I consider the second o

Re: Doxygen for QML components

2012-10-17 Thread Aurélien Gâteau
Le mardi 16 octobre 2012 21:39:15 Marco Martin a écrit : > On Tuesday 16 October 2012, Aurélien Gâteau wrote: > > - document class elements with comment blocks located next to the > > documented element > > > > The code is available from kde:scratch/gateau/doxyqml .

Doxygen for QML components

2012-10-16 Thread Aurélien Gâteau
Hi, Currently documentation for QML components is done via a big comment at the beginning of the QML files, which leads to duplication of information and thus sometimes synchronization problems. For example right now the TabBar documentation states it inherits from DualStateButton. I have been

Re: Review Request: Humble attempt to improve the ScreenSaver KCM

2012-10-15 Thread Aurélien Gâteau
wrote: > I agree. On the other hand, showing new dialogs it's considered bad > practice. :/ > Maybe we can show the xscreensaver part when it's enabled? > > **Aleix turns the agateau-sign on** (see bat-signal for further reference) > > Aurélien Gâ

Re: Review Request: Humble attempt to improve the ScreenSaver KCM

2012-10-14 Thread Aurélien Gâteau
wrote: > I agree. On the other hand, showing new dialogs it's considered bad > practice. :/ > Maybe we can show the xscreensaver part when it's enabled? > > **Aleix turns the agateau-sign on** (see bat-signal for further reference) > > Aurélien Gâ

Re: Review Request: Make it possible to use QtCreator QML profiler and debugger with KDE Applications

2012-10-11 Thread Aurélien Gâteau
://codereview.qt-project.org/#change,36859 - Aurélien Gâteau On Sept. 21, 2012, 3:29 p.m., Aurélien Gâteau wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.

Re: Review Request: Humble attempt to improve the ScreenSaver KCM

2012-10-08 Thread Aurélien Gâteau
> On Oct. 8, 2012, 5:31 p.m., Martin Gräßlin wrote: > > just an idea: what about hiding the whole X screen saver stuff behind > > another configure dialog. Looking at the screenshot I find the design puts > > emphasis on the wrong part: what we want to remove takes more than 50 % of > > the av

Re: Display Configuration KCM design review

2012-10-05 Thread Aurélien Gâteau
Le jeudi 4 octobre 2012 14:44:40 Aaron J. Seigo a écrit : > On Thursday, October 4, 2012 14:01:04 Aurélien Gâteau wrote: > > Yes, that happens to me when I use my laptop with an external monitor: the > > bottom of the two screens do not line up, > > in which case does one NO

Re: share-like-connect in 4.10

2012-10-05 Thread Aurélien Gâteau
Le jeudi 4 octobre 2012 13:22:38 Aaron J. Seigo a écrit : > On Thursday, October 4, 2012 09:06:10 Aurélien Gâteau wrote: > > Le jeudi 4 octobre 2012 00:25:53 Aaron J. Seigo a écrit : > > > > I also want to see this discussed properly on kde-devel and > > > > kd

  1   2   3   >