[Konsole-devel] Review Request: Rearrange the 'File' menu
Jekyll Wu
adaptee at gmail.com
Sat Feb 11 17:40:30 UTC 2012
> On Feb. 11, 2012, 5:28 p.m., Kurt Hindenburg wrote:
> > The shortcuts might be a bit confusing - Ctrl W = closes tab, Ctrl Q = closes window.
Yes, kind of. But "Ctrl+W" for tab and "Ctrl+Q" for application/window seem to be the common convention in applications supporting tabs.
- Jekyll
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103920/#review10510
-----------------------------------------------------------
On Feb. 10, 2012, 8:06 p.m., Jekyll Wu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103920/
> -----------------------------------------------------------
>
> (Updated Feb. 10, 2012, 8:06 p.m.)
>
>
> Review request for Konsole.
>
>
> Description
> -------
>
> This is kind of a follow up of review #103859.
>
> There are basically two changes:
>
> 1). Rename "Quit" into "Close Window", which is more accurate. That action only close current window, not necessarilly quitting the application.
>
> 2). Move "Close Tab" downwards to be adjacent to "Close Window". Those two actions are conceptually related just like "New Window" and "New Tab".
>
>
> I don't know why, but reviewboard just wouldn't let me upload the patch. So I put the change here:
>
> https://github.com/adaptee/konsole/commit/a7e45130b831a9386ec6195087c52ddf56bbff51
>
>
> Diffs
> -----
>
>
> Diff: http://git.reviewboard.kde.org/r/103920/diff/
>
>
> Testing
> -------
>
>
> Screenshots
> -----------
>
> File Menu
> http://git.reviewboard.kde.org/r/103920/s/429/
>
>
> Thanks,
>
> Jekyll Wu
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120211/6abd1215/attachment.html>
More information about the konsole-devel
mailing list