Review Request 110340: Factored out the tabbing code from DolphinMainWindow into different classes + Implemented tab-bar drag and drop (with drop indicator)

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Tue May 7 10:31:15 BST 2013


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

(Updated May 7, 2013, 9:31 a.m.)


Review request for Dolphin.


Changes
-------

Updated to current Dolphin Master.

Only open folders (files doesn't make sense here) in new tabs, when dropping files/folders over the empty tab-bar area.


Description
-------

* Implemented tab-bar drag and drop (with drop indicator) - When you drag folders to the empty tab-bar area, a drop indicator appears, 
  when you drop the folders there, all folders will be opened in new tabs.

* Factored out the tabbing code from DolphinMainWindow into different classes (This is smth. Peter had already in mind some time ago). 
  It reduces the LOC in DolphinMainWindow and should make the maintenance easier in the future.

* Use the the power of QTabWidget + tab pages instead of just hiding/showing the DolphinViews in the layout (No flickering on tab change anymore)

New classes:
* DolphinTabPage - handles the two views + splitter stuff + layout stuff -> no need of hiding/showing view widgets anymore ;)
* DolphinTabBar - handles drag and drop events + context menu requests on the tab bar. (QTabBar based instead of KTabBar)
* DolphinTabWidget - based on QTabWidget, does the view handling (connector between all tab pages + tab bar)

The DolphinTabWidget is the new central widget for DolphinMainWindow.

To all Dolphin/KDE devs out there, please help me to find all the regressions/bugs caused by this patch and please review this code. Thanks! :)


This addresses bug 216433.
    http://bugs.kde.org/show_bug.cgi?id=216433


Diffs (updated)
-----

  dolphin/src/CMakeLists.txt ffb232c 
  dolphin/src/dolphinmainwindow.h 7da5801 
  dolphin/src/dolphinmainwindow.cpp 347489d 
  dolphin/src/dolphintabbar.h PRE-CREATION 
  dolphin/src/dolphintabbar.cpp PRE-CREATION 
  dolphin/src/dolphintabpage.h PRE-CREATION 
  dolphin/src/dolphintabpage.cpp PRE-CREATION 
  dolphin/src/dolphintabwidget.h PRE-CREATION 
  dolphin/src/dolphintabwidget.cpp PRE-CREATION 
  dolphin/src/dolphinviewcontainer.cpp de1ae4b 

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


Testing
-------


Thanks,

Emmanuel Pescosta

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


More information about the kfm-devel mailing list