Review Request 108399: Add option to make Dolphin split the view horizontally instead of vertically

[Kai Uwe Broulik]

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

Review request for Dolphin and Frank Reininghaus.


Description
-------

Turned out to be a rather straight-foward patch. This adds a config option to Dolphin's behavior settings so you can set it to split the view horizontally (ie. top and bottom) rather than the default vertically (left and right).

I am planning to enhance this overall behavior a bit, but this is the initial thing. Please comment on my idea/plan:
[Split] gets a delayed popup menu: Split Vertically, Split Horizontally. When you just click the button it will just split according to the setings (Vertically by default)
[Close] gets a delayed popup menu: Close left view, Close right view (top and bottom, respectively, so you can control which view to close. When you just click the button it will just close the active view, like it does right now. (Maybe I will even add an option for that as well, so if you preferr that it closes the inactive view by default, that you can do this, since we now already have a "Split" section in the settings). Let me know what you think. :-)


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


Diffs
-----

  dolphin/src/settings/general/behaviorsettingspage.cpp cbbde1d 
  dolphin/src/settings/general/behaviorsettingspage.h 7a9c2f0 
  dolphin/src/dolphinmainwindow.cpp 11e03d0 
  dolphin/src/settings/dolphin_generalsettings.kcfg 849a9c7 

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


Testing
-------

Opened dolphin, split view, closed split view, changed the setting, split again, closed again. And tried with --split cmdline parameter. All worked.


Thanks,

Kai Uwe Broulik

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