[Konsole-devel] Review Request: The "Close Tab" action in context menu is problematic

[Adrián Chaves Fernández]

> On Feb. 10, 2012, 11:17 a.m., Jekyll Wu wrote:
> > Any idea?
> 
> Kurt Hindenburg wrote:
>     Before showing menu, can we check to see if current tab has split view and if so, change the 'Close Tab' to 'Close Split Session'?
>     
>     KPart - As I recall, most that use KPart simply reopen a new konsole part upon 'Close Tab'.  I have often thought it should 'toggle' the open konsole part.  Perhaps allow the 'Close Tab' to be hidden or allow the program to control what happens upon calling it.
>
> 
> Francesco Cecconi wrote:
>     But 'Close Current Terminal' ?

Maybe you could perform a rename in coordination with the Yakuake developer? Having a look at the differences between Konsole and Yakuake, it seems that whay Yakuake calls “sessions” are actually “tabs” (you can “Move Session Left/Right”!!!), where you can have several “sessions” (as understood by KonsolePart) by splitting the tab content in different views — whereas Konsole always has the same session in the same tab, and when the view is splitted in the tab, each view shows the same session.

So I think you could change “session” to “tab” in Yakuake and “Close Tab” to “Close Session” in KonsolePart. 


- Adrián


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


On Jan. 20, 2012, 3:54 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103743/
> -----------------------------------------------------------
> 
> (Updated Jan. 20, 2012, 3:54 a.m.)
> 
> 
> Review request for Konsole and Eike Hein.
> 
> 
> Description
> -------
> 
> The patch is trivial. It just renames that action into "Close Session". What I want to bring into discussion is something more essential.
> 
> The basic problems of "Close Tab" in the context menu :
> 
> 1). It does not make sense in the context of konsolepart, since konsolepart does not support tabs now (probably never?).
> 
> 2). Even in the context of stand-alone Konsole, it is inaccurate and misleading. It not only close current tab, bug also close current session. That difference manifest itself more clearly when using split view. When using split view, there are multiple views(thus tabs) for one session in multiple ViewContainers. Users might get the wrong idea that "Close Tab" only close current tab in this container, until they find the session and all related tabs are gone.
> 
> However, simply renaming it into "Close Session" is also problematic :
> 
> 1). From the point of view of users, 'Session' is less straight forward than the concept of 'Tab'. And We currently provide various operations based upon the concept of 'Tab':  'New Tab', 'Rename Tab', 'Detach Tab', "Copy input to", and the '--new-tab' option. So we either change them all or stick to the concept of 'Tab'.
> 
> 2). Yakuake, the most important user of konsolepart, has its own concept of session and provide its own "Close Session" action. So renaming that action to "Close Session" will cause confusion to yakuake users.
> 
> Review #102630 suggests removing that action from the context menu of konsolepart. I don't think that is a good idea.
> 
> Basically, I think there is no ideal solution to the essential problem until we change the way split view works. Although the current idea is interesting, I guess that is not what most users expect and want. They mostly want the split feature in screen/tmux, terminator and yakuake. In that mode, the relationship between view and session is one-to-one.
> 
> 
> This addresses bug 225854.
>     http://bugs.kde.org/show_bug.cgi?id=225854
> 
> 
> Diffs
> -----
> 
>   src/SessionController.cpp 23b307c 
> 
> Diff: http://git.reviewboard.kde.org/r/103743/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20121203/63ca953a/attachment.html>