Re: [PATCH] New tab-bar-detach-tab command

[Juri Linkov <juri@linkov.net>]

>> Maybe there is no problem when the command name will contain the word
>> "clone", but its menu item will be like in web browsers:
>>
>>         `(menu-item "Duplicate" (lambda () (interactive)
>>             (tab-bar-clone-tab
>
> I'd vote for naming the menu item to match the command, but I don't
> feel strongly about it.  Maybe the menu could be seen as a place to
> use terms more common in other software, in which case the disparity
> would make sense.

Another problem is that web browsers have such context menu items:
"Duplicate" and "Move tab to new window", but we have "Detach"
for the latter.  OTOH, "Move tab to new window" is overly long.
So only the help string could use the word "Move" like in browsers.

>> And wouldn't it be too error-prone when the name 'tab-bar-clone-tab'
>> will differ only by one letter from another name 'tab-bar-close-tab'?
>
> Seems okay to me.  It's probably not the only case in which two
> commands' names differ by a letter.

Currently it's easy to complete for M-x with just 'M-x tab-d RET'
since there is no 'tab-detach', only 'tab-duplicate'.
But with 'tab-clone' similar to 'tab-close', even 'M-x tab-clo'
is still not unique.

BTW, maybe an alias 'tab-detach' could be added as well?

>> > For example, `clone-buffer' and `clone-process' both
>> > "Create a twin copy of...".
>>
>> 'tab-duplicate' is bound to 'C-x t n' and not to 'C-x t c' because
>> 'clone-buffer' is bound to 'C-x x n' and not 'C-x x c'.
>
> Hm, "C-x x c" is unbound by default, so I'd vote for moving
> `clone-buffer' to it.  :)  But, again, I don't feel strongly about
> this, as these aren't commands I use often.

I guess 'clone-buffer' was bound to 'C-x x n' globally because
the same 'clone-buffer' is bound to 'M-n' in Info-mode.