[Konversation-devel] Review Request: Enable advanced bookmarks functionality

Luigi Toscano luigi.toscano at tiscali.it
Mon Sep 14 23:00:58 CEST 2009


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

Review request for konversation.


Summary
-------

- Enable advanced bookmarks functionality, by overriding KBookmarkOwner's methods in KonviBookmarkHandler. Support for "Bookmark Tabs as Folder" and "Open Folder in Tabs" actions.
- More verbose description for new bookmarks: "channel (network)".
- Move title and URI generators from ViewContainer to more generic methods of ChatWindow.
- Remove currentURL and currentTitle from mainWindow; they are a left-over from the days when there was no separate view container.

- Make ConnectionManager::decodeIrcUrl public.
- Add new code to ConnectionSettings to support a set of channels (m_oneShotChannelList) that should be used only one for the current ConnectionSettings object (used by KonviBookmarkHandler::openFolderInTabs).
- Move the code that generates a JOIN command from a ChannelList to a separate function.


Diffs
-----

  /trunk/extragear/network/konversation/src/bookmarkhandler.h 1023510 
  /trunk/extragear/network/konversation/src/bookmarkhandler.cpp 1023510 
  /trunk/extragear/network/konversation/src/connectionmanager.h 1023510 
  /trunk/extragear/network/konversation/src/connectionmanager.cpp 1023510 
  /trunk/extragear/network/konversation/src/connectionsettings.h 1023510 
  /trunk/extragear/network/konversation/src/connectionsettings.cpp 1023510 
  /trunk/extragear/network/konversation/src/irc/server.h 1023510 
  /trunk/extragear/network/konversation/src/irc/server.cpp 1023510 
  /trunk/extragear/network/konversation/src/mainwindow.h 1023510 
  /trunk/extragear/network/konversation/src/mainwindow.cpp 1023510 
  /trunk/extragear/network/konversation/src/viewer/chatwindow.h 1023510 
  /trunk/extragear/network/konversation/src/viewer/chatwindow.cpp 1023510 
  /trunk/extragear/network/konversation/src/viewer/viewcontainer.h 1023510 
  /trunk/extragear/network/konversation/src/viewer/viewcontainer.cpp 1023510 

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


Testing
-------


Thanks,

Luigi



More information about the Konversation-devel mailing list