D29667: Export metalist to json file

2020-05-12 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Alright! Ship it REPOSITORY R264 KApiDox BRANCH cblack/metadata (branched from master) REVISION DETAIL https://phabricator.kde.org/D29667 To: cblack, ochurlaud Cc: bcooksley,

D29667: Export metalist to json file

2020-05-12 Thread Olivier Churlaud
ochurlaud added a comment. Don't use file as a variable name as it's a python keyword. Do you really want to dump those files everywhere? REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D29667 To: cblack, ochurlaud Cc: bcooksley, kde-frameworks-devel,

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le 27 avril 2020 22:33:12 GMT+02:00, Ben Cooksley a écrit : >On Tue, Apr 28, 2020 at 8:31 AM Albert Astals Cid >wrote: >> >> El dilluns, 27 d’abril de 2020, a les 13:19:07 CEST, Ben Cooksley va >escriure: >> > On Mon, Apr 27, 2020 at 11:12 PM Olivier Chur

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le 27 avril 2020 22:33:12 GMT+02:00, Ben Cooksley a écrit : >On Tue, Apr 28, 2020 at 8:31 AM Albert Astals Cid >wrote: >> >> El dilluns, 27 d’abril de 2020, a les 13:19:07 CEST, Ben Cooksley va >escriure: >> > On Mon, Apr 27, 2020 at 11:12 PM Olivier Chur

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le 27 avril 2020 22:33:12 GMT+02:00, Ben Cooksley a écrit : >On Tue, Apr 28, 2020 at 8:31 AM Albert Astals Cid >wrote: >> >> El dilluns, 27 d’abril de 2020, a les 13:19:07 CEST, Ben Cooksley va >escriure: >> > On Mon, Apr 27, 2020 at 11:12 PM Olivier Chur

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Hi, Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > > replies] > > > > Hello Community members, > > > > In view of upcoming Gitlab migration, we sysadmin

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le lundi 27 avril 2020, 13:19:07 CEST Ben Cooksley a écrit : > On Mon, Apr 27, 2020 at 11:12 PM Olivier Churlaud > wrote: > > > > Hi, > > > > Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > > > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wr

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le lundi 27 avril 2020, 13:19:07 CEST Ben Cooksley a écrit : > On Mon, Apr 27, 2020 at 11:12 PM Olivier Churlaud > wrote: > > > > Hi, > > > > Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > > > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wr

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Le lundi 27 avril 2020, 13:19:07 CEST Ben Cooksley a écrit : > On Mon, Apr 27, 2020 at 11:12 PM Olivier Churlaud > wrote: > > > > Hi, > > > > Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > > > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wr

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Hi, Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > > replies] > > > > Hello Community members, > > > > In view of upcoming Gitlab migration, we sysadmin

Re: Information regarding upcoming Gitlab Migration

2020-04-27 Thread Olivier Churlaud
Hi, Le lundi 27 avril 2020, 12:38:42 CEST Aleix Pol a écrit : > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > > replies] > > > > Hello Community members, > > > > In view of upcoming Gitlab migration, we sysadmin

D28811: Add drawer on mobile and clean code

2020-04-16 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. In D28811#649209 , @ognarb wrote: > I now generate the CSS from aether-sass. The code can be found https://invent.kde.org/websites/aether-sass/-/blob/master/css/aether-sidebar.scss.

D28811: Add drawer on mobile and clean code

2020-04-15 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. OK it's clearer: Please add somewhere that it's a generated css so that no-one tries to change it by hand and ship it REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28811 To: ognarb, cblack,

D28811: Add drawer on mobile and clean code

2020-04-15 Thread Olivier Churlaud
ochurlaud added a comment. Except these clarifications, I'm not against this change REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28811 To: ognarb, cblack, ochurlaud Cc: kde-frameworks-devel, kde-doc-english, LeGast00n, cblack, gennad, fbampaloukas, michaelh,

D28811: Add drawer on mobile and clean code

2020-04-15 Thread Olivier Churlaud
ochurlaud requested changes to this revision. ochurlaud added a comment. This revision now requires changes to proceed. I really dislike how you close brackets in css : it's more dense and harder to read. You indent sometimes with 1,2 or 3 spaces: it should be consistent. Is it

D28734: Port to Aether Bootstrap 4

2020-04-11 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. OK for me REPOSITORY R264 KApiDox BRANCH cblack+ognarb/bootstrap-4 (branched from master) REVISION DETAIL https://phabricator.kde.org/D28734 To: cblack, #vdg, ochurlaud Cc:

D28712: Redo api.kde.org to look more like Aether

2020-04-09 Thread Olivier Churlaud
ochurlaud added a comment. In D28712#645095 , @ognarb wrote: > Not tested yet but I like the style :) Though to be more consistent with the style used in other KDE websites, I would change the background color to #fff and use #f8f9fa for the

D28712: Redo api.kde.org to look more like Aether

2020-04-09 Thread Olivier Churlaud
ochurlaud accepted this revision. This revision is now accepted and ready to land. REPOSITORY R264 KApiDox BRANCH cblack/aether (branched from master) REVISION DETAIL https://phabricator.kde.org/D28712 To: cblack, #vdg, ognarb, ochurlaud Cc: ngraham, kde-frameworks-devel,

D28712: Redo api.kde.org to look more like Aether

2020-04-09 Thread Olivier Churlaud
ochurlaud added a comment. Nice work. This is a ship it for me! REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D28712 To: cblack, #vdg, ognarb, ochurlaud Cc: ngraham, kde-frameworks-devel, kde-doc-english, LeGast00n, cblack, gennad, fbampaloukas, GB_2, michaelh,

Re: KDE Developer Documentation

2019-12-27 Thread Olivier Churlaud
live in Git and then embedded in the >wiki, >that would at least solve half the problem (there are MediaWiki plugins >but >I think they are deemed unsafe). > >Any guidance on the matter is much appreciated! > >[1] https://doc.qt.io/qt-5/qtwidgets-tutorials-notepad-example.html > >

D26197: Display fully qualified class/namespace name as page header

2019-12-26 Thread Olivier Churlaud
ochurlaud added a comment. One minor correction then you can ship it INLINE COMMENTS > generator.py:456 > +else: > +mapping['classname'] = mapping['fullname'] = None > Prefer the use of 2 settings to None instead of this double one. It's more explicit for

Re: KDE Developer Documentation

2019-12-24 Thread Olivier Churlaud
Hi all (and especially Juan Carlos), I just read your report and it's very interesting. I agree with most of what you wrote. I have some minor precisions to give about Techbase: The goal was to remove (or only archive) the tutorials on techbase and replace them by examples within the

D25632: Fix module imports with Python3

2019-12-01 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. If it still works it's OK to push. REPOSITORY R264 KApiDox BRANCH bugfix/graphvizImportPython3 REVISION DETAIL https://phabricator.kde.org/D25632 To: cordlandwehr, ochurlaud

D24568: Provide clang-format target with a KDE Frameworks style file

2019-10-13 Thread Olivier Churlaud
ochurlaud added a comment. In D24568#546289 , @kossebau wrote: > There is also https://techbase.kde.org/Policies/Frameworks_Coding_Style which though missed the move from techbase to community, other than the other policies. > > I suspect

D22421: generate list of data files on the fly

2019-07-12 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Nice! Thx Harald! REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D22421 To: sitter, ochurlaud Cc: kde-frameworks-devel, kde-doc-english,

Re: Revisiting the TechBase situation (2 proposals)

2019-04-04 Thread Olivier Churlaud
Hello Le jeudi 4 avril 2019, 11:33:10 CEST Juan Carlos Torres a écrit : > On Thu, Apr 4, 2019 at 3:52 AM Olivier Churlaud > wrote: > > > Hello, > > > > I agree 100% with what Friedrich wrote, which is nice because it's exactly > > the line we decide

Re: Revisiting the TechBase situation (2 proposals)

2019-04-03 Thread Olivier Churlaud
Hello, Le mardi 2 avril 2019, 18:49:59 CEST Friedrich W. H. Kossebau a écrit : > Hi Juan, > > Am Dienstag, 2. April 2019, 16:20:42 CEST schrieb Juan Carlos Torres: > > Greetings, > > > > I won't be going into history but right now we have two developer-facing > > wikis: TechBase and Community.

Re: Updating the Framework apidocs part 1: fixing the presentation

2019-04-01 Thread Olivier Churlaud
Le lundi 1 avril 2019, 08:54:12 CEST Juan Carlos Torres a écrit : > On Sat, Mar 30, 2019 at 4:11 PM Olivier Churlaud > wrote: > > > I'm the one who is supposed to be in charge of api.kde.org and specially > > kapidox, the program that generate the website. > > &g

Re: Updating the Framework apidocs part 1: fixing the presentation

2019-03-30 Thread Olivier Churlaud
Le vendredi 29 mars 2019, 09:47:34 CET Juan Carlos Torres a écrit : > Hello everyone! > Hi again! > It's that docs guy again! Hope you don't mind this brief interruption of > coding activities to give our apidocs some TLC. > > I recently went over the KDE Frameworks apidocs, one framework

Re: KDE Developer Documentation Support update

2019-03-28 Thread Olivier Churlaud
Hi Juan Carlos, Le mercredi 27 mars 2019, 14:03:54 CET Juan Carlos Torres a écrit : > Greetings KDE Community! > > I'm Juan Carlos Torres (Jucato on IRC) and I've recently been hired as a > contractor to get the ball rolling on updating our developer documentation. > Over the years, our

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
This revision was automatically updated to reflect the committed changes. Closed by commit R236:6748d24d5fb2: Create a first example of the use of KMessageBox (authored by ochurlaud). REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40094=40095

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
ochurlaud marked an inline comment as done. REPOSITORY R236 KWidgetsAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D14957 To: ochurlaud, #frameworks, dfaure Cc: dfaure, kde-frameworks-devel, michaelh, ngraham, bruns

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
ochurlaud updated this revision to Diff 40094. ochurlaud added a comment. Fix as dfaure said REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40092=40094 BRANCH master REVISION DETAIL https://phabricator.kde.org/D14957 AFFECTED FILES

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
This revision was automatically updated to reflect the committed changes. Closed by commit R311:87a84ac866c5: Move example from techbase to own repo (authored by ochurlaud). REPOSITORY R311 KWallet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14955?vs=40090=40093 REVISION DETAIL

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
ochurlaud updated this revision to Diff 40092. ochurlaud added a comment. Added licences and add some changes to fit what was done on D14955 REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14957?vs=40082=40092

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
ochurlaud marked 4 inline comments as done. REPOSITORY R311 KWallet REVISION DETAIL https://phabricator.kde.org/D14955 To: ochurlaud, #frameworks, valir, dfaure Cc: dfaure, aacid, elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
ochurlaud updated this revision to Diff 40090. ochurlaud added a comment. Added Licence, made it compile within the whole project and fix issues raised by dfaure. I think we shouldn't care about making the example self-standing. It's aimed at tutoring people to use the lib, not ECM nor

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
ochurlaud added a comment. In D14955#312279 , @aacid wrote: > -1 > > The example is not getting compiled it will rot I'm not sure to understand: it *can* be compiled, whereas on the current status (in the wiki) it will never be !

[Tutorials] Proposition of tutorial organization

2018-08-20 Thread Olivier Churlaud
Dear framework maintainers, Hidden somewhere on techbase.kde.org, one can find this: https://techbase.kde.org/Development/Tutorials Not everything is up-to-date nor at the right level. It's often too complex, not showing fully that Framework libs can be used alone (currently it seems that if

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
ochurlaud added a reviewer: Frameworks. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D14957 To: ochurlaud, #frameworks Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14957: Create a first example of the use of KMessageBox

2018-08-20 Thread Olivier Churlaud
ochurlaud created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ochurlaud requested review of this revision. REVISION SUMMARY Based on https://techbase.kde.org/Development/Tutorials/First_program TEST PLAN It compiles REPOSITORY R236

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
ochurlaud added reviewers: Frameworks, valir. REPOSITORY R311 KWallet REVISION DETAIL https://phabricator.kde.org/D14955 To: ochurlaud, #frameworks, valir Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14955: Move example from techbase to own repo

2018-08-20 Thread Olivier Churlaud
ochurlaud created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ochurlaud requested review of this revision. REVISION SUMMARY Add example based on https://techbase.kde.org/Development/Tutorials/KWallet so that it can be kept up-to-date. The

Re: Closing old Plasma 4 bugs

2018-05-26 Thread Olivier Churlaud
Hello, Just to remind you that companies using KDE/Plasma on RHEL7 are still using plasma 4. I don't know if it has an impact but in case I wanted to point it out. Cheers, Olivier Le samedi 26 mai 2018, 05:09:06 CEST Nate Graham a écrit : > Kubuntu 18.04 has now now been out for a month, so

D11330: Remove shebangs from non-executable files

2018-03-14 Thread Olivier Churlaud
ochurlaud accepted this revision. This revision is now accepted and ready to land. REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D11330 To: lbeltrame, #frameworks, ochurlaud Cc: #documentation, michaelh, ngraham, skadinna

D11315: Look first for qhelpgenerator-qt5 for help generation

2018-03-14 Thread Olivier Churlaud
ochurlaud accepted this revision. ochurlaud added a comment. This revision is now accepted and ready to land. Thx for this change REPOSITORY R264 KApiDox BRANCH master REVISION DETAIL https://phabricator.kde.org/D11315 To: lbeltrame, #frameworks, ochurlaud Cc: fvogt, #documentation,

Using KCharts/KDiagram vs Qt Chart

2018-01-27 Thread Olivier Churlaud
Hi, I was seeing the beautiful graphs shown by Qt in a blogpost about Qt Chart. In the same time, I've seen graphs within KDE Apps (such as KMyMoney) which are not so eye-candy. I was wondering: now that Qt Chart was opensourced, what is the benefit of using the KDE API versus the Qt one? Is

KWalletManager improvement

2018-01-27 Thread Olivier Churlaud
Hey, I want to improve the kwalletmanager gui so that it can also store other passwords (to be more like a KeyPass for instance). I see that there are two backend repos: KWallet and KSecret. With the KSecret API I could add data as attribute (for instance the corresponding login, url, ...)

D6604: add a metainfo.yaml to make ECM a proper framework

2017-07-11 Thread Olivier Churlaud
ochurlaud added a comment. At the time I thought about merging it but we would have to read document everything in doxygen , which would be completely different from what upstream cmake does. The ECM page is already linked from the left menu. Does it make sense to have it in

D6604: add a metainfo.yaml to make ECM a proper framework

2017-07-10 Thread Olivier Churlaud
ochurlaud added a comment. In https://phabricator.kde.org/D6604#123635, @sitter wrote: > related to https://phabricator.kde.org/D6603 > > @ochurlaud I hope this will not break api.kde, given ECM hasn't had a metainfo.yaml previously it seems to have some sort of special arrangement?

Re: Wayland porting notes

2017-06-20 Thread Olivier Churlaud
> Le 20 juin 2017 à 13:44, Elvis Angelaccio <elvis.angelac...@kde.org> a écrit : > >> On martedì 20 giugno 2017 00:00:32 CEST, Elvis Angelaccio wrote: >> On domenica 18 giugno 2017 01:20:52 CEST, Olivier Churlaud wrote: >>>> ... >>> I don't real

Re: Wayland porting notes

2017-06-17 Thread Olivier Churlaud
> Le 16 juin 2017 à 22:51, Elvis Angelaccio <elvis.angelac...@kde.org> a écrit : > >> On venerdì 16 giugno 2017 10:07:22 CEST, Olivier Churlaud wrote: >> Hello, >> Here comes the wiki bogeyman. >> This page is orphaned so no one will ever find it,

Re: Wayland porting notes

2017-06-16 Thread Olivier Churlaud
Hello, Here comes the wiki bogeyman. This page is orphaned so no one will ever find it, except if they think of reading this particular email. Please link the page from a relevant one. Guides a day howto / development maybe? I don't know, but to make your great notes useful, they have to

D5275: Kapidox generates qch files

2017-04-07 Thread Olivier Churlaud
ochurlaud added a comment. In https://phabricator.kde.org/D5275#99677, @antonanikin wrote: > Hi, Olivier. > > Your QCH files displayed normally but with one bug - broken links for some JavaScript code. For example open `KTipDialog` class doc from your `Frameworks` file. Then try to

D5275: Kapidox generates qch files

2017-04-07 Thread Olivier Churlaud
ochurlaud updated this revision to Diff 13209. ochurlaud added a comment. Corrected antonanikin issue REPOSITORY R264 KApiDox CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5275?vs=13076=13209 BRANCH olivier/do_qch REVISION DETAIL https://phabricator.kde.org/D5275 AFFECTED

D5275: Kapidox generates qch files

2017-04-03 Thread Olivier Churlaud
ochurlaud updated this revision to Diff 13076. ochurlaud edited the summary of this revision. ochurlaud added a comment. Update: Order alphabetically the libs in a product REPOSITORY R264 KApiDox CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5275?vs=13045=13076 BRANCH

D5275: Kapidox generates qch files

2017-04-03 Thread Olivier Churlaud
ochurlaud added a comment. Here 3 examples. Frameworks only contains KApiDox, KConfigWidgets and KItemModels. I could have generated all of them of course... F3526219: okular.qch F3526218: frameworks.qch

D5275: Kapidox generates qch files

2017-04-03 Thread Olivier Churlaud
ochurlaud added reviewers: mwolff, apol. ochurlaud added a comment. I add some people who may be able to tell me if the output can be used in the kdevelop generation (because I heard you use it). I don't use qch files myself, so I need an external view REPOSITORY R264 KApiDox

D5275: Kapidox generates qch files

2017-04-02 Thread Olivier Churlaud
ochurlaud edited the summary of this revision. ochurlaud added reviewers: kossebau, KDevelop. REPOSITORY R264 KApiDox REVISION DETAIL https://phabricator.kde.org/D5275 To: ochurlaud, kossebau, #kdevelop Cc: #frameworks, #documentation, skadinna

D5275: Kapidox generates qch files

2017-04-02 Thread Olivier Churlaud
ochurlaud created this revision. Restricted Application added projects: Frameworks, Documentation. Restricted Application added subscribers: Documentation, Frameworks. REPOSITORY R264 KApiDox BRANCH olivier/do_qch REVISION DETAIL https://phabricator.kde.org/D5275 AFFECTED FILES

All your answers about Where to put documentation?

2017-02-27 Thread Olivier Churlaud
Hi all, I'm often being asked about where to put documentation. I just wrote the final guide about this. On https://community.kde.org, we have since last year a Guide and HowTos page [1], which lists all KDE tutorial and resources. There is a chapter about Quality and Documentation [2] in

Re: Phabricator: All repositories registered - upcoming workflow changes

2017-01-29 Thread Olivier Churlaud
Le dimanche 29 janvier 2017, 16:24:46 CET Luigi Toscano a écrit : > Il 29 gennaio 2017 16:16:51 CET, Olivier Churlaud <oliv...@churlaud.com> ha scritto: > >Le dimanche 29 janvier 2017, 08:32:21 CET Ben Cooksley a écrit : > >> We've just completed the registration of a

Re: Phabricator: All repositories registered - upcoming workflow changes

2017-01-29 Thread Olivier Churlaud
Le dimanche 29 janvier 2017, 08:32:21 CET Ben Cooksley a écrit : > Hi everyone, Hi, > We've just completed the registration of all mainline repositories > (not including Websites or Sysadmin namespaced ones) on Phabricator. > Thanks go to Luigi Toscano for providing significant assistance with >

api.kde.org not generated

2016-11-25 Thread Olivier Churlaud
Hi, I just realized that api.kde.org was not generated since the 18th (one week). I just wanted to say that it is known and currently under investigation. I'll fix this as soon as I can (before Monday anyway) @Ben: I would be happy to discuss if an email alert could be set from the server to

Re: Review Request 129445: Add FreeBSD to metainfo.yaml.

2016-11-21 Thread Olivier Churlaud
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129445/#review101028 --- Ship it! Ship It! - Olivier Churlaud On Nov. 20, 2016

Re: how to appstream

2016-11-05 Thread Olivier Churlaud
Le vendredi 4 novembre 2016, 21:33:11 CET Matthias Klumpp a écrit : > 2016-11-04 13:57 GMT+01:00 Olivier Churlaud <oliv...@churlaud.com>: > > Hi, > > > > Le vendredi 4 novembre 2016, 00:12:13 CET Matthias Klumpp a écrit : > >> 2016-11-03 15:19 GMT+01:00 Hara

Re: how to appstream

2016-11-04 Thread Olivier Churlaud
Hi, Le vendredi 4 novembre 2016, 00:12:13 CET Matthias Klumpp a écrit : > 2016-11-03 15:19 GMT+01:00 Harald Sitter : > > I couldn't find reasonable documentation on how to best add appstream > > data to a kde application, so I threw this together here: > > > >

Re: Review Request 119723: Show q_properties at the top of class documentation

2016-10-21 Thread Olivier Churlaud
don't have anything against the change. +1 for me - Olivier Churlaud On Oct. 21, 2016, 1:14 p.m., David Edmundson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Defaults values in KConfig

2016-10-12 Thread Olivier Churlaud
Hello, I'm working on the Color KCM. The default theme doesn't work, because it cannot find the default values. (https://bugs.kde.org/show_bug.cgi?id=370305) I'm therefore wondering: where are theses values stored? Thiago said "in XDG_CONFIG_DIRS". I looked there, but I do not see any default

Re: Review Request 128606: [kapidox] Read library fancyname from metainfo.yaml before parsing it from CMakeLists.txt

2016-08-04 Thread Olivier Churlaud
where the CMakeLists.txt is unavailable (like a sub-library). Thx for the fix! - Olivier Churlaud On Aug. 5, 2016, 1:34 a.m., Daniel Vrátil wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: kapidox: supporting also QML (and cmake, Python,, ...)

2016-08-01 Thread Olivier Churlaud
Hi, I tried to address each point raised here. Don't hesitate to criticize or questions my answers and even propose something else. I try to find the best solutions but it doesn't mean my solutions are actually the best. Le 01/08/2016 à 19:50, Friedrich W. H. Kossebau a écrit : Hi, Olivier

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

2016-07-23 Thread Olivier Churlaud
g/r/12/#comment65872> Please add `@return Unit as a string` here - Olivier Churlaud On July 23, 2016, 11 a.m., Garret Wassermann wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

2016-07-23 Thread Olivier Churlaud
before friday, take it for a "Ship it" (and don't forget to add a line "REVIEW: 12" in your commit message) - Olivier Churlaud On July 23, 2016, 11 a.m., Garret Wassermann wrote: > > --- > This is an a

Re: Review Request 128476: Add scrollbars to kcm started without systemsettings5

2016-07-20 Thread Olivier Churlaud
2016, 4:20 p.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128476/ > --- > &

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Olivier Churlaud
> On July 20, 2016, 3:48 p.m., Thomas Pfeiffer wrote: > > +1 > > It may still scare people, but I think it's a good compromise. If we get bad feedback , it can be reverted. But I think we won't - Olivier --- This is an automatically

Re: Review Request 128413: Add clipboard buttons in the Checksums tab

2016-07-20 Thread Olivier Churlaud
> On July 16, 2016, 12:30 p.m., Elvis Angelaccio wrote: > > Ping? > > > > (String freeze for frameworks is next Saturday, right?) I would say: replace the [copy] button by a select on click. and then ship it - Olivier --- This is an

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Olivier Churlaud
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128490/#review97664 --- Ship it! Ship It! - Olivier Churlaud On July 20, 2016

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-19 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 120000: New Thermal and Electrical Units and Unit Convienience Function for KUnitConversion

2016-07-18 Thread Olivier Churlaud
see anything to say against the patch. Yet as I never worked on the project, I cannot give the ship it... src/unit.h (line 182) <https://git.reviewboard.kde.org/r/12/#comment65787> You should remove theses trailing spaces. - Olivier Churlaud On July 18, 2016, 11:55 p.m.,

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
to calculate it without verifying > against another checksum) > - more explanatory text (the intro sentenced comes from Wikipedia) to > inform the user > > This is the result: > > ``` > Checksum > > > A checksum

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
ed in > Rangar's comment in the review #128283) > > I do agree that the two work-flows are separate. Still, the UI does not > make that obvious to me. The UI does not make it obvious that it *has* two > work-flows. > > So, from my point of view, these are the p

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-18 Thread Olivier Churlaud
> On July 18, 2016, 2:05 p.m., Sebastian Kügler wrote: > > Please don't ship it, yet. > > > > > > I find the UI illogical. There's a groupbox grouping the checksum buttons, > > but then you can input the checksum above, so essentially, the groupbox is > > unnecessary and confusing. > > > >

Review Request 128476: Add scrollbars to kcm started without systemsettings5

2016-07-18 Thread Olivier Churlaud
://git.reviewboard.kde.org/r/128476/diff/ Testing --- Compiles, behave as expected (expect for the size) Thanks, Olivier Churlaud ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-18 Thread Olivier Churlaud
marked as submitted. Review request for KDE Frameworks and Eike Hein. Changes --- Submitted with commit 574abcd05ece484da77635e2b8f23d3f4c1af5dd by Olivier CHURLAUD to branch master. Bugs: 362234 https://bugs.kde.org/show_bug.cgi?id=362234 Repository: kconfigwidgets Description

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-18 Thread Olivier Churlaud
83c96b6 Diff: https://git.reviewboard.kde.org/r/128464/diff/ Testing --- Compile and tested: Good behavior Thanks, Olivier Churlaud ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Re: Re: Need help: Use of KConfig

2016-07-17 Thread Olivier Churlaud
Le 16/07/2016 à 18:11, Michael Pyne a écrit : On Sat, July 16, 2016 01:15:38 Olivier Churlaud wrote: Hi, I spent a lot of time tonight, trying to port a config dialog out of KButtonGroup (in Okular). I read that I should instead use QButtonGroup + QGroupBox. I did this, and this part works

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-16 Thread Olivier Churlaud
. With this fix I can. The config page has now scroll bars when needed. Diffs (updated) - src/kconfigdialog.cpp 83c96b6 Diff: https://git.reviewboard.kde.org/r/128464/diff/ Testing --- Compile and tested: Good behavior Thanks, Olivier Churlaud

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-16 Thread Olivier Churlaud
appear too small. I don't know how to define a good default size... - Olivier Churlaud On July 16, 2016, 12:02 p.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-16 Thread Olivier Churlaud
it: https://git.reviewboard.kde.org/r/128464/#review97467 --- On July 16, 2016, 2:52 a.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 128464: Allow scrolling in config windows for small screens

2016-07-15 Thread Olivier Churlaud
Churlaud On July 16, 2016, 2:52 a.m., Olivier Churlaud wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Need help: Use of KConfig

2016-07-15 Thread Olivier Churlaud
, this is commented in https://api.kde.org/frameworks/kconfigwidgets/html/kconfigdialogmanager_8cpp_source.html#l00153. How should I do to have several radio buttons to select one system configuration? Thank you Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de Lyon in Dual

Re: QIcon::fromTheme() does not load scalable icons

2016-07-13 Thread Olivier Churlaud
From: Antonio Rojas <aro...@archlinux.org> To:kde-frameworks-devel@kde.org Subject: Re: QIcon::fromTheme() does not load scalable icons Message-ID:<nm2t7t$tlh$1...@ger.gmane.org> Content-Type: text/plain; charset="ISO-8859-1" Olivier Churlaud wrote: > >Can you

Re: QIcon::fromTheme() does not load scalable icons

2016-07-12 Thread Olivier Churlaud
the problem comes from upstream or packaging. Where should I open the bug? Cheers Olivier Cheers Elvis [1]: http://lists-archives.com/kde-devel/34061-how-to-properly-install-app-icons.html Thank you Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de Lyon in Dual Degree at TU

QIcon::fromTheme() does not load scalable icons

2016-07-12 Thread Olivier Churlaud
as it makes a lot of redundancy. Can you point me to the right place where to open the bug? Thank you Olivier -- Olivier CHURLAUD Engineer Student at Ecole Centrale de Lyon in Dual Degree at TU Berlin, M.Sc. Elektrotechnik @: oliv...@churlaud.com tel: +49 (0)1575-2931348 in: http://linkedin.com

Re: Review Request 128397: KIconEngine: Fix QIcon::hasThemeIcon always returning true

2016-07-08 Thread Olivier Churlaud
://sourceforge.net/p/texstudio/bugs/1837/ which was related to bug 365130. Not completely though, as the icon of the app is not shown in the taskbar (but it might be another bug). Looking forward for the release. - Olivier Churlaud On July 8, 2016, 2:24 p.m., David Rosca wrote

Re: Review Request 128397: KIconEngine: Fix QIcon::hasThemeIcon always returning true

2016-07-08 Thread Olivier Churlaud
/show_bug.cgi?id=365130 - Olivier Churlaud On July 8, 2016, 12:52 p.m., David Rosca wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

  1   2   >