Re: Review Request 110833: Remove KIconLoader dependency to KWidgets

2013-06-05 Thread Aleix Pol Gonzalez
/utils/kglobalsettings.h 5fb1d9b staging/kwidgets/src/utils/kglobalsettings.cpp 53b648e Diff: http://git.reviewboard.kde.org/r/110833/diff/ Testing --- it builds Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks

Re: Review Request 110833: Remove KIconLoader dependency to KWidgets

2013-06-05 Thread Aleix Pol Gonzalez
/kiconloader.cpp 795ec93 staging/kwidgets/src/utils/kglobalsettings.h 5fb1d9b staging/kwidgets/src/utils/kglobalsettings.cpp 53b648e Diff: http://git.reviewboard.kde.org/r/110833/diff/ Testing --- it builds Thanks, Aleix Pol Gonzalez ___ Kde

Re: Review Request 110674: Moving KToolBarSpacerAction to KWidgetsAddons

2013-06-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110674/#review33795 --- ping! - Aleix Pol Gonzalez On June 3, 2013, 4:08 p.m

Re: Review Request 110815: Move KPassivePopup to KWidgetAddons

2013-06-05 Thread Aleix Pol Gonzalez
af65f34 tier1/kwidgetsaddons/tests/CMakeLists.txt 85a8732 tier1/kwidgetsaddons/tests/kpassivepopuptest.h PRE-CREATION tier1/kwidgetsaddons/tests/kpassivepopuptest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/110815/diff/ Testing --- It builds fine Thanks, Aleix Pol

Re: Review Request 110674: Moving KToolBarSpacerAction to KWidgetsAddons

2013-06-05 Thread Aleix Pol Gonzalez
and installs fine. File Attachments ::addSeparator() http://git.reviewboard.kde.org/media/uploaded/files/2013/06/03/k3b-nospaceraction.png KToolBarSpacerAction http://git.reviewboard.kde.org/media/uploaded/files/2013/06/03/k3b-spaceraction.png Thanks, Aleix Pol Gonzalez

Review Request 110847: Move KPixmapRegionSelector* to KWidgetsAddons

2013-06-05 Thread Aleix Pol Gonzalez
/ Testing --- builds and the test seems to work Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 110847: Move KPixmapRegionSelector* to KWidgetsAddons

2013-06-05 Thread Aleix Pol Gonzalez
/tests/CMakeLists.txt 2bb9306 tier1/kwidgetsaddons/tests/kpixmapregionselectordialogtest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/110847/diff/ Testing --- builds and the test seems to work Thanks, Aleix Pol Gonzalez ___ Kde

Re: Review Request 110858: Cleanup KDE development scripts

2013-06-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110858/ --- (Updated June 6, 2013, 11:44 p.m.) Review request for KDE Frameworks and

Re: Review Request 110815: Move KPassivePopup to KWidgetAddons

2013-06-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110815/#review33920 --- ping! - Aleix Pol Gonzalez On June 5, 2013, 2:12 p.m

Re: Review Request 110833: Remove KIconLoader dependency to KWidgets

2013-06-11 Thread Aleix Pol Gonzalez
--- On June 10, 2013, 11:07 a.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110833

Re: Review Request 110833: Remove KIconLoader dependency to KWidgets

2013-06-12 Thread Aleix Pol Gonzalez
point). Aleix Pol Gonzalez wrote: Well, then the KWidgets not depending on KIconItems contradicts this review request, right? Kevin Ottens wrote: Well, not really. Here it's more about removing the dependency on KGlobalSettings than anything, and that is definitely welcome. Your

Re: Review Request 110833: Remove KIconLoader dependency to KWidgets

2013-06-12 Thread Aleix Pol Gonzalez
staging/kwidgets/src/utils/kglobalsettings.h 5fb1d9b staging/kwidgets/src/utils/kglobalsettings.cpp 53b648e Diff: http://git.reviewboard.kde.org/r/110833/diff/ Testing --- it builds Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing

Review Request 110997: Move KPageWidgets and KPageDialog to KWidgets

2013-06-13 Thread Aleix Pol Gonzalez
staging/kwidgets/tests/kpagewidgettest.h PRE-CREATION staging/kwidgets/tests/kpagewidgettest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/110997/diff/ Testing --- Everything builds, the test seems good. Thanks, Aleix Pol Gonzalez

Re: Review Request 110997: Move KPageWidgets and KPageDialog to KWidgets

2013-06-13 Thread Aleix Pol Gonzalez
/#review34320 --- On June 13, 2013, 2:57 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110997

Re: Review Request 110997: Move KPageWidgets and KPageDialog to KWidgets

2013-06-14 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/110997/diff/ Testing --- Everything builds, the test seems good. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 110997: Move KPageWidgets and KPageDialog to KWidgets

2013-06-17 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110997/#review34475 --- ping... - Aleix Pol Gonzalez On June 14, 2013, 10:27 a.m

Review Request 111103: KPage* belongs to KWidgetsAddons

2013-06-18 Thread Aleix Pol Gonzalez
--- builds Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 111106: Fix export marco in kde4support

2013-06-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/06/#review34625 --- Ship it! Cleaning up! ^^ good stuff - Aleix Pol Gonzalez

Re: Review Request 111171: Deprecate (in)active(Title/Text)Color in favor of KColorScheme

2013-06-22 Thread Aleix Pol Gonzalez
On June 22, 2013, 10:58 a.m., Thomas Lübking wrote: kdeui/util/kglobalsettings.cpp, line 308 http://git.reviewboard.kde.org/r/71/diff/1/?file=165092#file165092line308 These are the colors for the window titlbar (with ambiguous function names, though), the default for

Re: Review Request 111178: KSwitchLanguageDialog: Reconsider how KLocalizedString is initialized

2013-06-25 Thread Aleix Pol Gonzalez
--- On June 22, 2013, 5:09 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/78

Re: Review Request 111178: KSwitchLanguageDialog: Reconsider how KLocalizedString is initialized

2013-06-25 Thread Aleix Pol Gonzalez
depending on the desktop environment I'm using. Aleix Pol Gonzalez wrote: Ok, then we have a problem: - We don't want ki18n to decide what language we run (chusslove) - We don't want kde's QPlatformTheme to decide what language we run (albert) Then who decides what language

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-06-28 Thread Aleix Pol Gonzalez
this. Diffs - staging/frameworkintegration/src/integrationplugin/frameworkintegrationplugin.cpp b8f5fa7 Diff: http://git.reviewboard.kde.org/r/111298/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list

Review Request 111298: Add a URL handler for help:/// urls

2013-06-28 Thread Aleix Pol Gonzalez
/frameworkintegration/src/integrationplugin/frameworkintegrationplugin.cpp b8f5fa7 Diff: http://git.reviewboard.kde.org/r/111298/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-07-01 Thread Aleix Pol Gonzalez
/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 111348: Don't require KApplication for using KGlobalSettings

2013-07-01 Thread Aleix Pol Gonzalez
against kde4support Diffs - kdeui/util/kglobalsettings.cpp 3e60632 Diff: http://git.reviewboard.kde.org/r/111348/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-07-02 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111298/#review35425 --- On July 1, 2013, 5:13 p.m., Aleix Pol Gonzalez wrote

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-07-02 Thread Aleix Pol Gonzalez
--- On July 1, 2013, 5:13 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111298

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-07-02 Thread Aleix Pol Gonzalez
/openurltest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/111298/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 111298: Add a URL handler for help:/// urls

2013-07-03 Thread Aleix Pol Gonzalez
/kwidgets/tests/openurltest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/111298/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Re: Review Request 111405: Move KGlobalSettings::createApplicationPalette to KColorScheme

2013-07-05 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/111405/#review35636 --- On July 5, 2013, 2:25 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org

Review Request 111408: Move KConfigDialogManager to KConfigWidgets

2013-07-05 Thread Aleix Pol Gonzalez
staging/kconfigwidgets/src/CMakeLists.txt cd0afd2 Diff: http://git.reviewboard.kde.org/r/111408/diff/ Testing --- Builds Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Review Request 111409: Deprecate KGlobalSettings::*Font methods in favor of QFontsDatabase::systemFont

2013-07-05 Thread Aleix Pol Gonzalez
6f1832f tier1/kwidgetsaddons/src/kfontchooser.cpp 53a4a90 Diff: http://git.reviewboard.kde.org/r/111409/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Re: Review Request 111409: Deprecate KGlobalSettings::*Font methods in favor of QFontsDatabase::systemFont

2013-07-08 Thread Aleix Pol Gonzalez
--- On July 5, 2013, 6:24 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111409

Re: Review Request 111409: Deprecate KGlobalSettings::*Font methods in favor of QFontsDatabase::systemFont

2013-07-08 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/111409/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 111408: Move KConfigDialogManager to KConfigWidgets

2013-07-16 Thread Aleix Pol Gonzalez
/ Testing --- Builds Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 111409: Deprecate KGlobalSettings::*Font methods in favor of QFontsDatabase::systemFont

2013-07-19 Thread Aleix Pol Gonzalez
generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111409/#review35826 --- On July 8, 2013, 4:09 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated

Review Request 111833: Remove small dependency between KGlobalSettings::naturalCompare and KDirSortFilterProxyModel

2013-08-01 Thread Aleix Pol Gonzalez
class. There's KDE_DEFAULT_* but I'll see how to do that in a different patch, there are other cases where it's needed. Diffs - Diff: http://git.reviewboard.kde.org/r/111833/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde

Re: Review Request 111833: Remove small dependency between KGlobalSettings::naturalCompare and KDirSortFilterProxyModel

2013-08-01 Thread Aleix Pol Gonzalez
that in a different patch, there are other cases where it's needed. Diffs (updated) - KDE5PORTING.html 9d84525 kfile/kdirsortfilterproxymodel.cpp c8ea9b2 Diff: http://git.reviewboard.kde.org/r/111833/diff/ Testing --- Thanks, Aleix Pol Gonzalez

Review Request 111834: Move KHelpClient into xmlgui

2013-08-01 Thread Aleix Pol Gonzalez
/xmlgui/src/CMakeLists.txt 255960f staging/xmlgui/src/khelpclient.h e5f9946 staging/xmlgui/src/khelpclient.cpp dda0b13 Diff: http://git.reviewboard.kde.org/r/111834/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel

Review Request 111841: Prefer job-exec() to synchronousRun

2013-08-01 Thread Aleix Pol Gonzalez
/kunitconversion/currency.cpp 8d18f633 Diff: http://git.reviewboard.kde.org/r/111841/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks

Re: Review Request 111841: Prefer job-exec() to synchronousRun

2013-08-01 Thread Aleix Pol Gonzalez
staging/kunitconversion/src/kunitconversion/currency.cpp 8d18f633 Diff: http://git.reviewboard.kde.org/r/111841/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https

Re: Review Request 111841: Prefer job-exec() to synchronousRun

2013-08-02 Thread Aleix Pol Gonzalez
/kunitconversion/currency.cpp 8d18f633 Diff: http://git.reviewboard.kde.org/r/111841/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Review Request 111843: cut some kglobalsettings.h uses

2013-08-02 Thread Aleix Pol Gonzalez
/r/111843/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 111897: Move KFileMetaData (and friends) to kde4support

2013-08-05 Thread Aleix Pol Gonzalez
Diff: http://git.reviewboard.kde.org/r/111897/diff/ Testing --- builds... Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 111905: Actually emit iconChanged

2013-08-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111905/#review37197 --- Ship it! Ship It! - Aleix Pol Gonzalez On Aug. 6, 2013, 9

Re: Review Request 111897: Move KFileMetaData (and friends) to kde4support

2013-08-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111897/#review37193 --- On Aug. 5, 2013, 6:06 p.m., Aleix Pol Gonzalez wrote

Re: Review Request 111897: Move KFileMetaData (and friends) to kde4support

2013-08-06 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/111897/#review37195 --- On Aug. 5, 2013, 6:06 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Aleix Pol Gonzalez
On Aug. 6, 2013, 2:52 p.m., David Edmundson wrote: tier1/kconfig/src/gui/kconfigloader.h, line 112 http://git.reviewboard.kde.org/r/111908/diff/1/?file=176360#file176360line112 This looks like it should be const I suspect it wasn't because KCoreConfigSeleton::findItem

Re: Review Request 111897: Move KFileMetaData (and friends) to kde4support

2013-08-07 Thread Aleix Pol Gonzalez
staging/kde4support/src/kioslave/metainfo/CMakeLists.txt PRE-CREATION staging/kde4support/tests/CMakeLists.txt 41e35ce Diff: http://git.reviewboard.kde.org/r/111897/diff/ Testing --- builds... actually i'm not sure if there's Qt5 soprano/strigi. what's hte status? Thanks, Aleix Pol

Re: Review Request 111886: Stop using kde_file.h in kdesu

2013-08-07 Thread Aleix Pol Gonzalez
...) - Aleix Pol Gonzalez On Aug. 5, 2013, 12:52 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111886

Re: Review Request 111897: Move KFileMetaData (and friends) to kde4support

2013-08-08 Thread Aleix Pol Gonzalez
status? Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 112118: Move KGlobalSettings to kde4support, stop compiling kdeui while at it

2013-08-16 Thread Aleix Pol Gonzalez
CMakeLists.txt fb51ff4 interfaces/kimproxy/library/CMakeLists.txt 0dcee60 staging/kde4support/autotests/CMakeLists.txt 12fa24c staging/kde4support/src/CMakeLists.txt 9e9b1d1 Diff: http://git.reviewboard.kde.org/r/112118/diff/ Testing --- Thanks, Aleix Pol Gonzalez

Review Request 112175: Implement Ctrl+Enter dialog acceptation in KStyle::polish()

2013-08-20 Thread Aleix Pol Gonzalez
staging/frameworkintegration/src/kstyle/kstyle.cpp 63506476 Diff: http://git.reviewboard.kde.org/r/112175/diff/ Testing --- Created a kstyletest that shows a dialog with a text edit and makes sure that this works. Thanks, Aleix Pol Gonzalez

Re: Review Request 112175: Implement Ctrl+Enter dialog acceptation in KStyle::polish()

2013-08-20 Thread Aleix Pol Gonzalez
staging/frameworkintegration/src/kstyle/CMakeLists.txt 265eff4 Diff: http://git.reviewboard.kde.org/r/112175/diff/ Testing --- Created a kstyletest that shows a dialog with a text edit and makes sure that this works. Thanks, Aleix Pol Gonzalez

Re: Review Request 112217: copy KStyle as K4Style in KDE4Support

2013-08-22 Thread Aleix Pol Gonzalez
, then reviewboard doesn't complain. The semantics are a bit different but it will work for you, hopefully. Cheers! - Aleix Pol Gonzalez On Aug. 22, 2013, 11:34 p.m., Wojciech Kapuscinski wrote: --- This is an automatically generated e

Re: Review Request 112175: Implement Ctrl+Enter dialog acceptation in KStyle::polish()

2013-08-23 Thread Aleix Pol Gonzalez
staging/frameworkintegration/tests/kstyletest.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/112175/diff/ Testing --- Created a kstyletest that shows a dialog with a text edit and makes sure that this works. Thanks, Aleix Pol Gonzalez

Re: Review Request 112277: Fix deadlock by avoid calling Qfont::QFont() in KFontSettingsData

2013-08-25 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112277/#review38568 --- Ship it! Makes sense to me, good catch! - Aleix Pol Gonzalez

Re: Review Request 112367: Include the icon's theme in the cache key

2013-08-29 Thread Aleix Pol Gonzalez
the cache whenever the theme is changed. The cache is not useful anymore after a theme change. - Aleix Pol Gonzalez On Aug. 29, 2013, 8 p.m., Àlex Fiestas wrote: --- This is an automatically generated e-mail. To reply, visit: http

Re: Review Request 112401: Let ki18n report the list of available translations of the application

2013-08-31 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/112401/#comment28775 Maybe this could be a QSetQString? You're doing many sorts that we can probably delay anyway (in case it's even needed). - Aleix Pol Gonzalez On Aug. 31, 2013, 11:44 a.m., Chusslove Illich wrote

Re: Review Request 112331: Splitting KUnitConversion

2013-08-31 Thread Aleix Pol Gonzalez
--- On Aug. 28, 2013, 4:17 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112331

Re: Review Request 112331: Splitting KUnitConversion

2013-08-31 Thread Aleix Pol Gonzalez
--- On Aug. 28, 2013, 4:17 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112331

Re: Review Request 112331: Splitting KUnitConversion

2013-08-31 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/112331/diff/ Testing --- Builds, unit tests pass. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 112449: Move KUnitConverter files to src/

2013-09-02 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/112449/diff/ Testing --- builds, tests pass. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 112450: Move KScanDialog to kde4support

2013-09-02 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112450/#review39171 --- Ship it! Looks good to me. - Aleix Pol Gonzalez On Sept. 2

Review Request 112451: Actually install the modules in the ecm attic

2013-09-02 Thread Aleix Pol Gonzalez
missing something. Please tell me if that's the case. Diffs - CMakeLists.txt 5a22f85 attic/modules/CMakeLists.txt c832bed Diff: http://git.reviewboard.kde.org/r/112451/diff/ Testing --- Installs and seems to work. Thanks, Aleix Pol Gonzalez

Re: Review Request 112451: Actually install the modules in the ecm attic

2013-09-02 Thread Aleix Pol Gonzalez
. To reply, visit: http://git.reviewboard.kde.org/r/112451/#review39177 --- On Sept. 2, 2013, 3:09 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail

Review Request 112457: Start cleanup of KDocTools

2013-09-02 Thread Aleix Pol Gonzalez
no tests. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 112401: Let ki18n report the list of available translations of the application

2013-09-02 Thread Aleix Pol Gonzalez
approved. - Aleix Pol Gonzalez On Sept. 1, 2013, 12:22 p.m., Chusslove Illich wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112401

Re: Review Request 112451: Actually install the modules in the ecm attic

2013-09-03 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/112451/diff/ Testing --- Installs and seems to work. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 112457: Start cleanup of KDocTools

2013-09-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112457/#review39242 --- On Sept. 2, 2013, 4:42 p.m., Aleix Pol Gonzalez wrote

Review Request 112485: Cleaning up KCompletion

2013-09-03 Thread Aleix Pol Gonzalez
/klineedit.cpp b2b430d staging/kcompletion/src/config-kcompletion.h.cmake 35b64a2 staging/kcompletion/src/CMakeLists.txt f08a47d staging/kcompletion/src/klineedit_p.h 4d3cfbc Diff: http://git.reviewboard.kde.org/r/112485/diff/ Testing --- builds, tests pass Thanks, Aleix Pol Gonzalez

Re: Review Request 112451: Adapt the DocBook finders to newer naming conventions

2013-09-04 Thread Aleix Pol Gonzalez
-modules/FindDocBookXSL.cmake PRE-CREATION Diff: http://git.reviewboard.kde.org/r/112451/diff/ Testing --- Installs and seems to work. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Re: Review Request 112451: Adapt the DocBook finders to newer naming conventions

2013-09-04 Thread Aleix Pol Gonzalez
. Diffs (updated) - attic/modules/FindDocBookXML.cmake 26053bf attic/modules/FindDocBookXSL.cmake ab93c9e Diff: http://git.reviewboard.kde.org/r/112451/diff/ Testing --- Installs and seems to work. Thanks, Aleix Pol Gonzalez

Re: Review Request 119590: Disable the DDS and JPEG-2000 plugins when Qt version is 5.3 or later

2014-08-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119590/#review63719 --- Ship it! Makes sense to me. - Aleix Pol Gonzalez On Aug

Re: Review Request 119593: Ensure there's a plastform window before restoring the window geometry

2014-08-03 Thread Aleix Pol Gonzalez
. - Aleix Pol Gonzalez On Aug. 3, 2014, 7:13 p.m., Thomas Lübking wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119593

Re: Review Request 119594: ensure there's ::windowHandle() and then restore the a file dialogs size before calling ::exec()

2014-08-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119594/#review63721 --- Ship it! Makes sense to me. - Aleix Pol Gonzalez On Aug

Re: Review Request 119588: Remove Qt5 debugfull hack from ECMConfig.cmake

2014-08-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119588/#review63888 --- Ship it! Looks reasonable to me. - Aleix Pol Gonzalez

Re: Review Request 119698: Save radio button index in QGroupBox that are composed only by radio buttons

2014-08-10 Thread Aleix Pol Gonzalez
you're not accessing through them. And maybe you can use the more generic type QAbstractButton, only maybe, I'm unsure, up to you. - Aleix Pol Gonzalez On Aug. 10, 2014, 8:28 p.m., Albert Astals Cid wrote: --- This is an automatically

Re: Review Request 119698: Save radio button index in QGroupBox that are composed only by radio buttons

2014-08-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119698/#review64263 --- Ship it! Makes sense to me. - Aleix Pol Gonzalez On Aug

Re: Review Request 119713: Don't use hicolor if we have breeze or oxygen are available

2014-08-11 Thread Aleix Pol Gonzalez
/119713/#comment44914 Wouldn't it be better to use something like this? QIcon::setThemeSearchPaths(QIcon::themeSearchPaths()+thePathFor(breeze)); - Aleix Pol Gonzalez On Aug. 11, 2014, 1:24 p.m., Albert Astals Cid wrote

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

2014-08-11 Thread Aleix Pol Gonzalez
the properties' methods (get/set/signal) from the rest of the documentation? It would be really cool to document them all together. - Aleix Pol Gonzalez On Aug. 11, 2014, 10:35 p.m., David Edmundson wrote: --- This is an automatically generated e

Re: Review Request 114933: KF5 Port of kdeui/kmessagewidgetdemo

2014-08-12 Thread Aleix Pol Gonzalez
in the sprint, I think we should move this into the kde:kwidgetsaddons repository, in an examples subdirectory. - Aleix Pol Gonzalez On Aug. 12, 2014, 9:33 a.m., Laurent Navet wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 119745: Hide member documentation for classes in imports

2014-08-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119745/#review64405 --- Ship it! Ship It! - Aleix Pol Gonzalez On Aug. 12, 2014

Review Request 119798: Generating PkgConig files from ECM

2014-08-14 Thread Aleix Pol Gonzalez
/kde5/lib64 Cflags: -I/home/kde-devel/kde5/include/KF5/KCoreAddons Requires: Qt5Core Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119808: Move module metadata to after class picker

2014-08-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119808/#review64750 --- Maybe a screenshot will make it easier to review? - Aleix

Re: Review Request 119798: Generating PkgConig files from ECM

2014-08-18 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119798/#review64585 --- On Aug. 14, 2014, 11:10 p.m., Aleix Pol Gonzalez wrote

Re: Review Request 119798: Generating PkgConig files from ECM

2014-08-18 Thread Aleix Pol Gonzalez
, on my system: Name: KF5CoreAddons Version: 5.1.0 Libs: -L/home/kde-devel/kde5/lib64 -l/home/kde-devel/kde5/lib64 Cflags: -I/home/kde-devel/kde5/include/KF5/KCoreAddons Requires: Qt5Core Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list

Re: Review Request 119808: Move module metadata to after class picker

2014-08-19 Thread Aleix Pol Gonzalez
to interact with the File List more often. I think this shows we want a new design after all. For the moment, I won't +1 or -1. - Aleix Pol Gonzalez On Aug. 18, 2014, 10:04 p.m., Alex Merry wrote: --- This is an automatically generated e-mail

Re: Review Request 119798: Generating PkgConfig files from ECM

2014-08-19 Thread Aleix Pol Gonzalez
: KF5CoreAddons Version: 5.1.0 Libs: -L/home/kde-devel/kde5/lib64 -l/home/kde-devel/kde5/lib64 Cflags: -I/home/kde-devel/kde5/include/KF5/KCoreAddons Requires: Qt5Core Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org

Re: Review Request 119798: Generating PkgConig files from ECM

2014-08-19 Thread Aleix Pol Gonzalez
that don't use KDEInstallDirs can create their own variable. Also, pkconfig - pkgconfig. Aleix Pol Gonzalez wrote: I'm unsure about that, first ECM_MKSPECS_INSTALL_DIR is declared the same way (again, copypaste) then I understand that we can override the variable from

Re: Review Request 119808: Move module metadata to after class picker

2014-08-20 Thread Aleix Pol Gonzalez
On Aug. 19, 2014, 10:19 a.m., Aleix Pol Gonzalez wrote: I think it's disputable that the developers will want to interact with the File List more often. I think this shows we want a new design after all. For the moment, I won't +1 or -1. Alex Merry wrote: I'm admittedly going

Re: Review Request 119713: Switch to breeze or oxygen if they are available and icon theme is either not set or hicolor

2014-08-24 Thread Aleix Pol Gonzalez
break integration quite a bit, wouldn't it be possible to set it as fallback? - Aleix Pol Gonzalez On Aug. 13, 2014, 8:55 p.m., Albert Astals Cid wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 119713: Switch to breeze or oxygen if they are available and icon theme is either not set or hicolor

2014-08-24 Thread Aleix Pol Gonzalez
On Aug. 24, 2014, 9:56 p.m., Aleix Pol Gonzalez wrote: Changing the theme for everyone using kconfigwidgets would break integration quite a bit, wouldn't it be possible to set it as fallback? Albert Astals Cid wrote: Break integration with what? It only changes the theme

Re: Review Request 119713: Switch to breeze or oxygen if they are available and icon theme is either not set or hicolor

2014-08-24 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119713/#review65160 --- Ship it! Ship It! - Aleix Pol Gonzalez On Aug. 13, 2014

Re: Review Request 119901: Fix ECM to use qmake instead of hardcoding plugin install dirs

2014-08-25 Thread Aleix Pol Gonzalez
/CMake_Coding_Style#End_commands - Aleix Pol Gonzalez On Aug. 25, 2014, 6 p.m., Rohan Garg wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119901

Re: Review Request 119798: Generating PkgConfig files from ECM

2014-08-26 Thread Aleix Pol Gonzalez
/KCoreAddons Requires: Qt5Core Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119940: Use style primitive to render custom tooltip rather than custom code

2014-08-26 Thread Aleix Pol Gonzalez
screenshot would help in these reviews. - Aleix Pol Gonzalez On Aug. 26, 2014, 2:14 p.m., Hugo Pereira Da Costa wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119940

Review Request 119975: KDBusService shouldn't crash upon bad application name

2014-08-28 Thread Aleix Pol Gonzalez
, reproduced with an application. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119975: KDBusService shouldn't crash upon bad application name

2014-08-28 Thread Aleix Pol Gonzalez
/119975/diff/ Testing --- Ran tests, reproduced with an application. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 119991: Use Juvia to add comments on the API class pages

2014-08-29 Thread Aleix Pol Gonzalez
the comments are from human? - Aleix Pol Gonzalez On Aug. 29, 2014, 2:16 p.m., Denis Steckelmacher wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119991

<    9   10   11   12   13   14   15   16   17   18   >