Review Request: SpeedDial for Konqueror

[Thomas Fischer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2767/
-----------------------------------------------------------

(Updated 2010-02-05 07:17:45.226201)


Review request for kdelibs.


Changes
-------

Applying suggestions from David


Summary
-------

This patch adds a "speed dial" part to Konqueror. Within Konqueror it can be chosen as an alternative to a blank or an about start page. The speeddial part is accessible via the speeddial: url. This mimics the functionality of Opera' SpeedDial or the ShowCase add-on for Firefox.
The correspondig bug report is https://bugs.kde.org/show_bug.cgi?id=149596

The code has still some minor issues such as UI blocking and missing access to Konqueror's history.
Usability experts are invited to make comments on look&feel... ;-)

An older version of this code (without proper integration in Konqueror) has been presented here: http://www.unix-ag.uni-kl.de/~fischer/speeddial/index.html


Diffs (updated)
-----

  /trunk/KDE/kdebase/apps/konqueror/CMakeLists.txt 1082172 
  /trunk/KDE/kdebase/apps/konqueror/settings/konqhtml/generalopts.cpp 1082172 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/CMakeLists.txt PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/konq_speeddial.desktop PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/konqspeeddial.desktop PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/part.h PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/part.cpp PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/speeddial.xml PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/speeddialview.h PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/speeddialview.cpp PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/speeddialwidget.h PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/speeddial/speeddialwidget.cpp PRE-CREATION 
  /trunk/KDE/kdebase/apps/konqueror/src/konqfactory.cpp 1082172 
  /trunk/KDE/kdebase/apps/konqueror/src/konqmainwindow.cpp 1082172 
  /trunk/KDE/kdebase/apps/konqueror/src/konqmisc.cpp 1082172 
  /trunk/KDE/kdebase/apps/konqueror/src/konqviewmanager.cpp 1082172 

Diff: http://reviewboard.kde.org/r/2767/diff


Testing
-------


Screenshots
-----------

SpeedDial in Konqueror
  http://reviewboard.kde.org/r/2767/s/306/


Thanks,

Thomas