unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Duplicate bindings? Ref: Ibuffer: Mark buffers by content
@ 2016-07-05 15:03 Kaushal Modi
  2016-07-05 15:31 ` Tino Calancha
  0 siblings, 1 reply; 8+ messages in thread
From: Kaushal Modi @ 2016-07-05 15:03 UTC (permalink / raw)
  To: tino.calancha, Emacs developers

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

In this commit (
http://git.savannah.gnu.org/cgit/emacs.git/commit/?id=08974112ae68aefba658a8516c8faa3374edc924
),
I am wondering why we have two bindings "%c" and "%g" call the same command?

Why not have just one binding? That way, the user's get used to the same
binding; we won't have a split where some prefer %c and some prefer %g.
Then there are no bad feelings when/if one of the bindings might need to be
sacrificed in future for some reason.

OTOH, I am excited to try out this new feature.

Thanks!
-- 

-- 
Kaushal Modi

[-- Attachment #2: Type: text/html, Size: 892 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-05 15:03 Duplicate bindings? Ref: Ibuffer: Mark buffers by content Kaushal Modi
@ 2016-07-05 15:31 ` Tino Calancha
  2016-07-05 15:41   ` Kaushal Modi
  2016-07-05 16:36   ` Noam Postavsky
  0 siblings, 2 replies; 8+ messages in thread
From: Tino Calancha @ 2016-07-05 15:31 UTC (permalink / raw)
  To: Kaushal Modi; +Cc: Emacs developers, tino.calancha

[-- Attachment #1: Type: text/plain, Size: 1474 bytes --]



On Tue, 5 Jul 2016, Kaushal Modi wrote:

> In this commit ( http://git.savannah.gnu.org/cgit/emacs.git/commit/?id=08974112ae68aefba658a8516c8faa3374edc924 ), I am wondering
> why we have two bindings "%c" and "%g" call the same command?
> Why not have just one binding? That way, the user's get used to the same binding; we won't have a split where some prefer %c and
> some prefer %g. Then there are no bad feelings when/if one of the bindings might need to be sacrificed in future for some reason.
It was a compromise solution:
I)

My preference is '%g', which is the same as 
`dired-mark-files-containing-regexp'.
That is consistent with the design of Ibuffer, as stated in the
first line of ibuffer.el:
;;; ibuffer.el --- operate on buffers like dired

There are several equivalent commands Dired-Ibuffer with the same
key bindings, for instance:
dired-mark-directories
ibuffer-mark-dired-buffers
both are bound to '*/'

That very much helps to remember then.

Unfortunatelly, this symmetry Dired-Ibuffer is not perfect:
there are Ibuffer commands with the Dired 'partner' having
a different binding. For instance:
`dired-mark-files-regexp' bound to '%m'
`ibuffer-mark-by-name-regexp' bound to '%n'.

II)
Ibuffer has the command:
`ibuffer-filter-by-content' bound to '/c'

So i think one Ibuffer user may find easier to remember the
binding '%c' for `ibuffer-mark-by-content-regexp'.

If people agree to keep just one binding then i would suggest '%g'.
Regards,
Tino

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-05 15:31 ` Tino Calancha
@ 2016-07-05 15:41   ` Kaushal Modi
  2016-07-06  2:18     ` John Wiegley
  2016-07-05 16:36   ` Noam Postavsky
  1 sibling, 1 reply; 8+ messages in thread
From: Kaushal Modi @ 2016-07-05 15:41 UTC (permalink / raw)
  To: Tino Calancha; +Cc: Emacs developers

[-- Attachment #1: Type: text/plain, Size: 1052 bytes --]

On Tue, Jul 5, 2016 at 11:31 AM Tino Calancha <tino.calancha@gmail.com>
wrote:

> If people agree to keep just one binding then i would suggest '%g'.
>

I appreciate the detailed reply.  As I would be fine with either binding,
"%g" would work for me. As this is a new command being bound to a new
binding, I believe that should not be much resistance accepting that. But
let's see if people on this mailing list feel otherwise.

I also like the reasoning for using "%g".

> Unfortunatelly, this symmetry Dired-Ibuffer is not perfect:
there are Ibuffer commands with the Dired 'partner' having
a different binding. For instance:
`dired-mark-files-regexp' bound to '%m'
`ibuffer-mark-by-name-regexp' bound to '%n'.

If there are not many bindings that are inconsistent between the two, it
would be nice to sync up the bindings, deprecate the out-of-sync older
bindings, and then at some point in future, remove them. But while that's a
long process, using "%g" would at least get us getting the dired/ibuffer
bindings more in sync.
-- 

-- 
Kaushal Modi

[-- Attachment #2: Type: text/html, Size: 2632 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-05 15:31 ` Tino Calancha
  2016-07-05 15:41   ` Kaushal Modi
@ 2016-07-05 16:36   ` Noam Postavsky
  1 sibling, 0 replies; 8+ messages in thread
From: Noam Postavsky @ 2016-07-05 16:36 UTC (permalink / raw)
  To: Tino Calancha; +Cc: Emacs developers, Kaushal Modi

On Tue, Jul 5, 2016 at 11:31 AM, Tino Calancha <tino.calancha@gmail.com> wrote:
> If people agree to keep just one binding then i would suggest '%g'.

+1 for just %g



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-05 15:41   ` Kaushal Modi
@ 2016-07-06  2:18     ` John Wiegley
  2016-07-07 14:49       ` Kaushal Modi
  0 siblings, 1 reply; 8+ messages in thread
From: John Wiegley @ 2016-07-06  2:18 UTC (permalink / raw)
  To: Kaushal Modi; +Cc: Emacs developers, Tino Calancha

>>>>> Kaushal Modi <kaushal.modi@gmail.com> writes:

> If there are not many bindings that are inconsistent between the two, it
> would be nice to sync up the bindings, deprecate the out-of-sync older
> bindings, and then at some point in future, remove them. But while that's a
> long process, using "%g" would at least get us getting the dired/ibuffer
> bindings more in sync.

I agree, although I do like the connotation of %g for "grepping by pattern".

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-06  2:18     ` John Wiegley
@ 2016-07-07 14:49       ` Kaushal Modi
  2016-07-07 14:52         ` Tino Calancha
  0 siblings, 1 reply; 8+ messages in thread
From: Kaushal Modi @ 2016-07-07 14:49 UTC (permalink / raw)
  To: Tino Calancha; +Cc: Emacs developers

[-- Attachment #1: Type: text/plain, Size: 137 bytes --]

Hi Tino,

Looks like there no opposition on this proposal.
Can you please remove that extra "%c" binding?

Thanks.
-- 

-- 
Kaushal Modi

[-- Attachment #2: Type: text/html, Size: 317 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-07 14:49       ` Kaushal Modi
@ 2016-07-07 14:52         ` Tino Calancha
  2016-07-07 15:19           ` Kaushal Modi
  0 siblings, 1 reply; 8+ messages in thread
From: Tino Calancha @ 2016-07-07 14:52 UTC (permalink / raw)
  To: Kaushal Modi; +Cc: Emacs developers, Tino Calancha



On Thu, 7 Jul 2016, Kaushal Modi wrote:

> Hi Tino,
> Looks like there no opposition on this proposal.
> Can you please remove that extra "%c" binding?
Sure, i am going to do it now.

I will also start looking what bindings in Ibuffer could be
synchronized with Dired as you suggested.



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: Duplicate bindings? Ref: Ibuffer: Mark buffers by content
  2016-07-07 14:52         ` Tino Calancha
@ 2016-07-07 15:19           ` Kaushal Modi
  0 siblings, 0 replies; 8+ messages in thread
From: Kaushal Modi @ 2016-07-07 15:19 UTC (permalink / raw)
  To: Tino Calancha; +Cc: Emacs developers

[-- Attachment #1: Type: text/plain, Size: 254 bytes --]

Thanks!

On Thu, Jul 7, 2016 at 10:52 AM Tino Calancha <tino.calancha@gmail.com>
wrote:

> Sure, i am going to do it now.
>
> I will also start looking what bindings in Ibuffer could be
> synchronized with Dired as you suggested.
>
-- 

-- 
Kaushal Modi

[-- Attachment #2: Type: text/html, Size: 604 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2016-07-07 15:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-05 15:03 Duplicate bindings? Ref: Ibuffer: Mark buffers by content Kaushal Modi
2016-07-05 15:31 ` Tino Calancha
2016-07-05 15:41   ` Kaushal Modi
2016-07-06  2:18     ` John Wiegley
2016-07-07 14:49       ` Kaushal Modi
2016-07-07 14:52         ` Tino Calancha
2016-07-07 15:19           ` Kaushal Modi
2016-07-05 16:36   ` Noam Postavsky

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).