Review Request 113630: Port the original KDE4 ksplash theme to QML

[Martin Klapetek]

> On Nov. 5, 2013, 1:07 p.m., Sebastian Kügler wrote:
> > ksplash/ksplashqml/themes/Classic/main.qml, line 132
> > <http://git.reviewboard.kde.org/r/113630/diff/2/?file=209134#file209134line132>
> >
> >     This looks odd?

Yes, the last icon (the image itself) is actually bigger than the rest :S


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113630/#review43052
-----------------------------------------------------------


On Nov. 5, 2013, 10:42 a.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113630/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2013, 10:42 a.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> Last thing on my ksplash list is to port the good old KDE4 theme as "Classic" theme done in QML. I'm a QML noob so I'm putting it up for some comments. It works (and looks) just like the original theme. I reconstructed it using the original images.
> 
> I'll put it into kde-workspace/ksplash, where it can be further improved, then it can be moved to some Look&Feel package later on.
> 
> 
> Diffs
> -----
> 
>   ksplash/ksplashqml/themes/CMakeLists.txt c5ab367 
>   ksplash/ksplashqml/themes/Classic/CMakeLists.txt PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/Preview.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/Theme.rc PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/background.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/icon1.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/icon2.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/icon3.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/icon4.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/icon5.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/images/rectangle.png PRE-CREATION 
>   ksplash/ksplashqml/themes/Classic/main.qml PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/113630/diff/
> 
> 
> Testing
> -------
> 
> Works just like in KDE4.
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/11/04/b7596f8c-2df3-4e62-bcbd-ee0e5ad5afc4__ksplash_classic.png
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131105/7b789f57/attachment.htm>