D27502: Create ConfigView an unmanaged ConfigWidget

bport added inline comments. INLINE COMMENTS > ervin wrote in settings.cpp:109 > This is an odd bit of logic, isn't it? Yes indeed, but what we want is to save false if checked and true if not checked. I moved the logic there in order to work with kconfigxt and with current implementation > e

D27642: RFC WIP support OSX style device icons system for dns-sd discoveries

sitter created this revision. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. sitter requested review of this revision. REVISION SUMMARY looking for some general thoughts on the concept osx and newer sambas send model data in the dnssd

D27133: kconfig_compiler : generate kconfig settings with subgroup

crossi added a dependent revision: D27188: KCM Notifications : Manage app-specific notifications with KCconfigXT's magic. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27133 To: crossi, ervin, dfaure, #frameworks Cc: meven, kde-frameworks-devel, LeGast00n, cblack, GB_

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

crossi added a dependent revision: D27188: KCM Notifications : Manage app-specific notifications with KCconfigXT's magic. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27059 To: crossi, ervin, dfaure, #frameworks, mdawson Cc: kde-frameworks-devel, LeGast00n, cblack, G

KDE CI: Frameworks » kirigami » kf5-qt5 SUSEQt5.12 - Build # 356 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kirigami/job/kf5-qt5%20SUSEQt5.12/356/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 25 Feb 2020 10:23:34 + Build duration: 3 min 48 sec and counting BUILD ARTIFACTS abi-compatib

KDE CI: Frameworks » kirigami » kf5-qt5 SUSEQt5.13 - Build # 209 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kirigami/job/kf5-qt5%20SUSEQt5.13/209/ Project: kf5-qt5 SUSEQt5.13 Date of build: Tue, 25 Feb 2020 10:23:34 + Build duration: 3 min 50 sec and counting BUILD ARTIFACTS acc/KF5Kirig

D27644: do not hide enum warnings via switch default label

sitter created this revision. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. sitter requested review of this revision. REVISION SUMMARY should we ever extend the url types the compiler will not warn us that the new type may need handling.

D27644: do not hide enum warnings via switch default label

sitter added a reviewer: ngraham. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D27644 To: sitter, ngraham Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh,

D27616: [UrlHandler] optimise the code and less if nesting

ahmadsamir added a comment. I created D27615 first; then this one on top of it (optimising the code in a separate commit). INLINE COMMENTS > apol wrote in urlhandler.cpp:55 > won't this recurse forever? I could be wrong, but this is exactly equivalent to

D27646: smb: apply a whole bunch of static analyzer improvements

sitter created this revision. sitter added a reviewer: ngraham. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. sitter requested review of this revision. REVISION SUMMARY among other things: - {} initialize the posix structs - =defaul

D27647: throw out the windows variant of the smb slave

sitter created this revision. sitter added a reviewer: ngraham. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. sitter requested review of this revision. REVISION SUMMARY it hasn't been built in ages because the top level cmakelists actually

D27647: throw out the windows variant of the smb slave

sitter added a reviewer: Windows. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D27647 To: sitter, ngraham, #windows Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven,

D27633: Port to KPluginLoader

leinir added inline comments. INLINE COMMENTS > kaccountsdplugin.cpp:24 > > -KAccountsDPlugin::KAccountsDPlugin(QObject *parent) > +KAccountsDPlugin::KAccountsDPlugin(QObject *parent, const QVariantList& args) > : QObject(parent) This caused a compiler warning to show up - pushed a quick

D23339: Fix memory leak in ConfigView and Dialog

This revision was automatically updated to reflect the committed changes. Closed by commit R242:4b93b71a4924: Fix memory leak in ConfigView and Dialog (authored by dvratil). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23339?vs=76305&id=76

D27627: Fix initialization order

This revision was automatically updated to reflect the committed changes. Closed by commit R242:1e11099c1616: Fix initialization order (authored by dvratil). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27627?vs=76306&id=76361 REVISION DE

D27540: Status indicator for individual widgets in KCModule

ervin added a comment. In D27540#615091 , @davidre wrote: > The editmarks should probably respect the visibility of the associated widget. In this picture I use a invisble lineedit to manage the settings of the QButtonGroup beneath (QButtonGroup

D27540: Status indicator for individual widgets in KCModule

ervin updated this revision to Diff 76362. ervin added a comment. Handle visibility of the tracked widget, setters become slots, and add plural to "indicatorWidgets". REPOSITORY R265 KConfigWidgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27540?vs=76089&id=76362 REVISION D

D27540: Status indicator for individual widgets in KCModule

ervin edited the test plan for this revision. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27540 To: ervin, ngraham, davidedmundson, meven, crossi, bport, #vdg Cc: iasensio, davidre, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27540: Status indicator for individual widgets in KCModule

ervin added a comment. In D27540#615160 , @ervin wrote: > In D27540#615156 , @ngraham wrote: > > > Please add screenshots to the Test Plan section for patches that change the UI. :) Also it would be

D21902: Calendar: add method to show event details

dvratil added a reviewer: Plasma. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D21902 To: dvratil, mart, #frameworks, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D21902: Calendar: add method to show event details

dvratil updated this revision to Diff 76365. dvratil added a comment. - Rebase to master REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21902?vs=60891&id=76365 BRANCH arcpatch-D21902 REVISION DETAIL https://phabricator.kde.org/D2190

D27540: KCModule: Indicate when a setting has been changed from the default value

ngraham retitled this revision from "Status indicator for individual widgets in KCModule" to "KCModule: Indicate when a setting has been changed from the default value". REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27540 To: ervin, ngraham, davidedmundson, me

D27540: KCModule: Indicate when a setting has been changed from the default value

davidre added a comment. That title is now not optimal isn't it? 1. It's not only in KCModule 2. It also shows unsaved settings REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27540 To: ervin, ngraham, davidedmundson, meven, crossi, bport, #vdg Cc: iasensio,

D27497: Fix code generation for entries with min/max

ervin edited the summary of this revision. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27497 To: hchain, meven, crossi, ervin, bport, tcanabrava Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27540: KCModule: Indicate when a setting has been changed from the default or previous value

ngraham retitled this revision from "KCModule: Indicate when a setting has been changed from the default value" to "KCModule: Indicate when a setting has been changed from the default or previous value". REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27540 To:

D27497: Fix code generation for entries with min/max

ervin accepted this revision. This revision is now accepted and ready to land. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27497 To: hchain, meven, crossi, ervin, bport, tcanabrava Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27540: KCModule: Indicate when a setting has been changed from the default or previous value

ndavis added a comment. I can see the utility in indicating non-default values in some cases, particularly with software that has a lot of necessary complexity in the settings or where non-default values can cause problems (see SVG Cleaner GUI for an example of both). However, I don't think

D27497: Fix code generation for entries with min/max

hchain updated this revision to Diff 76368. hchain added a comment. add bugzilla REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27497?vs=76317&id=76368 REVISION DETAIL https://phabricator.kde.org/D27497 AFFECTED FILES autotests/kconfig_compiler/CMakeL

D27133: kconfig_compiler : generate kconfig settings with subgroup

dfaure added a comment. I really have zero experience with this stuff, but you're asking for my opinion nonetheless, so you'll get it, as crappy as it might be :-) Your comment refers to "group separator" as if it was an existing notion in KConfig. Can you point me where? Grepping for '2

D27497: Fix code generation for entries with min/max

This revision was automatically updated to reflect the committed changes. Closed by commit R237:d218b93a5350: Fix code generation for entries with min/max (authored by hchain). REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27497?vs=76368&id=76371 REVISION DE

KDE CI: Frameworks » kconfig » kf5-qt5 FreeBSDQt5.13 - Build # 85 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kconfig/job/kf5-qt5%20FreeBSDQt5.13/85/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 25 Feb 2020 14:58:30 + Build duration: 1 min 6 sec and counting JUnit Tests Name: project

KDE CI: Frameworks » kconfig » kf5-qt5 SUSEQt5.13 - Build # 91 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kconfig/job/kf5-qt5%20SUSEQt5.13/91/ Project: kf5-qt5 SUSEQt5.13 Date of build: Tue, 25 Feb 2020 14:58:30 + Build duration: 2 min 16 sec and counting BUILD ARTIFACTS abi-compatibil

D27133: kconfig_compiler : generate kconfig settings with subgroup

ervin added a comment. In D27133#617577 , @dfaure wrote: > I really have zero experience with this stuff, but you're asking for my opinion nonetheless, so you'll get it, as crappy as it might be :-) > > Your comment refers to "group separator

KDE CI: Frameworks » kconfig » kf5-qt5 SUSEQt5.12 - Build # 123 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kconfig/job/kf5-qt5%20SUSEQt5.12/123/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 25 Feb 2020 14:58:30 + Build duration: 4 min 54 sec and counting BUILD ARTIFACTS abi-compatibi

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

niccolove updated this revision to Diff 76373. niccolove added a comment. Tweaked values REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27122?vs=75787&id=76373 BRANCH contrast_effect_background_color (branched from master) REVISION DE

D27540: KCModule: Indicate when a setting has been changed from the default or previous value

alexde added a comment. Does this patch also fix indirectly #274629? REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D27540 To: ervin, ngraham, davidedmundson, meven, crossi, bport, #vdg Cc: alexde, ndavis, iasensio, davidre, kde-frameworks-devel, LeGast00n, cb

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

niccolove added a comment. I felt that the transparency was not working very well with the headerbar on some wallpapers, so I toned down the transparency a bit, going from F8131395: image.png to F8131397: image.png

D27097: Port from QRegExp to QRegularExpression

ahmadsamir marked an inline comment as done. ahmadsamir added a comment. In D27097#615648 , @dfaure wrote: > OK for deprecating Incremental. A full revert (at KF6 time) of commit 9ac82e27ad0322e444c16

D27647: throw out the windows variant of the smb slave

vonreth accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH smb-no-win REVISION DETAIL https://phabricator.kde.org/D27647 To: sitter, ngraham, #windows, vonreth Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast0

D27540: KCModule: Indicate when a setting has been changed from the default or previous value

ervin added a comment. In D27540#617589 , @alexde wrote: > Does this patch also fix indirectly #274629? I doubt it, this bug report seems confused BTW... it was never broken on the kdelibs side but *some* dialogs have bugs which break the

D27444: Added top area

niccolove updated this revision to Diff 76376. niccolove added a comment. Added toparea margins REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27444?vs=75824&id=76376 BRANCH top_area (branched from master) REVISION DETAIL https://ph

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

trmdi added a comment. Hmm, I think you need to rebase to remove unrelated files in this patch e.g. `dropjob.cpp` ... ? REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27122 To: niccolove, ndavis, ngraham, cblack Cc: trmdi, cblack, ndavis, filipf,

D27654: [kio] Fix running konsole on Wayland

wbauer created this revision. wbauer added a reviewer: dfaure. wbauer added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. wbauer requested review of this revision. REVISION SUMMARY %i expands to "--icon someicon", but Qt recognizes this option only on X11/xcb. So kon

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

niccolove added a comment. In D27122#617614 , @trmdi wrote: > Hmm, I think you need to rebase to remove unrelated files in this patch e.g. `dropjob.cpp` ... ? Definitively. *sigh* REPOSITORY R242 Plasma Framework (Library) REVISION DE

D27444: Added top area

niccolove updated this revision to Diff 76378. niccolove added a comment. Rebase REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27444?vs=76376&id=76378 BRANCH top_area (branched from master) REVISION DETAIL https://phabricator.kde.o

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

niccolove updated this revision to Diff 76381. niccolove added a comment. Rebase REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27122?vs=76373&id=76381 BRANCH contrast_effect_background_color (branched from master) REVISION DETAIL h

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

niccolove updated this revision to Diff 76382. niccolove added a comment. re-tweaked commits REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27122?vs=76381&id=76382 BRANCH contrast_effect_background_color (branched from master) REVISIO

D27654: [kio] Fix running konsole on Wayland

wbauer edited the test plan for this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D27654 To: wbauer, dfaure Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27502: Create ConfigView an unmanaged ConfigWidget

bport updated this revision to Diff 76384. bport added a comment. Rework, not expose anymore settings and loader but expose a new settings class REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27502?vs=75997&id=76384 REVISION DETAIL https://phabricator.kde

D27644: do not hide enum warnings via switch default label

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH nodefault REVISION DETAIL https://phabricator.kde.org/D27644 To: sitter, ngraham Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe,

D27646: smb: apply a whole bunch of static analyzer improvements

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras BRANCH tidy REVISION DETAIL https://phabricator.kde.org/D27646 To: sitter, ngraham Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe, fbam

D27647: throw out the windows variant of the smb slave

ngraham accepted this revision. REPOSITORY R320 KIO Extras BRANCH smb-no-win REVISION DETAIL https://phabricator.kde.org/D27647 To: sitter, ngraham, #windows, vonreth Cc: kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, cblack, MrPepe, fbampaloukas, alexde, GB_2,

D27133: kconfig_compiler : generate kconfig settings with subgroup

dfaure added a comment. OK. `man ascii` says 0x1d is "GS (group separator)", not completely bogus :) I wonder how/why it becomes ␝ in XML, but quick googling says that entity seems to be '␝' which indeed reads GS :) At the KConfig level, this is good for catching people who don't use

D27654: [kio] Fix running konsole on Wayland

wbauer updated this revision to Diff 76387. wbauer added a comment. Use `KShell::quoteArg()` in case the icon name contains spaces or other special chars. No idea if that's possible/allowed, but better be safe than sorry I suppose. REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://

D27502: Create ConfigView an unmanaged ConfigWidget

ervin requested changes to this revision. ervin added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > settings.cpp:62 > > -bool Settings::setCheckUppercase(bool check) > +void Settings::setCheckUppercase(bool check) > { Please rename it to "skip uppercase"...

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

ngraham added a comment. In D27122#617594 , @niccolove wrote: > F8131397: image.png Now that //is// relaxing :) I like it. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://pha

D27097: Port from QRegExp to QRegularExpression

ahmadsamir added a comment. Looking at the incremental stuff some more, there are no public methods that mention incremental; the only way is to set the KFind::FindIncremental flag; the only method I could find is in the private API (find_p.h) startNewIncrementalSearch(). So technically it's

D27660: Add Inno Setup syntax, including embedded Pascal scripting

zrax created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. zrax requested review of this revision. REPOSITORY R216 Syntax Highlighting BRANCH innosetup (branched from master) REVISION DETAIL https://phabricator.kde.org

D27637: kill twitter support

apol accepted this revision. REPOSITORY R495 Purpose Library BRANCH killtwitter REVISION DETAIL https://phabricator.kde.org/D27637 To: nicolasfella, #frameworks, apol, bshah Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27637: kill twitter support

This revision was automatically updated to reflect the committed changes. Closed by commit R495:13f9f45f2cce: kill twitter support (authored by nicolasfella). REPOSITORY R495 Purpose Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27637?vs=76334&id=76399 REVISION DETAIL htt

D27586: templates: Convert license statements to SPDX headers

cordlandwehr accepted this revision. cordlandwehr added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > kossebau wrote in main.cpp:31 > Hmpf, that slipped me. The regular expression overdoing its job :) > I guess that will be hit in more KAboutData instances. P

D27603: KDNSSD: Convert license headers to SPDX statements

This revision was automatically updated to reflect the committed changes. Closed by commit R272:563fc1416d90: KDNSSD: Convert license headers to SPDX statements (authored by cordlandwehr). REPOSITORY R272 KDNSSD CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27603?vs=76226&id=76400

D27602: KDbusAddons: Convert license statement to SPDX

This revision was automatically updated to reflect the committed changes. Closed by commit R271:a222808f86cd: KDbusAddons: Convert license statement to SPDX (authored by cordlandwehr). REPOSITORY R271 KDBusAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27602?vs=76225&id=76401

D27616: [UrlHandler] optimise the code and less if nesting

apol added inline comments. INLINE COMMENTS > ahmadsamir wrote in urlhandler.cpp:55 > I could be wrong, but this is exactly equivalent to the original code, i.e. > it won't reach this unless helpcenter is empty and it's not a kde app... and > I actually wanted to ask about this bit from the ori

D27601: KConfig: Convert to SPDX license statements

cordlandwehr marked 3 inline comments as done. cordlandwehr added inline comments. INLINE COMMENTS > cgiboudeaux wrote in kcfg.xsd:20 > What makes you think it's BSD? Actually, I though quite a time about this and at the end did a comparison of the permissions specifically provided by this stat

D27601: KConfig: Convert to SPDX license statements

cordlandwehr updated this revision to Diff 76403. cordlandwehr added a comment. Better handling of explenatory texts REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27601?vs=76224&id=76403 BRANCH spdx REVISION DETAIL https://phabricator.kde.org/D27601

D27601: KConfig: Convert to SPDX license statements

ognarb added inline comments. INLINE COMMENTS > cordlandwehr wrote in kcfg.xsd:20 > Actually, I though quite a time about this and at the end did a comparison of > the permissions specifically provided by this statement: use, copy and modify > (yet there is no explicit statement of sublicensing

D27463: KconfigXT: Add a value attribute to Enum field choices

meven updated this revision to Diff 76405. meven marked 6 inline comments as done. meven added a comment. Move KCoreConfigSkeleton::ItemEnum::Choice::value to cpp file, use a regex to filter valid choice name REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D

D27463: KconfigXT: Add a value attribute to Enum field choices

meven added inline comments. INLINE COMMENTS > kcoreconfigskeleton.h:764 > QString whatsThis; > +QString val; > + Something I have noticed while testing this. Since it changes the memory of a very common data struct in a very common lib, it creates a lot of crashes if a

D27038: [KMessageWidget] Allow the style to change our palette

davidre updated this revision to Diff 76408. davidre added a comment. Rebase REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27038?vs=74701&id=76408 BRANCH polish REVISION DETAIL https://phabricator.kde.org/D27038 AFFECTED FILES src/kmessagew

D27038: [KMessageWidget] Allow the style to change our palette

This revision was automatically updated to reflect the committed changes. Closed by commit R236:d946fb71ea22: [KMessageWidget] Allow the style to change our palette (authored by davidre). REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27038?vs=76408&id=

D27039: [KStyle] Set the color of KMessageWidgets to the correct one from the current color scheme

This revision was automatically updated to reflect the committed changes. Closed by commit R252:d3a1b5148008: [KStyle] Set the color of KMessageWidgets to the correct one from the current… (authored by davidre). REPOSITORY R252 Framework Integration CHANGES SINCE LAST UPDATE https://phabrica

D27296: [PlasmaComponents]Clean up ListItem code

This revision was automatically updated to reflect the committed changes. Closed by commit R242:10c112fccda8: [PlasmaComponents]Clean up ListItem code (authored by gvgeo). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27296?vs=75391&id=7641

KDE CI: Frameworks » kwidgetsaddons » kf5-qt5 FreeBSDQt5.13 - Build # 72 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kwidgetsaddons/job/kf5-qt5%20FreeBSDQt5.13/72/ Project: kf5-qt5 FreeBSDQt5.13 Date of build: Tue, 25 Feb 2020 18:59:03 + Build duration: 4 min 0 sec and counting JUnit Tests Name:

D27635: Restore kio-webdav plugin

nicolasfella updated this revision to Diff 76413. nicolasfella added a comment. - Adapt path REPOSITORY R155 KAccounts Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27635?vs=76328&id=76413 BRANCH restore REVISION DETAIL https://phabricator.kde.org/D27635 AFFECT

D27635: Restore kio-webdav plugin

nicolasfella updated this revision to Diff 76416. nicolasfella added a comment. - Simplify REPOSITORY R155 KAccounts Integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27635?vs=76413&id=76416 BRANCH restore REVISION DETAIL https://phabricator.kde.org/D27635 AFFECTED

D21795: [KAuth] Add support for action details in Polkit1 backend.

feverfew updated this revision to Diff 76419. feverfew added a comment. - Fix the diff REPOSITORY R283 KAuth CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21795?vs=76073&id=76419 BRANCH arcpatch-D21795_1 REVISION DETAIL https://phabricator.kde.org/D21795 AFFECTED FILES a

D27097: Port from QRegExp to QRegularExpression

dfaure added a comment. Deprecate the attribute, keep the code. There is more KF5-based code out there than the one visible by lxr.kde.org. Some of it not even public. We make the same promise as Qt, preserving SC and BC in major versions. OK for "Invalid PCRE pattern syntax" REPOSITORY

D27654: [kio] Fix running konsole on Wayland

dfaure added a comment. Since this is konsole specific, it's fine to use a Qt-specific argument. But IMHO you could also just remove the %i altogether. I never understood the idea of "take the icon from the desktop file, not the one the application would use by default". I mean, with som

D27662: Rewrite nextcloud plugin

nicolasfella created this revision. nicolasfella added reviewers: Frameworks, apol, bshah. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. nicolasfella requested review of this revision. REVISION SUMMARY Let CopyJob and the webdav KIO slave do the hard work.

D27662: Rewrite nextcloud plugin

nicolasfella updated this revision to Diff 76421. nicolasfella added a comment. - Remove debug REPOSITORY R495 Purpose Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27662?vs=76420&id=76421 BRANCH nextcloud REVISION DETAIL https://phabricator.kde.org/D27662 AFFECTED

D23339: Fix memory leak in ConfigView and Dialog

nicolasfella added a comment. This makes plasmashell crash for me as soon as a notification appears. https://invent.kde.org/snippets/730 REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D23339 To: dvratil, #plasma, #frameworks, apol, davidedmund

D27635: Restore kio-webdav plugin

nicolasfella edited the summary of this revision. REPOSITORY R155 KAccounts Integration REVISION DETAIL https://phabricator.kde.org/D27635 To: nicolasfella, bshah, leinir, #frameworks

D27665: Support SAE in securityTypeFromConnectionSetting

hexchain created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. Herald added a reviewer: jgrulich. hexchain requested review of this revision. REVISION SUMMARY Allow SAE to be returned from securityTypeFromConnectionSetting. TEST PLAN Not v

D27665: Support SAE in securityTypeFromConnectionSetting

hexchain added a dependent revision: D27666: Support WPA3-SAE in the applet popup. REPOSITORY R282 NetworkManagerQt REVISION DETAIL https://phabricator.kde.org/D27665 To: hexchain, jgrulich Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27363: KHolidays: Convert license statements to SPDX expressions

winterz accepted this revision. This revision is now accepted and ready to land. REPOSITORY R175 KHolidays BRANCH spdx REVISION DETAIL https://phabricator.kde.org/D27363 To: cordlandwehr, winterz Cc: cgiboudeaux, winterz, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham,

D27662: Rewrite nextcloud plugin

apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R495 Purpose Library BRANCH nextcloud REVISION DETAIL https://phabricator.kde.org/D27662 To: nicolasfella, #frameworks, apol, bshah Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham

D27557: Auto-generate 24px monochrome icons

usta added a comment. Freebsd side broken again : 00:31:45 ./build/icons-dark/generated/devices/24/network-wired-activated.svg:14: parser error : Opening and ending tag mismatch: g line 8 and svg 00:31:45 00:31:45^ 00:31:45 ./build/icons-dark/generated/devices/24/network-

D27586: templates: Convert license statements to SPDX headers

This revision was automatically updated to reflect the committed changes. Closed by commit R306:c10ca8623903: templates: Convert license statements to SPDX headers (authored by kossebau). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D27586?vs=76236&id=76437#toc REPOSITORY R306 KParts

D27665: Support SAE in securityTypeFromConnectionSetting

jgrulich accepted this revision. This revision is now accepted and ready to land. REPOSITORY R282 NetworkManagerQt BRANCH securityTypeFromConnectionSetting-support-SAE (branched from master) REVISION DETAIL https://phabricator.kde.org/D27665 To: hexchain, jgrulich Cc: kde-frameworks-devel