Review Request: load the images for the themes correctly
Giorgos Tsiapaliokas
terietor at gmail.com
Fri Dec 7 17:56:49 UTC 2012
> On Nov. 29, 2012, 1:53 p.m., Aaron J. Seigo wrote:
> > plasmate/packagemodel.cpp, lines 196-205
> > <http://git.reviewboard.kde.org/r/106680/diff/5/?file=93807#file93807line196>
> >
> > why not add "images", "config" and "animations" to m_themeImageDialogOptions and rename it to m_dialogOptions
I have put "images" and "config" into the hash, but how can it be done for "animations" too?
> else if(!qstrcmp(key, "animations") && packageType() != "Plasma/Theme")
we check the key and the packageType, so we can't use the hash here. No?
> On Nov. 29, 2012, 1:53 p.m., Aaron J. Seigo wrote:
> > plasmate/packagemodel.cpp, line 544
> > <http://git.reviewboard.kde.org/r/106680/diff/5/?file=93807#file93807line544>
> >
> > "Plasma/Theme" should not appear in this file.
the "colors" file isn't a requiredDirectory nor a requiredFile, so If I don't add it by hand in the m_topEntries
it won't be visible and also we want the "colors" file in the m_topEntries only when the package is a theme.
No? What am I missing?
- Giorgos
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106680/#review22754
-----------------------------------------------------------
On Nov. 5, 2012, 5:51 p.m., Giorgos Tsiapaliokas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106680/
> -----------------------------------------------------------
>
> (Updated Nov. 5, 2012, 5:51 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> create a new theme package->click on the new
>
> a file dialog should appear but instead a simple edit box appears requesting a new filename.
>
> This patch solves the issue
>
>
> Diffs
> -----
>
> plasmate/editors/editpage.h 5cb3ea6
> plasmate/editors/editpage.cpp 7e82ff2
> plasmate/packagemodel.h efa3001
> plasmate/packagemodel.cpp 9eb0914
>
> Diff: http://git.reviewboard.kde.org/r/106680/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Giorgos Tsiapaliokas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121207/79650a22/attachment.html>
More information about the Plasma-devel
mailing list