> On Enero 11, 2016, 6:35 p.m., Marco Martin wrote:
> > src/plasma/svg.cpp, line 164
> > <https://git.reviewboard.kde.org/r/126720/diff/1/?file=430382#file430382line164>
> >
> > coding style:
> > void SvgPrivate::updateCustomStyleCrc()
> > {
I don't totally get why QVariantMap is that bad:
The idea with this first patch is to have initially set the whole customColors
palette with the bare minimun necessary code (and memory), up to the little I
know, QVariantMap does this perfectly by don't consuming memmory when nothing
is in it and only saving strings when it has **any number** of rules. Maybe
there is some functionality Qt provides that in my research went out of my
sight, but can't think it woould be much better than 0-N rules with just saving
strings
In addition, when reading QML code:
```qml
PlasmaCore.SvgItem{
id: svgObject
svg: PlasmaCore.Svg{
customColorScheme: {
"#rule1":"color",
".rule2":"color"
}
}
}
```
seems pretty explanatory and when you gotta programatically change or add the
colors you have the **svgObject.svg.customColorRule("#colorX","color")**
function
>From the perspective of a QML programmer this is bearable enough and
>simplifies a lot the programming of rich UI plasmoids with little
>imperceptible processing or memory consumption, doesn't it?
*Please note that I'm totally new to this world and there are pretty much
obvious things that I don't know right now, I'm only trying to understand*
- Nico
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126720/#review90926
-----------------------------------------------------------
On Enero 11, 2016, 9:37 p.m., Nico Rodsevich wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126720/
> -----------------------------------------------------------
>
> (Updated Enero 11, 2016, 9:37 p.m.)
>
>
> Review request for Plasma, David Edmundson and Marco Martin.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> Enables to set rules that extends the #current-color-scheme CSS style of the
> Svg in order to change the Svg's items with currentColor fill to the
> specified in the customColors rules
>
>
> Diffs
> -----
>
> src/plasma/private/svg_p.h 1d1000d
> src/plasma/svg.h 3a4ca1d
> src/plasma/svg.cpp 28abd00
>
> Diff: https://git.reviewboard.kde.org/r/126720/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Nico Rodsevich
>
>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
