Meeting notes about Plasma::Svg

some singleton called by plasma Kirigami2 If we are going to directly call Theme/Units from C++, as SvgItem/FrameSvgItem do for the plasma counterparts -- Marco Martin

Re: kirigami2 and oxygen-icons5 in kf6

Can I suggest renaming "kirigami2" tar to be > "kirigami" again? +1 from me to rename it back to just kirigami -- Marco Martin

Splitting Plasma-frameworks

plsmoids to use that, and then remove dataengines from p-f I would like after ironing out the obvious big rough edges to make it a module in workspace (after workspace is kf6 of course) Opinions? comments? -- Marco Martin

Re: Urgent: kirigami unittest broken

is still missing (like things depending on custom shaders) they are still expected to fail -- Marco Martin

Re: KF 5.95-rc1 delayed

Marco: I need your input quickly on this one. > > https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/545 > > Isn't this also pretty relevant? > https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/535 Both are approved -- Marco Martin

Re: Moving plasma-frameworks & krunner to Plasma release set for *6? (was: Re: KF6 meeting notes 2021-04-17)

n several pieces, some of which will go into frameworks, some in frameworks, some in workspace -- Marco Martin

D29390: Respect QIcon::fallbackSearchpaths()

mart accepted this revision. mart added a comment. This revision is now accepted and ready to land. go for it :) REPOSITORY R302 KIconThemes BRANCH fallback REVISION DETAIL https://phabricator.kde.org/D29390 To: nicolasfella, #plasma, #frameworks, mart Cc: mart, kossebau, aacid,

D27223: add kirigami-gallery icon

mart added a comment. can we fix this and go ahead? @mbruchert can you still work on it? want to put it on invent? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D27223 To: mbruchert, mart, #vdg, ndavis, ngraham Cc: ngraham, ognarb, ndavis,

Re: Recent breakage in kwallet

would the recent patch https://invent.kde.org/frameworks/kwallet/-/merge_requests/1 fix anything? On Sat, May 16, 2020 at 10:40 AM David Faure wrote: > > On vendredi 15 mai 2020 12:43:37 CEST Marco Martin wrote: > > Hi all, > > Recently, the package of KWallet framework for ne

D29232: Introduce the Header color set

mart closed this revision. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D29232 To: mart, #vdg, #plasma, cblack, ndavis Cc: davidre, ndavis, cblack, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D29232: Introduce the Header color set

mart marked 9 inline comments as done. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D29232 To: mart, #vdg, #plasma, cblack Cc: davidre, ndavis, cblack, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D29232: Introduce the Header color set

mart retitled this revision from "[WIP][RFC]Introduce the Header color set" to "Introduce the Header color set". REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D29232 To: mart, #vdg, #plasma, cblack Cc: davidre, ndavis, cblack, kde-frameworks-devel, LeGast00n,

D29232: [WIP][RFC]Introduce the Header color set

mart updated this revision to Diff 83090. mart added a comment. - update to new colors REPOSITORY R265 KConfigWidgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29232?vs=81363=83090 BRANCH arcpatch-D29232 REVISION DETAIL https://phabricator.kde.org/D29232 AFFECTED

D29463: Fix Kirigami.Units.devicePixelRatio=1.3 when it should be 1.0 at 96dpi

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D29463 To: Zren, #kirigami, mart Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29711: Create kcmshell.openSystemSettings() and kcmshell.openInfoCenter() functions

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R296 KDeclarative BRANCH create-kcmshell-openSystemSettings (branched from master) REVISION DETAIL https://phabricator.kde.org/D29711 To: ngraham, #plasma, mart Cc: ltoscano, cfeck,

D29711: Create kcmshell.openSystemSettings() and kcmshell.openInfoCenter() functions

mart added a comment. hmm, should probably check before whether systemsettings5 and kinfocenter exist beforehand and fallback to kcmshell if not? REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D29711 To: ngraham, #plasma, mart Cc: ltoscano, cfeck,

Recent breakage in kwallet

that with a manual build it works instead. The only kinda suspicious commit is 850219f83e7d746 fix compilation with -Werror=undef which seems to fix.. big endian for everyone? still no idea why is broken only the distro-built package -- Marco Martin

D29527: Add MapProxySource

mart accepted this revision. mart added a comment. This revision is now accepted and ready to land. works perfectly :) REPOSITORY R1049 KQuickCharts BRANCH mapproxysource REVISION DETAIL https://phabricator.kde.org/D29527 To: ahiemstra, mart Cc: mart, kde-frameworks-devel, LeGast00n,

D29412: Add smallFont to Kirigami plasma style

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH cblack/smallfont-kirigami (branched from master) REVISION DETAIL https://phabricator.kde.org/D29412 To: cblack, #kirigami, mart Cc: kde-frameworks-devel,

D29222: Fix update auto selection

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R304 KNewStuff BRANCH fix-update-autoselection (branched from master) REVISION DETAIL https://phabricator.kde.org/D29222 To: leinir, #frameworks, #plasma, bugseforuns, ngraham, mart Cc:

D28701: Add KPackage support to KNewStuffCore

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R304 KNewStuff BRANCH add-kpackage-support (branched from master) REVISION DETAIL https://phabricator.kde.org/D28701 To: leinir, #plasma, #knewstuff, #frameworks, ngraham, mart, davidedmundson,

D29201: KCMUtils: Add option to append service file to list of arguments

mart added a comment. indeed, a bit more documentation then go for it REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D29201 To: alex, #plasma, ngraham, meven, broulik Cc: mart, apol, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28701: Add KPackage support to KNewStuffCore

mart added a comment. +1 from me. does it need to go into next release or can also get in next+1? REPOSITORY R304 KNewStuff REVISION DETAIL https://phabricator.kde.org/D28701 To: leinir, #plasma, #knewstuff, #frameworks, ngraham, mart, davidedmundson, broulik, bshah Cc: alex,

D29232: [WIP][RFC]Introduce the Header color set

mart added inline comments. INLINE COMMENTS > davidre wrote in kcolorscheme.cpp:271 > Because the new colors are the replacement for theses colors. I thought one > of those might map to this i also added a question on the task, whether final colors are decided for it, they should go up here

D29232: [WIP][RFC]Introduce the Header color set

mart requested review of this revision. REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D29232 To: mart, #vdg, #plasma, cblack Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D29232: [WIP][RFC]Introduce the Header color set

mart retitled this revision from "[WIP][RFC]Introduce the Tools color set" to "[WIP][RFC]Introduce the Header color set". mart edited the summary of this revision. mart edited the test plan for this revision. REPOSITORY R265 KConfigWidgets BRANCH phab/toolsSet REVISION DETAIL

D29232: [WIP][RFC]Introduce the Tools color set

mart updated this revision to Diff 81363. mart added a comment. call it Header REPOSITORY R265 KConfigWidgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29232?vs=81352=81363 BRANCH phab/toolsSet REVISION DETAIL https://phabricator.kde.org/D29232 AFFECTED FILES

D29232: [WIP][RFC]Introduce the Tools color set

mart retitled this revision from "Introduce the Tools color set" to "[WIP][RFC]Introduce the Tools color set". REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D29232 To: mart, #vdg, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29232: Introduce the Tools color set

mart created this revision. mart added reviewers: VDG, Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY This introduces two concepts: a new color set for Tools (or: titlebar? open question)

D28701: Add KPackage support to KNewStuffCore

mart added inline comments. INLINE COMMENTS > kpackagejob.cpp:55 > + > +class KPackageTask : public QObject, public QRunnable > +{ is it worth encapsulating it in a runnable? the installation of a package is already in a separathe thread. > kpackagejob.cpp:97 > +}); > +

D28805: Tweak kcmcontrols docs

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R296 KDeclarative BRANCH tweakdocs REVISION DETAIL https://phabricator.kde.org/D28805 To: nicolasfella, #frameworks, jucato, mart Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D28470: [PlasmaCore.IconItem] Refactor source handling for different types

mart added inline comments. INLINE COMMENTS > iconitem.cpp:40 > > +class IconItemSource : public QObject > +{ does it have to be a QObject? it doesn't have properties, signals or invokables.. qobject is an expensive class so if you don't have to use oits features is better to avoid

D28620: Add readmes clarifying state of plasma component versions

mart accepted this revision. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D28620 To: davidedmundson, #plasma, apol, romangg, mart Cc: romangg, apol, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28606: Add iconSize property to PC2 ToolButton

mart added a comment. I don't think we should put any new feature in plasmacomponents2. it will be removed in qt6 anyways and work should be more concentrated to a) port away from it altogether abd b) if the pc3 version doesn't work, put effort in fixing that REPOSITORY R242 Plasma

D28466: Added Page element

mart accepted this revision. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D28466 To: niccolove, #plasma, ngraham, mart Cc: ngraham, davidedmundson, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D28372: Added a merged look to the plasmoidheading and remove roundedborders

mart added inline comments. INLINE COMMENTS > niccolove wrote in PlasmoidHeading.qml:77 > I'd go with no SVG, to make sure that the theme creators will see that it's > broken and add a merged- look. Keep in mind that the plasmoidheading svg was > added barely one month ago, and no big Plasma

D28144: Make animation durations consistent with Kirigami values

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH improved-durations (branched from master) REVISION DETAIL https://phabricator.kde.org/D28144 To: ngraham, #plasma, davidedmundson, mart Cc:

D28033: Create ExpandableListItem

mart added a comment. In D28033#630450 , @ngraham wrote: > > shortDuration: 150ms > longDuration: 250ms > veryLongDuration: 500ms veryLongDuration doesn't sound too good, but if longduration goes 500ms probably a lot of

D28130: Introduce api for passive notifications

This revision was automatically updated to reflect the committed changes. Closed by commit R296:cebaeb4daf59: Introduce api for passive notifications (authored by mart). REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28130?vs=77983=77987 REVISION DETAIL

D28131: listen to passiveNotificationRequested

This revision was automatically updated to reflect the committed changes. Closed by commit R295:3fe22e067a73: listen to passiveNotificationRequested (authored by mart). REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28131?vs=77984=77988 REVISION DETAIL

D28131: listen to passiveNotificationRequested

mart updated this revision to Diff 77984. mart added a comment. - const QVariant REPOSITORY R295 KCMUtils CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28131?vs=77941=77984 BRANCH phab/passiveNotifications REVISION DETAIL https://phabricator.kde.org/D28131 AFFECTED FILES

D28130: Introduce api for passive notifications

mart updated this revision to Diff 77983. mart added a comment. - const QVariant & REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28130?vs=77982=77983 BRANCH phab/passivenot REVISION DETAIL https://phabricator.kde.org/D28130 AFFECTED FILES

D28130: Introduce api for passive notifications

mart updated this revision to Diff 77982. mart added a comment. - const QVariant & REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28130?vs=77940=77982 BRANCH phab/passivenot REVISION DETAIL https://phabricator.kde.org/D28130 AFFECTED FILES

D28130: Introduce api for passive notifications

mart added a comment. In D28130#630325 , @davidre wrote: > I wonder if it's for the use case wehere an InlineMessage doesn't work does it need to contain actions? doesn't really *need* it, but matches the existing api already existing in

D28033: Create ExpandableListItem

mart added inline comments. INLINE COMMENTS > davidedmundson wrote in ExpandableListItem.qml:230 > If it's multiplied by a factor it's not a "standard duration" anymore. to me this tells more that - default durations right now are waaay too short - we don't take into account that for movements

D28033: Create ExpandableListItem

mart added a comment. In D28033#630073 , @ngraham wrote: > (WRT duration) What should I do then? Hardcode something? Add a new duration in a separate patch? The only standard durations we have are all quite short; neither one feels suitable for

D28130: Introduce api for passive notifications

mart added a comment. In D28130#630325 , @davidre wrote: > I wonder if it's for the use case wehere an InlineMessage doesn't work does it need to contain actions? doesn't really *need* it, but matches the existing api already existing in

D28131: listen to passiveNotificationRequested

mart edited the test plan for this revision. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D28131 To: mart Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28131: listen to passiveNotificationRequested

mart created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY show passive notifications when the kcm asks to see D28130 REPOSITORY R295 KCMUtils

D28130: Introduce api for passive notifications

mart edited the summary of this revision. mart edited the test plan for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D28130 To: mart Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28130: Introduce api for passive notifications

mart created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY showPassiveNotification which mirrors ApplicationWindow.showPassiveNotification some kcms need to show a tiny non

D28033: Create ExpandableListItem

mart added a comment. to me it should be in plasma extracomponents (and with a warning in the docs that such an item should be used only in lists that are known to have always very few items) REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL

D28028: [KCM Controls GridDelegate] Use `ShadowedRectangle`

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D28028 To: broulik, #plasma, mart Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27695: WIP: Create a new TopArea element using widgets/toparea svg

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D27695 To: niccolove, mart Cc: davidedmundson, ahiemstra, kde-frameworks-devel, LeGast00n, cblack, GB_2,

D27695: WIP: Create a new TopArea element using widgets/toparea svg

mart added inline comments. INLINE COMMENTS > PlasmoidHeading.qml:52 > +enabledBorders: { > +var borders = new Array() > +borders.push('LeftBorder') this can be an int and instead of borders.push('LeftBorder') borders |= PlasmaCore.FrameSvg.LeftBorder REPOSITORY

D27695: WIP: Create a new TopArea element using widgets/toparea svg

mart added a comment. In D27695#621985 , @niccolove wrote: > - Is code on line 51 fine? Lines are a bit long there yeah, is fine.. unfortunately sunch superlong lines end up being very common in QML REPOSITORY R242 Plasma Framework

D27695: WIP: Create a new TopArea element using widgets/toparea svg

mart added a comment. I would call the component PlasmoidHeading, to make clear it's only for use in plasmoids INLINE COMMENTS > TopArea.qml:28 > + > +property bool flipped: false > + enum Location { Header, Footer (maybe a middle as well?) } property Location

D27628: Buttons: allow to scale up icons

This revision was automatically updated to reflect the committed changes. Closed by commit R242:0b89e7f3f089: Buttons: allow to scale up icons (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27628?vs=76310=76769 REVISION

D27589: Try to apply the colorscheme of the current theme to QIcons

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH qiconcolor (branched from master) REVISION DETAIL https://phabricator.kde.org/D27589 To: davidre, #plasma, cblack, ngraham, mart Cc: mart, wbauer, cblack,

D27628: Buttons: allow to scale up icons

mart edited the test plan for this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27628 To: mart, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27628: Buttons: allow to scale up icons

mart created this revision. mart added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY always scale icons unless a size is specified in the api. this makes icon sizing more in

D27589: Try to apply the colorscheme of the current theme to QIcons

mart requested changes to this revision. mart added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > iconitem.cpp:600 > } else if (!m_icon.isNull()) { > + >

D27444: Added top area

mart added a comment. i also wonder if the normal "toolbar2 element already existing can be used.. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27444 To: niccolove, #vdg, ngraham, ndavis Cc: mart, kde-frameworks-devel, LeGast00n, cblack, GB_2,

D27444: Added top area

mart added a comment. some considerations: - this toparea will need to "overflow" its parent for the exact distance needed to perfectly fuse with the background, and has to work with both dialogs and widgets-on-desktop (so we're kinda assuming those 2 backgrounds have the exact same

D27460: fix layout size hints for button labels

This revision was automatically updated to reflect the committed changes. Closed by commit R242:ba0b637de94a: fix layout size hints for button labels (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27460?vs=75976=75977

D27460: fix layout size hints for button labels

mart added inline comments. INLINE COMMENTS > davidedmundson wrote in Button.qml:62 > I don't understand this part about constraining the width to the parent > height (for all 3) > > If I remove it all the tests still pass. was an extra constraint to have buttons always square, but i added

D27460: fix layout size hints for button labels

mart updated this revision to Diff 75976. mart marked an inline comment as done. mart added a comment. - remove extra check for squareness REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27460?vs=75972=75976 BRANCH phab/buttonslayout

D27460: fix layout size hints for button labels

mart marked an inline comment as done. mart added inline comments. INLINE COMMENTS > davidedmundson wrote in button3.qml:90 > You didn't even run this :( > > 1. icon.name not icon.source > 2. now the comment next to it is wrong > 3. there's a binding loop > > 4. if I make a test without any

D27460: fix layout size hints for button labels

mart updated this revision to Diff 75972. mart marked an inline comment as done. mart added a comment. - fix button display property - add a test for tabbar REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27460?vs=75927=75972 BRANCH

D27460: fix layout size hints for button labels

mart updated this revision to Diff 75927. mart added a comment. - add an icon REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27460?vs=75922=75927 BRANCH phab/buttonslayout REVISION DETAIL https://phabricator.kde.org/D27460

D27460: fix layout size hints for button labels

mart marked an inline comment as done. mart added inline comments. INLINE COMMENTS > davidedmundson wrote in button3.qml:90 > this doesn't contain an icon, which is the majority of what this patch is > about added REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL

D27460: fix layout size hints for button labels

mart added a comment. latest version, plasmacontrols 2 and 3 one beside the other F8110874: Screenshot_20200218_160904.png REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27460 To: mart, #plasma,

D27460: fix layout size hints for button labels

mart updated this revision to Diff 75922. mart added a comment. - some quirks to make it more similar to pc2 REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27460?vs=75833=75922 BRANCH phab/buttonslayout REVISION DETAIL

D27460: fix layout size hints for button labels

mart edited the test plan for this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27460 To: mart, #plasma Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27460: fix layout size hints for button labels

mart created this revision. mart added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY icon sizes and label placements tested correct in several scenarios -

D26769: Always strip html if server does not support it

mart accepted this revision. REPOSITORY R289 KNotifications BRANCH stripcorrently REVISION DETAIL https://phabricator.kde.org/D26769 To: nicolasfella, #frameworks, broulik, mart Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27399: make sure the size hints are integer and even

This revision was automatically updated to reflect the committed changes. Closed by commit R242:667dce502acf: make sure the size hints are integer and even (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D27399: make sure the size hints are integer and even

mart created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY for retrocompatibility with pc2 units.gridUnit * 1.6 is used a lot as size hint (we have to guarantee buttons and

D27396: support icon.width/height

This revision was automatically updated to reflect the committed changes. Closed by commit R242:3de1c2462eda: support icon.width/height (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27396?vs=75677=75678 REVISION DETAIL

D27396: support icon.width/height

mart updated this revision to Diff 75677. mart added a comment. - don't expose externally default size REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27396?vs=75675=75677 BRANCH phab/iconSize REVISION DETAIL

D27396: support icon.width/height

mart updated this revision to Diff 75675. mart added a comment. - default icon sizes REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27396?vs=75672=75675 BRANCH phab/iconSize REVISION DETAIL https://phabricator.kde.org/D27396

D27396: support icon.width/height

mart added a comment. In D27396#611425 , @broulik wrote: > Don't we want to set a default? I keep seeing people hardcoding `icon.width` all over the place when they used PC3 and I want that to stop then should be i think smallMedium size

D27396: support icon.width/height

mart edited the test plan for this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D27396 To: mart, #plasma Cc: broulik, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27396: support icon.width/height

mart created this revision. mart added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY support the icon size hint of the control. Reinstate it in ToolButton and add the support

D27200: Make the attached property a simple QObject

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 R242:68a273aef6c1: Make the attached property a simple QObject (authored by mart). CHANGED PRIOR TO COMMIT

D27200: Make the attached proeprty a simple QObject

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

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

D26898: expose an id for the wallpaper

mart updated this revision to Diff 74320. mart added a comment. plasmoidId to avoid potential future clashes with QML id REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26898?vs=74319=74320 BRANCH phab/wallpaperid REVISION DETAIL

D26898: expose an id for the wallpaper

mart created this revision. mart added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY The id of the wallpaper, is really the id of the containment, as there will always be a

D26573: Add missing Import Env Variable

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R240 Extra CMake Modules BRANCH add_missing_env_var REVISION DETAIL https://phabricator.kde.org/D26573 To: tcanabrava, mart Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy,

D26791: prefer the new kcm plugins to the old

mart marked an inline comment as done. REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D26791 To: mart, #plasma, sitter, davidedmundson Cc: davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D26791: prefer the new kcm plugins to the old

This revision was automatically updated to reflect the committed changes. Closed by commit R295:9afc55415793: prefer the new kcm plugins to the old (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D26791?vs=73932=73936#toc REPOSITORY R295 KCMUtils CHANGES SINCE LAST

D26791: prefer the new kcm plugins to the old

mart created this revision. mart added reviewers: Plasma, sitter. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY swap the logic, if the same kcm has both the new and old plugins format (for instance

D26664: Same behavior for scrollbar as the desktop style

This revision was automatically updated to reflect the committed changes. Closed by commit R242:02e57319f791: Same behavior for scrollbar as the desktop style (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D26664: Same behavior for scrollbar as the desktop style

mart marked 3 inline comments as done. REPOSITORY R242 Plasma Framework (Library) BRANCH phab/newscrollbar REVISION DETAIL https://phabricator.kde.org/D26664 To: mart, #plasma, #vdg, ngraham Cc: davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D26664: Same behavior for scrollbar as the desktop style

mart updated this revision to Diff 73631. mart added a comment. - round things REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26664?vs=73621=73631 BRANCH phab/newscrollbar REVISION DETAIL https://phabricator.kde.org/D26664

D26664: Same behavior for scrollbar as the desktop style

mart updated this revision to Diff 73621. mart added a comment. - only show when *not* a plasmoid REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26664?vs=73545=73621 BRANCH phab/newscrollbar REVISION DETAIL

D26664: Same behavior for scrollbar as the desktop style

mart added a comment. I would put for now an exception for plasmoids to not have such line, thenin a second moment we think about a proper and more complicated solution to update plasmoids design to have something along the lines of the latter mockup REPOSITORY R242 Plasma Framework

D26664: Same behavior for scrollbar as the desktop style

mart added a comment. actually, doesn't look that good. F7884183: Screenshot_20200115_16.png this is because the scrollview has some margins and no frame, so now the scrollbar appears not centered. those plasmoids would need to not have any

D26664: Same behavior for scrollbar as the desktop style

mart added a comment. In D26664#594322 , @ngraham wrote: > I don't see the separator lines in plasma applets. Is that intentional? most of them are still qqc1, i can look into adding them there as well REPOSITORY R242 Plasma Framework

  1   2   3   4   5   6   7   8   9   10   >