Copyright 2009 Riccardo Iaconelli <riccardo@kde.org>

this should have your name and email address

    void setDirectory(const KUrl& image);

    QString directory();

this should probably be setImage and QString image() const, since it doesn't actually set a directory to view but a specific image.

    void commonImages();

    void svgImages();

these could be private, or even merged into setImage

Copyright 2009-2010 Aaron Seigo <aseigo@kde.org>

Copyright 2009-2010 Artur Duque de Souza <asouza@kde.org>

Copyright 2009-2010 Laurent Montel <montel@kde.org>

Copyright 2009-2010 Shantanu Tushar Jha <jhahoneyk@gmail.com>

Copyright 2009-2010 Sandro Andrade <sandroandrade@kde.org>

Copyright 2009-2010 Lim Yuen Hoe <yuenhoe@hotmail.com>

this should have your name and email address instead

there is no layout (e.g. a QHBoxLayout) that the QLabel/QSvgWidget can be put into. this is why the image is showing up small and in the top corner.

    m_directory = image.path();

after setting the path, the code that is currently in init() should be called. i'd recommend getting rid fo the init() method, in fact, and merging it with this one.

    m_label = new QLabel(this);

    m_label->setPixmap(image);

this is a memory leak if commonImages() is called more than once. it should be something like:

if (!m_label) {
    m_label = new QLabel(this);
}

m_label->setPixmap(image);

this also implies initializing the m_label pointer to 0 in the constructor

    QSvgWidget image;

this should be new'd and added to the layout, otherwise the svg won't be visible, correct?

    image.renderer();

seems unnecessary 

void MainWindow::loadImageViewer(const KUrl& target)

    if(m_imageViewer) {

        delete m_imageViewer;

    }

    m_imageViewer = new ImageViewer(this);

    m_imageViewer->setDirectory(target);

    m_imageViewer->init();

would it be better to do this as something like:

if (!m_imageViewer) {
    m_imageViewer = new ImageViewer(this);
    m_imageViewer->init();
}

m_imageViewer->setDirectory(target);
... etc