Review Request 113630: Port the original KDE4 ksplash theme to QML
Sebastian Kügler
sebas at kde.org
Tue Nov 5 13:06:25 GMT 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113630/#review43052
-----------------------------------------------------------
ksplash/ksplashqml/themes/Classic/Theme.rc
<http://git.reviewboard.kde.org/r/113630/#comment31052>
KDE SC :)
ksplash/ksplashqml/themes/Classic/main.qml
<http://git.reviewboard.kde.org/r/113630/#comment31053>
You can use a Row or ListView here, and have the layouting done automatically. Same for the other icons.
I'd suggest an ObjectModel, or indeed a Row: http://qt-project.org/doc/qt-5.1/qtqml/qml-qtqml-models2-objectmodel.html
ksplash/ksplashqml/themes/Classic/main.qml
<http://git.reviewboard.kde.org/r/113630/#comment31054>
Encapsulate the NumberAnimation into its own file. It should be easy to change.
ksplash/ksplashqml/themes/Classic/main.qml
<http://git.reviewboard.kde.org/r/113630/#comment31055>
This looks odd?
- Sebastian Kügler
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/6558abaa/attachment.htm>
More information about the kde-core-devel
mailing list