D25123: New yakuake logo/icon

2020-02-06 Thread David Redondo
davidre added a comment. If you land this please update the summary before. No need to have all the variants in the commit message it should be about the actual change. Thanks :) REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D25123 To: mglb, #vdg, #yakuake,

D27141: Drop Policykit backend

2020-02-06 Thread Luca Beltrame
lbeltrame accepted this revision. This revision is now accepted and ready to land. REPOSITORY R283 KAuth BRANCH no_policykit REVISION DETAIL https://phabricator.kde.org/D27141 To: davidedmundson, lbeltrame Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27067: Change "AutoSkip" to "Skip All"

2020-02-06 Thread Nathaniel Graham
ngraham added a comment. Ping. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D27067 To: ngraham, #frameworks, #vdg, dfaure Cc: broulik, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27080: Don't warn for invalid plugin metata

2020-02-06 Thread Nathaniel Graham
ngraham updated this revision to Diff 75149. ngraham added a comment. Use `qCDebug()` instead of removing it REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27080?vs=74823=75149 BRANCH less-warning-spam (branched from master) REVISION

D27185: Use the right translation catalog to show translations

2020-02-06 Thread Victor Ryzhykh
victorr added a comment. Before F8089258: Before.jpg After F8089261: After.jpg REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D27185 To: yurchor, #frameworks Cc: victorr,

D25123: New yakuake logo/icon

2020-02-06 Thread Mariusz Glebocki
mglb retitled this revision from "[WIP] New yakuake logo/icon" to "New yakuake logo/icon". REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D25123 To: mglb, #vdg, #yakuake, hein Cc: GB_2, ndavis, ognarb, ngraham, hein, kde-frameworks-devel, numerodix, vmarinescu,

D25123: [WIP] New yakuake logo/icon

2020-02-06 Thread Mariusz Glebocki
mglb updated this revision to Diff 75148. mglb added a comment. Huh, I was waiting for @hein :) oh well, here it is - C icon with cleaned up source REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D25123?vs=69507=75148 BRANCH yakuake REVISION DETAIL

D27203: Don't try to open files we can't figure out where they are

2020-02-06 Thread Aleix Pol Gonzalez
apol created this revision. apol added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. TEST PLAN Been using it for a few days, no regressions. Opening sftp:///localhost/.../somefile.text twice, doesn't

D27200: Make the attached property a simple QObject

2020-02-06 Thread Aleix Pol Gonzalez
apol added a comment. LGTM otherwise INLINE COMMENTS > colorscope.cpp:57 > +connect(parentItem, ::windowChanged, > this, [this]() { > findParentScope(); let's make this a proper method? it's used twice here and a normal method should be quicker to use

D27122: Tweaked contrast effect values, adjusted transparency accordingly, switched from Background to ViewBackground

2020-02-06 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. Can confirm that it works, and I do think it looks better now. I'm not an expert on the SVG changes but I don't see anything obviously wrong in there. Please add a comment in the Description section of this patch explaining why

D27122: Tweaked contrast effect values, adjusted transparency accordingly, switched from Background to ViewBackground

2020-02-06 Thread Niccolò Venerandi
niccolove marked an inline comment as done. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27122 To: niccolove, ndavis, ngraham Cc: ndavis, filipf, ngraham, davidedmundson, kde-frameworks-devel, manueljlin, Orage, LeGast00n, The-Feren-OS-Dev,

D27122: Tweaked contrast effect values, adjusted transparency accordingly, switched from Background to ViewBackground

2020-02-06 Thread Niccolò Venerandi
niccolove updated this revision to Diff 75145. niccolove added a comment. Fixed mistakes in breeze dark cmakelist REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27122?vs=75019=75145 BRANCH contrast_effect_background_color (branched

D27103: Convert license headers to SPDX markers

2020-02-06 Thread Christophe Giboudeaux
cgiboudeaux added inline comments. INLINE COMMENTS > modeltest.h:4 > > +SPDX-FileCopyrightText: 2013 Digia Plc and /or its subsidiary(-ies) > > +SPDX-FileCopyrightText: 2010 Stephen Kelly redirects to https://www.qt.io/terms-conditions/ nowadays

D27103: Convert license headers to SPDX markers

2020-02-06 Thread Christophe Giboudeaux
cgiboudeaux added a comment. same remark as for D27106 . Check if FileContributor is more appropriate. REPOSITORY R275 KItemModels REVISION DETAIL https://phabricator.kde.org/D27103 To: cordlandwehr, dfaure Cc: cgiboudeaux, kde-frameworks-devel,

D27091: Convert license statements to SPDX

2020-02-06 Thread Christophe Giboudeaux
cgiboudeaux accepted this revision. This revision is now accepted and ready to land. REPOSITORY R246 Sonnet BRANCH spdx REVISION DETAIL https://phabricator.kde.org/D27091 To: cordlandwehr, cgiboudeaux Cc: cgiboudeaux, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27187: update travel-family icons

2020-02-06 Thread Nathaniel Graham
ngraham added a comment. These are lovely icons, BTW. Very nice work. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg, ndavis Cc: ngraham, nicolasfella, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D27187: update travel-family icons

2020-02-06 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:cf10c681f350: update travel-family icons (authored by mbruchert, committed by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75132=75135

D27187: update travel-family icons

2020-02-06 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. This revision is now accepted and ready to land. This looks ready to go. I'll do some additional optimization later. REPOSITORY R266 Breeze Icons BRANCH 4px REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg,

D27200: Make the attached property a simple QObject

2020-02-06 Thread Nicolas Fella
nicolasfella retitled this revision from "Make the attached proeprty a simple QObject" to "Make the attached property a simple QObject". REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27200 To: mart, bshah, #plasma Cc: kde-frameworks-devel,

D27200: Make the attached proeprty a simple QObject

2020-02-06 Thread Marco Martin
mart edited the test plan for this revision. mart added reviewers: bshah, Plasma. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27200 To: mart, bshah, #plasma Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27200: Make the attached proeprty a simple QObject

2020-02-06 Thread Marco Martin
mart created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY move all the logic in an attached property QObject it's a bit less heavy to create so should make things slightly better

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75132. mbruchert added a comment. remove accidentially commited icons REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75131=75132 BRANCH 4px REVISION DETAIL https://phabricator.kde.org/D27187

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75131. mbruchert added a comment. remove accidentially commited files REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75130=75131 BRANCH 4px REVISION DETAIL https://phabricator.kde.org/D27187

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75130. mbruchert added a comment. - screenshot icon - 4px border REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75124=75130 BRANCH 4px REVISION DETAIL https://phabricator.kde.org/D27187

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Christoph Feck
cfeck added a comment. Refresh is fine, Reload could be irritating for applications that don't load anything, but still know the user could be faced with a view state that needs redrawing, e.g. when cleaning corruption caused by non-integer scale factors. REPOSITORY R265 KConfigWidgets

D27187: update travel-family icons

2020-02-06 Thread Noah Davis
ndavis added a comment. Much better. Now the margins need to be 4px rather than 3px. If you haven't already, make sure you read the HIG section on icons and the workflow tips wiki page it links to. https://hig.kde.org/style/icons/index.html

D27187: update travel-family icons

2020-02-06 Thread Nicolas Fella
nicolasfella edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg, ndavis Cc: ngraham, nicolasfella, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert added a comment. F8088755: image.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg, ndavis Cc: ngraham, nicolasfella, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns

D27197: Add symbolic screenshot icons

2020-02-06 Thread Mathis Brüchert
mbruchert created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mbruchert requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH sreenshot REVISION DETAIL https://phabricator.kde.org/D27197 AFFECTED FILES

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75124. mbruchert added a comment. fix everything for real REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75121=75124 BRANCH screenshot REVISION DETAIL https://phabricator.kde.org/D27187 AFFECTED

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Nathaniel Graham
ngraham added a comment. All right, "Refresh" it is, then! REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27195 To: ngraham, #vdg, The-Feren-OS-Dev, ndavis Cc: kde-frameworks-devel, The-Feren-OS-Dev, LeGast00n, GB_2, michaelh, ngraham, bruns

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R265:fe1d3a3f451a: Change Redisplay to Refresh (authored by ngraham). REPOSITORY R265 KConfigWidgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27195?vs=75113=75122 REVISION DETAIL

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75121. mbruchert added a comment. - fix everything(px) REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75116=75121 BRANCH screenshot REVISION DETAIL https://phabricator.kde.org/D27187 AFFECTED

D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0)

2020-02-06 Thread Noah Davis
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 R273:2dc164e18a95: [KColorUtils] Change getHcy() hue range to [0.0, 1.0) (authored by ndavis). REPOSITORY R273

D27017: [KColorUtils] Add hue(), chroma(), hcyColor() and update documentation

2020-02-06 Thread Noah Davis
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 R273:4de9a9318e61: [KColorUtils] Add hue(), chroma(), hcyColor() and update documentation (authored by ndavis). REPOSITORY

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75116. mbruchert added a comment. fix REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75114=75116 BRANCH screenshot REVISION DETAIL https://phabricator.kde.org/D27187 AFFECTED FILES

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75114. mbruchert added a comment. - screenshot icon REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75092=75114 BRANCH screenshot REVISION DETAIL https://phabricator.kde.org/D27187 AFFECTED FILES

D27187: update travel-family icons

2020-02-06 Thread Nathaniel Graham
ngraham added a comment. I think he meant to add some screenshots to the Test Plan section of this patch so people can see the new icons without having to apply it. :) REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg, ndavis Cc:

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Dominic Hayes
The-Feren-OS-Dev added a comment. In D27195#606954 , @ngraham wrote: > "Reload" could also work, as that's also a very common and recognizable term. That is a good point but if you call it "Reload" you might as well proceed to also

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Noah Davis
ndavis accepted this revision. REPOSITORY R265 KConfigWidgets BRANCH redisplay-to-refresh (branched from master) REVISION DETAIL https://phabricator.kde.org/D27195 To: ngraham, #vdg, The-Feren-OS-Dev, ndavis Cc: kde-frameworks-devel, The-Feren-OS-Dev, LeGast00n, GB_2, michaelh, ngraham,

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Dominic Hayes
The-Feren-OS-Dev accepted this revision. The-Feren-OS-Dev added a comment. This revision is now accepted and ready to land. This is something I've recently suggested myself, so having this happen would be a small change but still a good change. Furthermore this'll make it consistent with

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Nathaniel Graham
ngraham added a comment. "Reload" could also work, as that's also a very common and recognizable term. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27195 To: ngraham, #vdg Cc: kde-frameworks-devel, The-Feren-OS-Dev, LeGast00n, GB_2, michaelh, ngraham,

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Nathaniel Graham
ngraham edited the summary of this revision. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27195 To: ngraham, #vdg Cc: kde-frameworks-devel, The-Feren-OS-Dev, LeGast00n, GB_2, michaelh, ngraham, bruns

D27195: Change "Redisplay" to "Refresh"

2020-02-06 Thread Nathaniel Graham
ngraham created this revision. ngraham added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ngraham requested review of this revision. REVISION SUMMARY "Redisplay" is an unusual and archaic term which has ben supplanted in common use

D27150: Add ecm_qt_install_logging_categories & ecm_qt_export_logging_category

2020-02-06 Thread Laurent Montel
mlaurent added a comment. I don't see other case. For me your macro works for all case that we can implement. No other idea. But avoiding to create by hand the file is a good idea. I never successed to create a macro for it So thanks REPOSITORY R240 Extra CMake Modules BRANCH

D27122: Tweaked contrast effect values, adjusted transparency accordingly, switched from Background to ViewBackground

2020-02-06 Thread Niccolò Venerandi
niccolove added a comment. Nope, I'm absolutely unable to build plasma-framework right now, I have many problems that I don't understand :-/ Also, I have 0 Cmake experience If somebody has time, can they look into how to make Cmake read that file? REPOSITORY R242 Plasma Framework

D27150: Add ecm_qt_install_logging_categories & ecm_qt_export_logging_category

2020-02-06 Thread Friedrich W. H. Kossebau
kossebau added a comment. @mlaurent Thanks for review :) Do you happen to know any more complex usages of ecm_qt_declare_logging_category and/or manual category definitions which can and should be checked for how these new methods would work out? I have not really researched bigger parts

D27187: update travel-family icons

2020-02-06 Thread Nicolas Fella
nicolasfella added a comment. Please add some screenshots REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27187 To: mbruchert, #vdg, ndavis Cc: nicolasfella, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27017: [KColorUtils] Add hue(), chroma(), hcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis added a reviewer: mwoehlke. REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27017 To: ndavis, #frameworks, dfaure, mwoehlke Cc: mwoehlke, broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0)

2020-02-06 Thread Noah Davis
ndavis added a reviewer: mwoehlke. REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27192 To: ndavis, #frameworks, dfaure, mwoehlke Cc: kde-frameworks-devel, mwoehlke, LeGast00n, GB_2, michaelh, ngraham, bruns

D27017: [KColorUtils] Add hue(), chroma(), hcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis retitled this revision from "[KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation" to "[KColorUtils] Add hue(), chroma(), hcyColor() and update documentation". ndavis edited the summary of this revision. REPOSITORY R273 KGuiAddons REVISION DETAIL

D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0)

2020-02-06 Thread Noah Davis
ndavis added a dependency: D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation. REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27192 To: ndavis, #frameworks, dfaure Cc: kde-frameworks-devel, mwoehlke, LeGast00n, GB_2, michaelh,

D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis marked an inline comment as done. REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27017 To: ndavis, #frameworks, dfaure Cc: mwoehlke, broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis added a dependent revision: D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0). REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27017 To: ndavis, #frameworks, dfaure Cc: mwoehlke, broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham,

D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0)

2020-02-06 Thread Noah Davis
ndavis added a reviewer: dfaure. REPOSITORY R273 KGuiAddons REVISION DETAIL https://phabricator.kde.org/D27192 To: ndavis, #frameworks, dfaure Cc: kde-frameworks-devel, mwoehlke, LeGast00n, GB_2, michaelh, ngraham, bruns

D27192: [KColorUtils] Change getHcy() hue range to [0.0, 1.0)

2020-02-06 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY As suggested by @mwoehlke in D27017 Users

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

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwayland/job/kf5-qt5%20FreeBSDQt5.13/51/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Thu, 06 Feb 2020 11:29:16 + Build duration: 32 min and counting JUnit Tests Name:

D27187: update travel-family icons

2020-02-06 Thread Noah Davis
ndavis requested changes to this revision. ndavis added a comment. This revision now requires changes to proceed. Whoops, looks like you accidentally made them 48x48 mm instead of 48x48 px :) You'll need to work on the pixel alignment a bit because shrinking them down to the correct size

KDE CI: Frameworks » kwayland » kf5-qt5 SUSEQt5.12 - Build # 98 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwayland/job/kf5-qt5%20SUSEQt5.12/98/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 06 Feb 2020 11:29:15 + Build duration: 17 min and counting BUILD ARTIFACTS

KDE CI: Frameworks » kwayland » kf5-qt5 SUSEQt5.13 - Build # 57 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwayland/job/kf5-qt5%20SUSEQt5.13/57/ Project: kf5-qt5 SUSEQt5.13 Date of build: Thu, 06 Feb 2020 11:29:14 + Build duration: 17 min and counting BUILD ARTIFACTS

D27190: [surface] Fix buffer offset when attaching buffers to surfaces

2020-02-06 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R127:f8760c97b65e: [surface] Fix buffer offset when attaching buffers to surfaces (authored by davidedmundson). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE

D27190: [surface] Fix buffer offset when attaching buffers to surfaces

2020-02-06 Thread Vlad Zahorodnii
zzag accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D27190 To: davidedmundson, #kwin, zzag Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D27190: [surface] Fix buffer offset when attaching buffers to surfaces

2020-02-06 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. TEST PLAN Unit test Will be tested by hotspots in drag icons REPOSITORY R127

D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis added inline comments. INLINE COMMENTS > mwoehlke wrote in kcolorutils.cpp:60 > I guess it will be less confusing if this is also normalized? > > *h = khcy.h + (khcy.h < 0.0 ? 1.0 : 0.0); I should probably put this change in another commit so as not to do too much at once, so I'm

D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis updated this revision to Diff 75097. ndavis added a comment. - Remove getHcy hue value change REPOSITORY R273 KGuiAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27017?vs=75094=75097 BRANCH hue-chroma (branched from master) REVISION DETAIL

D27017: [KColorUtils] Add hue(), chroma(), getHcyColor() and update documentation

2020-02-06 Thread Noah Davis
ndavis updated this revision to Diff 75094. ndavis added a comment. - Update @since version and fix hcyColor comment edit I missed REPOSITORY R273 KGuiAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27017?vs=75022=75094 BRANCH hue-chroma (branched from master) REVISION

KDE CI: Frameworks » kitemmodels » kf5-qt5 SUSEQt5.13 - Build # 63 - Fixed!

2020-02-06 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20SUSEQt5.13/63/ Project: kf5-qt5 SUSEQt5.13 Date of build: Thu, 06 Feb 2020 10:37:23 + Build duration: 8 min 5 sec and counting BUILD ARTIFACTS

KDE CI: Frameworks » kitemmodels » kf5-qt5 SUSEQt5.12 - Build # 86 - Fixed!

2020-02-06 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20SUSEQt5.12/86/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 06 Feb 2020 10:37:24 + Build duration: 6 min 35 sec and counting BUILD ARTIFACTS

KDE CI: Frameworks » kitemmodels » kf5-qt5 WindowsMSVCQt5.14 - Build # 5 - Fixed!

2020-02-06 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20WindowsMSVCQt5.14/5/ Project: kf5-qt5 WindowsMSVCQt5.14 Date of build: Thu, 06 Feb 2020 10:37:24 + Build duration: 5 min 49 sec and counting JUnit Tests

KDE CI: Frameworks » kitemmodels » kf5-qt5 FreeBSDQt5.13 - Build # 60 - Fixed!

2020-02-06 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20FreeBSDQt5.13/60/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Thu, 06 Feb 2020 10:37:23 + Build duration: 3 min 39 sec and counting JUnit Tests Name:

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert updated this revision to Diff 75092. mbruchert added a comment. optimize svgs REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27187?vs=75091=75092 BRANCH travel-family REVISION DETAIL https://phabricator.kde.org/D27187 AFFECTED FILES

D27187: update travel-family icons

2020-02-06 Thread Mathis Brüchert
mbruchert created this revision. mbruchert added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mbruchert requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH travel-family REVISION DETAIL

D27059: KConfigSkeletonItem : allow to set a KconfigGroup to read and write items in nested groups

2020-02-06 Thread Cyril Rossi
crossi edited the summary of this revision. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27059 To: crossi, ervin, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

KDE CI: Frameworks » kitemmodels » kf5-qt5 SUSEQt5.13 - Build # 62 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20SUSEQt5.13/62/ Project: kf5-qt5 SUSEQt5.13 Date of build: Thu, 06 Feb 2020 09:35:14 + Build duration: 8 min 33 sec and counting BUILD ARTIFACTS

KDE CI: Frameworks » kitemmodels » kf5-qt5 SUSEQt5.12 - Build # 85 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20SUSEQt5.12/85/ Project: kf5-qt5 SUSEQt5.12 Date of build: Thu, 06 Feb 2020 09:35:12 + Build duration: 4 min 45 sec and counting BUILD ARTIFACTS

KDE CI: Frameworks » kitemmodels » kf5-qt5 WindowsMSVCQt5.14 - Build # 4 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20WindowsMSVCQt5.14/4/ Project: kf5-qt5 WindowsMSVCQt5.14 Date of build: Thu, 06 Feb 2020 09:35:15 + Build duration: 4 min 19 sec and counting JUnit Tests

KDE CI: Frameworks » kitemmodels » kf5-qt5 FreeBSDQt5.13 - Build # 59 - Unstable!

2020-02-06 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kitemmodels/job/kf5-qt5%20FreeBSDQt5.13/59/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Thu, 06 Feb 2020 09:35:13 + Build duration: 1 min 51 sec and counting JUnit Tests Name:

D25326: Move Plasma's SortFilterProxyModel into KItemModel's QML plugin

2020-02-06 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R275:4fd7a211bb8d: Move Plasmas SortFilterProxyModel into KItemModels QML plugin (authored by ahiemstra, committed by davidedmundson). CHANGED PRIOR TO COMMIT