Review Request 114745: [Kig] File Loading Based on MIME Types

Commit Hook null at kde.org
Tue Dec 31 13:46:35 UTC 2013


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


This review has been submitted with commit 985cbc9c1799b915b47692b17a110346754aea73 by David E. Narvaez to branch master.

- Commit Hook


On Dec. 31, 2013, 1:45 p.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114745/
> -----------------------------------------------------------
> 
> (Updated Dec. 31, 2013, 1:45 p.m.)
> 
> 
> Review request for KDE Edu and David Faure.
> 
> 
> Repository: kig
> 
> 
> Description
> -------
> 
> These are changes I worked months ago when I found and fixed bug 322578. Back then I had a brief e-mail exchange with dfaure about this change and he thought it is better to load files based on their MIME types instead of the file extensions. Besides the fact that it loads the files  based on their MIME types, these changes also modernize the loading of the Kig part by using KService, and querying the KService for the supported MIME types to load. There's a noticeable change in the UI, see the attached picture.
> 
> 
> Diffs
> -----
> 
>   filters/filter.h 970da7b 
>   filters/filter.cc 8d5a676 
>   kig/kig.h fe56bfd 
>   kig/kig.cpp e426deb 
>   kig/kig_part.desktop 9c7139e 
> 
> Diff: https://git.reviewboard.kde.org/r/114745/diff/
> 
> 
> Testing
> -------
> 
> Loaded all files in the example folder of the kig repository, all loading OK. It lists exactly 8 files, same as my 4.12 installation of Kig.
> 
> 
> File Attachments
> ----------------
> 
> No file extensions shown in the Open File dialog
>   https://git.reviewboard.kde.org/media/uploaded/files/2013/12/31/87ba4f49-ab1e-411b-9491-422484d8f2a5__kignewfilters.png
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20131231/e33d5fc6/attachment-0001.html>


More information about the kde-edu mailing list