Review Request 112577: Created DolphinViewSignalAdapter class to get rid of DolphinMainWindow::connectViewSignals().

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sat Sep 7 13:52:15 BST 2013


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

Review request for Dolphin.


Description
-------

Move the tab-handling away from DolpinMainWindow (Patch-Series)
    
Created DolphinViewSignalAdapter class to get rid of DolphinMainWindow::connectViewSignals().
    
DolphinMainWindow connects to all the signals provided by the adapter class (only once) and the
adapter class connects himself to the signals of the actual view container. When the actual
view container has been changed, the adapter class disconnects all signals to the previous
view container and connects all signals to the new view container.
    
One positive side effect is, that only the active view container's signals are processed
in the main window instead of all signals from all view containers.

We need this for the future tab handling code.


Diffs
-----

  dolphin/src/CMakeLists.txt 48ea14c 
  dolphin/src/dolphinmainwindow.h 7da5801 
  dolphin/src/dolphinmainwindow.cpp 4128cdf 
  dolphin/src/dolphinviewsignaladapter.h PRE-CREATION 
  dolphin/src/dolphinviewsignaladapter.cpp PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/112577/diff/


Testing
-------

Signals like request context menu, ... work.


Thanks,

Emmanuel Pescosta

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130907/1084dd22/attachment.htm>


More information about the kfm-devel mailing list