[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#39512: 28.0.50; Add command isearch-yank-region
From: |
Juri Linkov |
Subject: |
bug#39512: 28.0.50; Add command isearch-yank-region |
Date: |
Thu, 15 Apr 2021 23:52:03 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (x86_64-pc-linux-gnu) |
> So without any other alternative, `M-s M-.` will be good enough. In
> general it would be better (for consistency) if we "reserve" `M-.`
> "suffixes" for future thing-at-point-or-region commands right?
>
>> In the previous message you sent a link to `ivy-thing-at-point` that
>> also uses (thing-at-point 'url) and also tries to get a filename at point.
>> Do you think `thing-at-point-or-region` should do the same?
>
> With the interactive experience in mind I think this could make sense as
> it does in `ivy-thing-at-point`.
>
> In my opinion from the api point of view, the important modification
> could be the method to know the "kind the thing" detected after calling
> the function (region, word, symbol, url) but also a function to get the
> bounds instead of the text (like bounds-of-thing-at-point-or-region).
I continued trying to do what you suggested some time ago
and immediately stumbled upon a question what "thing"
to use by default as an argument of '(thing-at-point thing)'?
Trying these priorities:
(or (thing-at-point 'region)
(thing-at-point 'url)
;; (thing-at-point 'filename)
;; (thing-at-point 'list)
(thing-at-point 'symbol))
has several problems:
1. There is no such "thing" as 'region'. Maybe could be added to thingatpt.el?
2. 'url' returns nil when there is no url at point, good. But
'filename' returns non-nil on any string, not only on real filenames.
3. It would be nice to use (thing-at-point 'list) only when point
is on the open/close parens. This is how double-clicking by mouse
selects the thing at point of mouse click. When clicked on a paren,
the whole list is selected by 'mouse-start-end'.
'isearch-forward-thing-at-point' could be the same logic.
- bug#39512: 28.0.50; Add command isearch-yank-region,
Juri Linkov <=