[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bugs related to tabs
On 2019-05-22 07:59, Boruch Baum wrote:
> On 2019-05-22 15:54, Katsumi Yamaoka wrote:
> > ...
> > 1.
> > In the emacs-w3m home page <http://emacs-w3m.namazu.org/>,
> > when I type <S-return> (`w3m-view-this-url-new-session') on
> > the name link `Features', the new tab appears and it shows
> > the Features section of the page. That's ok. However, if I
> > switch the tab to the original one, I can see the Features
> > section shown also there. But what the original tab displays
> > should not be changed of course.
>
> I can reproduce this bug by performing "M-x
> w3m-view-this-url-new-session" and "C-u C-u M-x w3m-view-this-url", but
> it does seem to function correctly when performing "M-x
> w3m-goto-url-new-session".
Does the following fix the situation for you?
(defun w3m-view-this-url-new-session ()
"Display the page of the link under point in a new session.
If the region is active, use the `w3m-open-all-links-in-new-session'
command instead."
(interactive)
(if (w3m-region-active-p)
(call-interactively 'w3m-open-all-links-in-new-session)
(let ((url (w3m-url-valid (w3m-anchor))))
(if url
(w3m-goto-url-new-session url)
; (w3m--message t 'w3m-error "No URL at point")))))
(w3m-message "No URL at point")))))
There is much duplication between the code of w3m-goto... and
w3m-view..., the only difference I can see from the user's perspective
is that w3m-view... doesn't prompt the user. If that's correct, then
much more of the code could be combined, and that would make maintenance
that much easier.
--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1 7286 0036 9E45 1595 8BC0