Need opinion on Gwenview design

[Aurélien Gâteau]

Hello,

In Gwenview 1.4, I added support for playing videos and showing SVG files.
To do so, Gwenview uses the corresponding KParts, while to show images, it
uses its own widget. The code roughly looked like this:

if (rasterImage) {
    initImageView();
} else {
    initKpartView();
}

I would like to keep this support for videos and SVG in Gwenview 2, but I'd
prefer to avoid having series of "if (rasterImage)" like in 1.4.

My idea is to load the image view from a KPart, just like the video and SVG
KParts. But I have two questions:

1. Will this be significantly slower than instantiating the image view
widget directly? (I want Gwenview to start as fast as possible when it's
given an image as argument)

2. Gwenview 2 will feature a sidebar to perform basic operations on images
(resize, crop...), so I need to show some image view specific actions in
it. Is there a way in the KPart system to define custom actions which I
could list from Gwenview but which would remain invisible from Konqueror or
other KPart host apps?

Aurélien