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

[Arjun Ak]

> On July 21, 2014, 3:26 a.m., Frank Reininghaus wrote:
> > Thank you very much for working on this, Arjun, and thanks also for your very good suggestions, Emmanuel!
> > 
> > I apologize for not responding earlier. I was a bit busy at work recently and did not really find the time to check all mails and review request notifications in my inbox and see which of them could (and should) be handled quickly nevertheless (like this one).
> > 
> > Looks very good from my point of view. Please go ahead and push it. I have one minor suggestion for a change, but feel free to ignore it if you prefer, it's probably a matter of taste.
> > 
> > Thanks again for adding a useful feature, which many users will appreciate.

Which branch should this be pushed to master or frameworks?


- Arjun


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


On July 6, 2014, 7:28 p.m., Arjun Ak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118994/
> -----------------------------------------------------------
> 
> (Updated July 6, 2014, 7:28 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 7c3bff4 
>   dolphin/src/dolphinmainwindow.cpp d9fe645 
>   dolphin/src/dolphinrecenttabsmenu.h 34d4153 
>   dolphin/src/dolphinrecenttabsmenu.cpp a39f994 
>   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: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140721/4542e171/attachment.htm>