search

skeeto / elfeed

An Emacs web feeds client
The Unlicense
1.51k stars 118 forks source link

Don't skip `link:` in `elfeed-show-next-link` #482

Open ParetoOptimalDev opened 1 year ago

ParetoOptimalDev commented 1 year ago

For instance in:

image

I'd like hitting TAB to go to Link: https://news.ycombinator.com/item?id=35062370.

It looks like a simple change if it's universally agreeable, or even agreeable behind a config flag with:

(defun elfeed-show-next-link ()
  "Skip to the next link, exclusive of the Link header."
  (interactive)
  (let ((properties (text-properties-at (line-beginning-position))))
    (when (memq 'message-header-name properties)
+    (previous-line)
      (forward-paragraph))
    (shr-next-link)))
plantarum commented 1 year ago

I had the same idea. After some investigating, I assume the reason this isn't the default behaviour is that you can already open the link for an entry with b, bound to elfeed-show-visit.