D24707: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R269 BluezQt BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24707 To: kossebau, #frameworks, drosca, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh,

D24710: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Laurent Montel
mlaurent accepted this revision. This revision is now accepted and ready to land. REPOSITORY R301 KEmoticons BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24710 To: kossebau, #frameworks, mlaurent Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh,

D24599: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R270:76460e89854f: Use ECMGenerateExportHeader to manage deprecated API better (authored by kossebau). REPOSITORY R270

D24643: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes. Closed by commit R268:b6a1e260c42a: Use ECMGenerateExportHeader to manage deprecated API better (authored by kossebau). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D24643?vs=67925=68104#toc REPOSITORY R268

D23927: Improve naming of KTitleWidget icon methods

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau added a comment. In D23927#548607 , @kossebau wrote: > Any chance you could pik this up and complete? Thing is, the deprecated API is also still used internally,so anything but deprecated. Actually the new API calls the "deprecated"

D23927: Improve naming of KTitleWidget icon methods

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau added a comment. Any chance you could pik this up and complete? Thing is, the deprecated API is also still used internally,so anything but deprecated. Actually the new API calls the "deprecated" one. I just tried to adapt the code to the new deprecation macros (forgot that

D24495: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau updated this revision to Diff 68093. kossebau added a comment. add some missed deprecations REPOSITORY R284 KCompletion CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24495?vs=67734=68093 BRANCH useECMGenerateExportHeader REVISION DETAIL

D24675: Deprecate the KTcpSocket-based variant of SslUi::askIgnoreSslErrors

2019-10-16 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Looks good. Can't use the new deprecation syntax until D24684 is in. REPOSITORY R241 KIO BRANCH master REVISION DETAIL

D24716: Fix memory leak of KQuickAddons::ConfigModule objects

2019-10-16 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY When we create a KQuickAddons::ConfigModule in the factory we

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

2019-10-16 Thread Christoph Cullmann
cullmann added a comment. In D24568#548490 , @dfaure wrote: > Do we want these, found in https://code.qt.io/cgit/qt/qt5.git/tree/_clang-format? > > # We use template< without space. > SpaceAfterTemplateKeyword: false > > #

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

2019-10-16 Thread Christoph Cullmann
cullmann updated this revision to Diff 68091. cullmann marked 4 inline comments as done. cullmann added a comment. - fix coding style issue, we don't want indented case labels - add initial docs - adjust style - just tell the user it will not work REPOSITORY R240 Extra CMake Modules

KDE CI: Frameworks » kservice » kf5-qt5 FreeBSDQt5.13 - Build # 40 - Still Unstable!

2019-10-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kservice/job/kf5-qt5%20FreeBSDQt5.13/40/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 16 Oct 2019 21:04:52 + Build duration: 11 min and counting JUnit Tests Name:

D24665: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D24665 To: kossebau, #frameworks, dfaure Cc: bcooksley, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D21783: [WIP]Show more details in warning dialog shown before starting a privileged operation

2019-10-16 Thread David Faure
dfaure requested changes to this revision. dfaure added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > jobuidelegateextension.h:279 > + */ > +virtual bool askPrivilegeOperationConfirmation(const QString > , > +

D24598: Update elisa icon

2019-10-16 Thread Nathaniel Graham
ngraham added a comment. Let's get this in! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt, LeGast00n,

D24598: Update elisa icon

2019-10-16 Thread Nathaniel Graham
ngraham accepted this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt, LeGast00n, carneirogustavo, jguidon,

KDE CI: Frameworks » kcalendarcore » kf5-qt5 FreeBSDQt5.13 - Build # 15 - Still Unstable!

2019-10-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kcalendarcore/job/kf5-qt5%20FreeBSDQt5.13/15/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 16 Oct 2019 20:14:27 + Build duration: 39 min and counting JUnit Tests Name:

D24598: Update elisa icon

2019-10-16 Thread Matthieu Gallien
mgallien accepted this revision. mgallien added a comment. Thanks REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt,

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

2019-10-16 Thread David Faure
dfaure requested changes to this revision. dfaure added a comment. This revision now requires changes to proceed. I like the idea very much. INLINE COMMENTS > KDEClangFormat.cmake:76 > +else() > +message(STATUS "Could not set up the clang-format target as the > clang-format

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

2019-10-16 Thread David Faure
dfaure added a comment. Do we want these, found in https://code.qt.io/cgit/qt/qt5.git/tree/_clang-format? # We use template< without space. SpaceAfterTemplateKeyword: false # macros for which the opening brace stays attached. ForEachMacros: [ foreach, Q_FOREACH,

D24651: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes. Closed by commit R279:04dd50d62842: Use ECMGenerateExportHeader to manage deprecated API better (authored by kossebau). REPOSITORY R279 ThreadWeaver CHANGES SINCE LAST UPDATE

D24682: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau closed this revision. REVISION DETAIL https://phabricator.kde.org/D24682 To: kossebau, #frameworks, vkrause

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread David Faure
dfaure added a comment. OK, next step is to consider writing that separate class now rather than hoping to dump that task on a random future reader of the TODO ;-) At least, starting it is the best way to find out how feasible it is. REPOSITORY R263 KXmlGui REVISION DETAIL

D24710: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added a reviewer: Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KEmoticons to hide deprecated API up to a

D24537: Icons for windy weather

2019-10-16 Thread Carson Black
This revision was automatically updated to reflect the committed changes. Closed by commit R266:ed5d288ae908: Icons for windy weather (authored by cblack). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24537?vs=68079=68080 REVISION DETAIL

D24537: Icons for windy weather

2019-10-16 Thread Carson Black
cblack updated this revision to Diff 68079. cblack added a comment. Run icons through SVG cleaner REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24537?vs=68009=68079 BRANCH windy-weather-icons (branched from master) REVISION DETAIL

D24537: Icons for windy weather

2019-10-16 Thread Carson Black
cblack edited the test plan for this revision. REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D24537 To: cblack, #vdg, ndavis, ngraham Cc: trickyricky26, GB_2, vkrause, ndavis, kossebau, ngraham,

D24537: Icons for windy weather

2019-10-16 Thread TrickyRicky
trickyricky26 added a comment. With the blue color, I like these icons. Please remember to update images in the test plan when you make changes such as this color change, they're very helpful for judging things e.g. on a phone. REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons

D24707: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, drosca. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to BluezQt to hide deprecated API up to

D24641: Collect more information from version control systems

2019-10-16 Thread Thomas Fischer
thomasfischer updated this revision to Diff 68072. thomasfischer added a comment. I rewrote the patch to address the comments: 1. Setting `ECM_SOURCE_UNDER_VERSION_CONTROL` has been refactored to set variable `ECM_SOURCE_VERSION_CONTROL_WHICH` as well to determine which of three known

D24598: Update elisa icon

2019-10-16 Thread Nathaniel Graham
ngraham added a comment. In D24598#548335 , @astippich wrote: > This is with the tape window moved up 1px > F7608368: elisaBigNew2.png > When I look at images of real cassettes, they are not centered,

D24598: Update elisa icon

2019-10-16 Thread Alexander Stippich
astippich added a comment. This is with the tape window moved up 1px F7608368: elisaBigNew2.png When I look at images of real cassettes, they are not centered, hence why I only did this slight adjustment. If everyone is okay with this, I will

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread Méven Car
meven updated this revision to Diff 68067. meven added a comment. Update TODO REPOSITORY R263 KXmlGui CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24695?vs=68043=68067 BRANCH master REVISION DETAIL https://phabricator.kde.org/D24695 AFFECTED FILES src/kxmlguiclient.h

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread Méven Car
meven added a comment. In D24695#548248 , @dfaure wrote: > In my experience 90 % of the users of this class don't care for the state feature, so I would rather move it to a separate class. > > Also this class is used everywhere, so the

D24537: Icons for windy weather

2019-10-16 Thread Noah Davis
ndavis added a comment. Oh yes, that would be a good thing. Any of these will be fine: https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#SVG_optimization REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella, #frameworks Cc: kde-frameworks-devel, LeGast00n,

D24703: Port (most of) KSslInfoDialog from KSslError to QSslError

2019-10-16 Thread Volker Krause
vkrause added a task: T11620: Port from KSslError to QSslError. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24703 To: vkrause Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread David Edmundson
davidedmundson added a comment. Good rundown of the options. > remove SystemTray support from KdePlatformTheme It's not trivial. There's no way to access the relevant Qt classes. I should revisit the patch that allowed that. Our hopeful longterm plans for KF6 are to kill

D24703: Port (most of) KSslInfoDialog from KSslError to QSslError

2019-10-16 Thread Volker Krause
vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D24703 AFFECTED FILES src/core/ksslerror_p.h

D24700: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, ilic. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KI18n to hide deprecated API up to a

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread David Faure
dfaure added a comment. In my experience 90 % of the users of this class don't care for the state feature, so I would rather move it to a separate class. Also this class is used everywhere, so the change would have massive repercussions, while most users (both subclasses and code using

D24652: Add folder-recent, extend hand of clock in folder-temp

2019-10-16 Thread Nathaniel Graham
ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH recent-places (branched from master) REVISION DETAIL https://phabricator.kde.org/D24652 To: ndavis, #vdg, ngraham Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh,

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread Kai Uwe Broulik
broulik added a comment. > bool takeOwnership = true); If only Qt/we used modern C++ features to communicate object ownership :) REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella, #frameworks Cc:

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread Nathaniel Graham
ngraham edited the summary of this revision. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24697: Expose isImmutable to introspection (e.g. QML)

2019-10-16 Thread Kevin Ottens
This revision was automatically updated to reflect the committed changes. Closed by commit R237:3f68be12f87e: Expose isImmutable to introspection (e.g. QML) (authored by ervin). REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24697?vs=68046=68052 REVISION

D24697: Expose isImmutable to introspection (e.g. QML)

2019-10-16 Thread Kai Uwe Broulik
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D24697 To: ervin, #plasma, #frameworks, mart, dfaure, broulik Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24697: Expose isImmutable to introspection (e.g. QML)

2019-10-16 Thread Kevin Ottens
ervin created this revision. ervin added reviewers: Plasma, Frameworks, mart, dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ervin requested review of this revision. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D24697

D24663: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau added inline comments. INLINE COMMENTS > zzag wrote in plasmawindowmanagement_interface.cpp:741 > Hmm, PlasmaWindowInterface::setVirtualDesktop is wrapped in > KWAYLANDSERVER_ENABLE_DEPRECATED_SINCE in the header file. > > Is there a reason not to use

D24537: Icons for windy weather

2019-10-16 Thread Björn Feber
GB_2 added a comment. Can you clean the SVGs a bit? REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D24537 To: cblack, #vdg, ndavis, ngraham Cc: GB_2, vkrause, ndavis, kossebau, ngraham, kde-frameworks-devel,

D24537: Icons for windy weather

2019-10-16 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Shipit! REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D24537 To: cblack, #vdg, ndavis, ngraham Cc: vkrause, ndavis, kossebau, ngraham,

D24684: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau edited the summary of this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24684 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D18299: ComboBox: fix default delegate

2019-10-16 Thread Kai Uwe Broulik
broulik added a comment. I checked Qt source code and Qt indeed should handle this on its own, it also works fine hwere without the code. Worse, the explicit assignment from QML breaks any binding set on `currentIndex`. In the handler that's called when the delegate model creates a

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread Méven Car
meven added a comment. This is just to make sure this TODO makes sense and is consensual. REPOSITORY R263 KXmlGui REVISION DETAIL https://phabricator.kde.org/D24695 To: meven, #frameworks, kossebau, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24695: Add a KF6 TODO to make KXMLGUIClient generic over a state enum

2019-10-16 Thread Méven Car
meven created this revision. meven added reviewers: Frameworks, kossebau, dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY To avoid string typing, replace usage of string with a proper enum

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-16 Thread Ahmad Samir
ahmadsamir abandoned this revision. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel, mkoller Cc: cullmann, ognarb, kde-frameworks-devel

D24689: Add support for SAE authentication used by WPA3

2019-10-16 Thread Jan Grulich
jgrulich added a dependent revision: D24691: Add support for WPA3. REPOSITORY R282 NetworkManagerQt REVISION DETAIL https://phabricator.kde.org/D24689 To: jgrulich, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread Konrad Materka
kmaterka added a comment. I think the best would be to add additional parameter to KSNI. Change from: void KStatusNotifierItem::setContextMenu(QMenu *menu); to something like this: void KStatusNotifierItem::setContextMenu(QMenu *menu, bool takeOwnership = true); Then,

D24423: Rename "Internet" category to "Network" and remove "Internet>Terminal" sub-category

2019-10-16 Thread Yunhe Guo
guoyunhe added a comment. `kf5-network.directory` has been added to `plasma-workspace`. The trouble is still here: if we land this patch and released new frameworks, it won't work with older plasma workspace (like 5.17.0, 5.16.x). REPOSITORY R309 KService REVISION DETAIL

D24689: Add support for SAE authentication used by WPA3

2019-10-16 Thread Jan Grulich
jgrulich updated this revision to Diff 68029. jgrulich added a comment. Compare against lowercase value REPOSITORY R282 NetworkManagerQt CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24689?vs=68028=68029 BRANCH wpa3 REVISION DETAIL https://phabricator.kde.org/D24689

D24689: Add support for SAE authentication used by WPA3

2019-10-16 Thread Jan Grulich
jgrulich added a reviewer: Frameworks. REPOSITORY R282 NetworkManagerQt REVISION DETAIL https://phabricator.kde.org/D24689 To: jgrulich, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24689: Add support for SAE authentication used by WPA3

2019-10-16 Thread Jan Grulich
jgrulich created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. jgrulich requested review of this revision. REPOSITORY R282 NetworkManagerQt BRANCH wpa3 REVISION DETAIL https://phabricator.kde.org/D24689 AFFECTED FILES

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.13 - Build # 153 - Fixed!

2019-10-16 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.13/153/ Project: kf5-qt5 SUSEQt5.13 Date of build: Wed, 16 Oct 2019 08:39:36 + Build duration: 16 min and counting BUILD ARTIFACTS

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.13 - Build # 140 - Still Unstable!

2019-10-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/140/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 16 Oct 2019 08:39:36 + Build duration: 12 min and counting JUnit Tests Name: projectroot

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread Konrad Materka
kmaterka added a reviewer: Frameworks. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24667: [KStatusNotifierItem] Allow left click when menu is null

2019-10-16 Thread Konrad Materka
kmaterka retitled this revision from "Activate when both associatedWidget and menu are null" to "[KStatusNotifierItem] Allow left click when menu is null". REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D24667 To: kmaterka, davidedmundson, broulik, nicolasfella

D24463: Treat "application/x-ms-dos-executable" as executable on all platforms

2019-10-16 Thread Phabricator
This revision was automatically updated to reflect the committed changes. Closed by commit R241:591cc2a16e3e: Treat application/x-ms-dos-executable as executable on all platforms (authored by Guo Yunhe i...@guoyunhe.me). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE

D24663: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Vlad Zahorodnii
zzag edited reviewers, added: KWin; removed: Plasma. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D24663 To: kossebau, #kwin, #plasma Cc: zzag, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24663: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Vlad Zahorodnii
zzag added inline comments. INLINE COMMENTS > plasmawindowmanagement_interface.cpp:741 > > -#ifndef KWAYLANDSERVER_NO_DEPRECATED > void PlasmaWindowInterface::setVirtualDesktop(quint32 desktop) Hmm, PlasmaWindowInterface::setVirtualDesktop is wrapped in

D24684: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread David Faure
dfaure added a comment. The commit log talks about KCalendarCore instead of KIO :) REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D24684 To: kossebau, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24686: Replace usage of deprecated SlaveBase::config() by SlaveBase::configValue

2019-10-16 Thread Méven Car
meven created this revision. meven added a reviewer: dfaure. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY With D23523 and D24582

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.13 - Build # 139 - Still Unstable!

2019-10-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.13/139/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Wed, 16 Oct 2019 07:04:32 + Build duration: 40 min and counting JUnit Tests Name: projectroot

KDE CI: Frameworks » kio » kf5-qt5 SUSEQt5.13 - Build # 152 - Unstable!

2019-10-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.13/152/ Project: kf5-qt5 SUSEQt5.13 Date of build: Wed, 16 Oct 2019 07:07:14 + Build duration: 32 min and counting BUILD ARTIFACTS

D24684: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau updated this revision to Diff 68021. kossebau added a comment. update to latest master REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24684?vs=68018=68021 BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24684

D24685: Replace usage of SlaveBase::config()->readEntry by SlaveBase::configValue

2019-10-16 Thread Méven Car
meven created this revision. meven added a reviewer: dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY Followup after D24582 There are only 5 usage of

D24684: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau added a comment. No support for EXCLUDE_DEPRECATED_BEFORE_AND_AT here as well, that needs another round of work. Thus the removal of all the KIOWIDGETS_NO_DEPRECATED from the source files, no *_BUILD_DEPRECATED_SINCE there to properly replace them for now. So patch solely

D24684: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Friedrich W. H. Kossebau
kossebau created this revision. kossebau added reviewers: Frameworks, dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kossebau requested review of this revision. REVISION SUMMARY Allows - projects linking to KCalendarCore to hide deprecated API

D24582: Replace usage of SlaveBase::config() by SlaveBase::mapConfig()

2019-10-16 Thread Méven Car
This revision was automatically updated to reflect the committed changes. Closed by commit R241:0911763b8f19: Replace usage of SlaveBase::config() by SlaveBase::mapConfig() (authored by meven). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24582?vs=67943=68017

D24582: Replace usage of SlaveBase::config() by SlaveBase::mapConfig()

2019-10-16 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO BRANCH arcpatch-D24582 REVISION DETAIL https://phabricator.kde.org/D24582 To: meven, #frameworks, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24682: Use ECMGenerateExportHeader to manage deprecated API better

2019-10-16 Thread Volker Krause
vkrause accepted this revision. This revision is now accepted and ready to land. BRANCH useECMGenerateExportHeader REVISION DETAIL https://phabricator.kde.org/D24682 To: kossebau, #frameworks, vkrause

D24537: Icons for windy weather

2019-10-16 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. This revision is now accepted and ready to land. I'm fine with this blue. REPOSITORY R266 Breeze Icons BRANCH windy-weather-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D24537 To: cblack, #vdg, ndavis