unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: David De La Harpe Golden <david@harpegolden.net>
To: Chong Yidong <cyd@stupidchicken.com>
Cc: Miles Bader <miles@gnu.org>, emacs-devel@gnu.org
Subject: Re: Selection changes
Date: Sat, 17 Jul 2010 04:50:32 +0100	[thread overview]
Message-ID: <4C412888.9080805@harpegolden.net> (raw)
In-Reply-To: <87eif2n7d7.fsf@stupidchicken.com>

On 17/07/10 03:56, Chong Yidong wrote:
>  So, for the
> moment, I went ahead and changed x-select-enable-primary to nil, as you
> requested.
>

Thanks, sorry for the rant. It's 4:50am, time for bed...

The bad "(not (eq (region-beginning) (region-end)))" check is still 
present in deactivate-mark (~simple.el line 3690) and should just be 
removed, I did [try to] explain the problem with it in a previous mail, 
that's a further source of some  poor behaviour (perhaps 
counterintuitively, but that's lazy stuff for you) that might be related 
to some of your points below.

> But I think select-active-regions needs further improvement.

No doubt...

 > Perhaps
> its default behavior should be as follows: for an active region created
> using shift-selection or mouse dragging, Emacs supplies the region text
> to primary.  When such a region is deactivated, Emacs disowns primary
> (as some other apps do, tho not Firefox).

Hmm. I do rather prefer the freezing off of the region into a string 
selection (line just after the bad check above) when deactivating rather 
than completely disowning - I mean, once you disown, the text won't be 
made available for middle-click insertion anymore and you'll be seeing 
"No primary selection" warnings a lot more.

Don't forget, the bad check mentioned above is negatively impacting 
behaviour in this specific area right now, if you remove the bad check 
line, the freezing off will work properly again.

> For an active region created
> simply with C-SPC, no special x-selection handling should be performed.
>

Well, not immediately owning primary on C-SPC /would/ stop most (all?) 
of the possible zero-length-region annoyances (unlike the "bad check") 
in their tracks, but it might be overkill to not own it at all if 
there's an (efficient) way to make emacs just defer taking ownership 
until the newly active region first becomes nonzero length.

That way, visibly highlighted regions would still act the same 
regardless of wether they were mouse/keyboardA/keyboardB created.






  parent reply	other threads:[~2010-07-17  3:50 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-14 18:08 Selection changes Chong Yidong
2010-07-14 18:39 ` Jeff Clough
2010-07-14 18:53   ` Chong Yidong
2010-07-14 19:02     ` Jeff Clough
2010-07-14 19:25 ` Yann Hodique
2010-07-14 20:28   ` Chong Yidong
2010-07-14 23:51 ` David De La Harpe Golden
2010-07-16  1:31 ` Richard Stallman
2010-07-16  2:49   ` Miles Bader
2010-07-17  0:44 ` David De La Harpe Golden
2010-07-17  1:02   ` Miles Bader
2010-07-17  2:28     ` David De La Harpe Golden
2010-07-17  2:56       ` Chong Yidong
2010-07-17  3:30         ` Miles Bader
2010-07-17  3:49           ` Chong Yidong
2010-07-22 21:21           ` Drew Adams
2010-07-22 22:05             ` Chong Yidong
2010-07-23 10:32               ` Eli Zaretskii
2010-07-24 18:44                 ` David De La Harpe Golden
2010-07-24 20:28                   ` Eli Zaretskii
2010-07-24 21:48                     ` David De La Harpe Golden
2010-07-25 16:32                   ` David De La Harpe Golden
2010-07-17  3:50         ` David De La Harpe Golden [this message]
2010-07-17  3:55           ` Chong Yidong
2010-07-17  4:13             ` Chong Yidong
2010-07-17 16:55               ` David De La Harpe Golden
2010-07-18 16:24               ` David De La Harpe Golden
2010-07-17 10:50         ` Wojciech Meyer
2010-07-17 11:01           ` Miles Bader
  -- strict thread matches above, loose matches on Subject: below --
2010-07-16  1:00 Angelo Graziosi
2010-07-16  9:33 ` David De La Harpe Golden
2010-07-17 23:49   ` Angelo Graziosi
2010-07-18 19:28     ` David De La Harpe Golden
2010-07-18 22:39       ` Angelo Graziosi
2010-07-16 12:14 ` Angelo Graziosi
2011-05-27 16:25 Chong Yidong
2011-05-28  4:13 ` David De La Harpe Golden
2011-05-31  0:59   ` Taylor Venable
2011-05-28 11:16 ` Andreas Röhler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C412888.9080805@harpegolden.net \
    --to=david@harpegolden.net \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@gnu.org \
    --cc=miles@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).