Review Request 118994: make CTRL+SHIFT+T reopen last closed tab

[Frank Reininghaus]

> On June 29, 2014, 2:50 p.m., Emmanuel Pescosta wrote:
> > Thanks for the patch!
> > 
> > I really like the idea to implement CTRL+SHIFT+T to reopen the previously closed tab, makes Dolphin more browser like :)
> > 
> > But please implement this feature in master, because:
> > 1. Dolphins frameworks branch is currently only for fixing KF5/Qt5 related bugs
> > 2. We changed quite a few recent tabs related lines (new DolphinRecentTabsMenu)
> > 
> > So please implement this behavior in DolphinRecentTabsMenu. Thanks!

If you only have a Qt5/KF5 build environment, and you don't want to set up a separate Qt4+KDE SC 4.x build (which would be understandable, because development there will stop at some point), we can also backport small patches from the frameworks branch to master for you. In most cases, small patches should apply to both branches with minor modifications. At least if master is fully merged into frameworks, which I have done a few minutes ago, so Emmanuel's DolphinRecentTabsMenu is also in frameworks now.

Thanks for your help!


- Frank


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


On June 28, 2014, 2:44 p.m., Arjun Ak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118994/
> -----------------------------------------------------------
> 
> (Updated June 28, 2014, 2:44 p.m.)
> 
> 
> Review request for Dolphin and KDE Base Apps.
> 
> 
> Bugs: 336818
>     http://bugs.kde.org/show_bug.cgi?id=336818
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Upon pressing CTRL+SHIFT+T, the most recently closed tab is reopened (with the help of Go->'Recently closed tabs' menu).
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinmainwindow.h 1192f6e 
>   dolphin/src/dolphinmainwindow.cpp 7c9001b 
>   dolphin/src/dolphinui.rc 52826bb 
> 
> Diff: https://git.reviewboard.kde.org/r/118994/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun Ak
> 
>

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