[Kde-pim] Review Request 125577: Extend the purpose of GrantleeTheme library and refresh the KMail/Kontact "Introduction" pages theme

Sat Oct 10 12:30:34 BST 2015

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125577/#review86620
-----------------------------------------------------------


grantleetheme/src/grantleethememanager.cpp (line 202)
<https://git.reviewboard.kde.org/r/125577/#comment59625>

    Where do you migrate it ?


- Laurent Montel


On oct. 9, 2015, 10:54 après-midi, Daniel Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125577/
> -----------------------------------------------------------
> 
> (Updated oct. 9, 2015, 10:54 après-midi)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> So, this tiny patch does two things (it can be split when committed, but for discussion I think it's better to have it as one):
>   1) it repurposes GrantleeTheme to be able to not just manage Grantlee theme packs, but also to render them. I basically forked some of the logic from the Grantlee template management code I wrote for KCalUtils (once GrantleeTheme is split out of kdepim, I'll make KCalUtils use it). This is useful because the way we use Grantlee is very repetitive (Engine, Loader -> load plugins -> render), so wrapping that into GrantleeTheme::Theme looks like a good idea, it hides the ugliness of the theme/template path lookup.
>   
>   2) it creates new Grantlee templates for the "Introduction" pages for KMail and Kontact (Help -> Introduction), as well as the "Retrieving folder contents ..." page and similar info views shown in KMail. We do it using the extended GrantleeTheme functionality. See the attached screenshots - if this ain't superfancy, I don't know what is...:)
>   
> PS: Akregator has the same infopage too, but it's not using MessageViewer for obvious reasons, so the way there would be to copy the create Akregator's own Grantlee theme and make Akregator use GrantleeTheme as well. Let's do it once the underlaying stuff is in.
> 
> 
> Diffs
> -----
> 
>   grantleetheme/CMakeLists.txt 8e071a4 
>   grantleetheme/src/CMakeLists.txt c936c84 
>   grantleetheme/src/grantleeki18nlocalizer.cpp PRE-CREATION 
>   grantleetheme/src/grantleeki18nlocalizer_p.h PRE-CREATION 
>   grantleetheme/src/grantleetheme.h 995ee58 
>   grantleetheme/src/grantleetheme.cpp ae0cafd 
>   grantleetheme/src/grantleetheme_p.h PRE-CREATION 
>   grantleetheme/src/grantleethememanager.h d08bcb7 
>   grantleetheme/src/grantleethememanager.cpp 4e887a6 
>   grantleetheme/src/plugin/CMakeLists.txt PRE-CREATION 
>   grantleetheme/src/plugin/icon.h PRE-CREATION 
>   grantleetheme/src/plugin/icon.cpp PRE-CREATION 
>   grantleetheme/src/plugin/pimgrantleeplugin.h PRE-CREATION 
>   grantleetheme/src/plugin/pimgrantleeplugin.cpp PRE-CREATION 
>   kaddressbook/mainwidget.h 9c9946f 
>   kaddressbook/mainwidget.cpp 734a3c3 
>   kaddressbookgrantlee/src/widget/grantleecontactviewer.cpp 56c2a9f 
>   kmail/about/CMakeLists.txt c10672a 
>   kmail/about/introduction_kmail.html PRE-CREATION 
>   kmail/about/kmail.css 4cddf98 
>   kmail/about/main.html 6d86a45 
>   kmail/about/top-left-kmail.png f705363 
>   kmail/kmreaderwin.h f1e0f40 
>   kmail/kmreaderwin.cpp b824140 
>   kontact/src/CMakeLists.txt ca52422 
>   kontact/src/about/CMakeLists.txt 123fb3c 
>   kontact/src/about/introduction_kontact.html PRE-CREATION 
>   kontact/src/about/kontact.css 59de61d 
>   kontact/src/about/loading_kontact.html PRE-CREATION 
>   kontact/src/about/main.html 60845da 
>   kontact/src/about/top-left-kontact.png 94d7248 
>   kontact/src/mainwindow.h 7387cbc 
>   kontact/src/mainwindow.cpp 8725b5e 
>   messageviewer/CMakeLists.txt 5d56eb8 
>   messageviewer/src/CMakeLists.txt fd65489 
>   messageviewer/src/about/CMakeLists.txt PRE-CREATION 
>   messageviewer/src/about/default/CMakeLists.txt PRE-CREATION 
>   messageviewer/src/about/default/background.png PRE-CREATION 
>   messageviewer/src/about/default/main.css PRE-CREATION 
>   messageviewer/src/about/default/main.html PRE-CREATION 
>   messageviewer/src/about/default/splash.theme PRE-CREATION 
>   messageviewer/src/about/default/status.html PRE-CREATION 
>   messageviewer/src/header/grantleeheaderstyleplugin/grantleeheaderstyleinterface.h 1fdf939 
>   messageviewer/src/header/grantleeheaderstyleplugin/grantleeheaderstyleinterface.cpp 50325c4 
>   messageviewer/src/viewer/mailwebview_webkit.cpp c452660 
>   messageviewer/src/viewer/viewer.h ef98fe9 
>   messageviewer/src/viewer/viewer.cpp 1f1b12f 
>   messageviewer/src/viewer/viewer_p.h 03e2b77 
>   messageviewer/src/viewer/viewer_p.cpp 023ce11 
> 
> Diff: https://git.reviewboard.kde.org/r/125577/diff/
> 
> 
> Testing
> -------
> 
> Yes, works. The only problem seems to be that sometimes QtWebKit is ridiculously slow rendering the background wallpaper, so you can see it slowly render, as if you were downloading it via 56k modem - but I guess there's not much we can do about it.
> 
> 
> File Attachments
> ----------------
> 
> KMail Introduction
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/09/6a6b83c8-aca5-4269-9e66-f79e1e1c10e7__kmail.png
> KMail "busy" page
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/09/aef41331-c1a0-4778-b342-45f9e868d1bc__kmail3.png
> Kontact Introduction
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/09/bb6c4cbb-97d3-42b1-b677-d2c8c10bd111__kontact.png
> 
> 
> Thanks,
> 
> Daniel Vrátil
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/