Re: VDG application design sprint?

for me would work either first half of june or july On Thu, Apr 11, 2024 at 4:26 PM Nicolas Fella wrote: > > On 1/24/23 00:43, Nicolas Fella wrote: > > Hi, > > > > I think it would make sense for us to have a VDG sprint of sorts in the > > near-ish future. This would allow to discuss some larger

Re: Proposal unify back our release schedules

significant hindrance for distributions to the point "we don't feel we can adequately support it"? a part of this discussion should have the point of a good relatioship with downstream in consideration (also because is part of the broader "getting to more users") -- Marco Martin

Re: KSvg in kdereview

this time. > > Would be appreciated, given this is looking to be promoted to Frameworks, if > people could please have a further look at this repository and comment as > appropriate. > > Thanks, > Ben > > On Wed, Jun 14, 2023 at 9:12 PM Marco Martin wrote: >> >&

Re: KSvg in kdereview

the rest depending from it? On Thu, Apr 20, 2023 at 10:25 AM Marco Martin wrote: > > Hi all, > A part of plasma-framewrok, which is the one to do SVG-based themes, > has now been splitted in a standalone library which is intended to be > a new framework in KF6 (all usages of the p

Re: KSvg in kdereview

ry is really about svgs and nothing else, and the Svg class is about rendering one single svg, so I don't see many ways around about both contianing "svg" ? (I was aware there was an old kde3 ksvg library, though talking about it in some tuesday meeting didn't seem to be an issue) Open to ideas :) -- Marco Martin

Re: KSvg in kdereview

still some stray documentation which doesn't completely reflect truth anymore (right now it doesn't use that config file anymore, it's up tp the user's code) will correct -- Marco Martin

KSvg in kdereview

) The repo for now lives in https://invent.kde.org/libraries/plasmasvg In the end it will be renamed in ksvg Comments? reviews? -- Marco Martin

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: Request for help for developing new XDG StatusNotifier/AppIndicator spec for KDE+GNOME

eam in Qt -- 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

Re: Plasma Bigscreen in Kdereview

/CMakeLists.txt:25 (kpackage_install_package) yes, is exactly like kcms in plasma-desktop, where all the package names are all library name, so kcm_foo i would prefer to keep that consistent with those? (then, is definitely possible to do kcm_org.kde.foo ir required, for both package and library) -- Marco Martin

Re: Plasma Bigscreen in Kdereview

-settings-shell have i18n calls but as far as I can see > not included in any Message.sh those are fixed > files in ./shell/contents/configuration have i18n calls but as far as I can > see not included in any Message.sh those are "special" and can't really be translatable, but none of those qml files have strings anymore -- Marco Martin

Re: Plasma Bigscreen in Kdereview

On Fri, Jan 22, 2021 at 12:38 PM Yuri Chornoivan wrote: > > пʼятниця, 22 січня 2021 р. 12:48:07 EET Marco Martin написано: > > Hi all, > > We would like a release of the Plasma Bigscreen project > > which has been moved to kdereview, its repo is: > > https://invent.k

Plasma Bigscreen in Kdereview

converted license headers to SPDX, the code should be in fairly good shape by now -- 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

  1   2   3   4   5   6   7   8   9   10   >