Review Request 118440: A quick shortcut to show/hide active panels in order to gain screen space

[James Abtahi]

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

(Updated June 1, 2014, 7:56 p.m.)


Review request for Dolphin.


Repository: kde-baseapps


Description
-------

In smaller screens, various panels of Dolphin can occupy a good portion of the screen. At many situation, I just want to quickly hide all those panels to have a better control over the content and make them reappear again with just one quick shortcut. At the moment, there only solution is to use all the F4, F7, F9, and F11 individually to achieve this which is a very inefficient and irritating solution. 

NOTE: I'm NOT looking for just simply toggling panel's visibility on and off (that's too simple to implement). The mechanism I'm looking for is that Dolphin saves the visibility of current panels, hide them when the assigned key (e.g. ESC) is pressed, and make EXACTLY the same previously shown panels to reappear again when ESC is pressed. A similar (yet simpler) solution exist in Nemo (Cinommon's file manager). 

The "panelvisibility.patch" does the job very well and works with the latest Dolphin 4.99.0. There are some refinements yet to be done for this patch but first I wanted to know what you think about this feature getting added to mainstream Dolphin (really handy for people like me).

This feature is available through "View > Panels > Show/Hide Active Panels" and ESC shortcut.   

    


Diffs
-----

  dolphin/src/dolphinmainwindow.h 1192f6e 
  dolphin/src/dolphinmainwindow.cpp 049c440 
  dolphin/src/statusbar/dolphinstatusbar.cpp 9f17c8e 

Diff: https://git.reviewboard.kde.org/r/118440/diff/


Testing
-------


Thanks,

James Abtahi

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