unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* help-go-back
@ 2004-05-08 20:44 Karl Chen
  2004-05-08 21:32 ` help-go-back Juri Linkov
  2004-05-19  0:57 ` help-go-back Stefan Monnier
  0 siblings, 2 replies; 10+ messages in thread
From: Karl Chen @ 2004-05-08 20:44 UTC (permalink / raw)



`Help-go-back' hasn't been working.  Here are two possible fixes.

I think `help-go-back' was kludgey before (that's why it broke) so
patch #2 is better.

It would also be nice if there were a `help-go-forward' (opposite
of `help-go-back').  I can write this if there is interest.


--- /home/quarl/local/Node-Linux-i686/stow/emacs-cvs/share/emacs/21.3.50/lisp/help-mode.el	Sat May  8 13:24:44 2004
+++ /tmp/buffer-content-33160vh	Sat May  8 13:24:58 2004
@@ -579,7 +579,7 @@
 (defun help-go-back ()
   "Invoke the [back] button (if any) in the Help mode buffer."
   (interactive)
-  (let ((back-button (button-at (1- (point-max)))))
+  (let ((back-button (button-at (- (point-max) 2))))
     (if back-button
 	(button-activate back-button)
       (error "No [back] button"))))


--- /home/quarl/local/Node-Linux-i686/stow/emacs-cvs/share/emacs/21.3.50/lisp/.backup/help-mode.el.~1~	Thu Apr 29 11:43:32 2004
+++ /home/quarl/local/Node-Linux-i686/stow/emacs-cvs/share/emacs/21.3.50/lisp/help-mode.el	Sat May  8 13:35:05 2004
@@ -577,12 +577,11 @@
 	(goto-char position)))))
 
 (defun help-go-back ()
-  "Invoke the [back] button (if any) in the Help mode buffer."
+  "Go back to previous help buffer."
   (interactive)
-  (let ((back-button (button-at (1- (point-max)))))
-    (if back-button
-	(button-activate back-button)
-      (error "No [back] button"))))
+  (if help-xref-stack
+      (help-xref-go-back (current-buffer))
+    (error "No previous help buffer.")))
 
 (defun help-do-xref (pos function args)
   "Call the help cross-reference function FUNCTION with args ARGS.



-- 
Karl 2004-05-08 13:26

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

* Re: help-go-back
  2004-05-08 20:44 help-go-back Karl Chen
@ 2004-05-08 21:32 ` Juri Linkov
  2004-05-08 22:24   ` help-go-back Karl Chen
  2004-05-19  0:57 ` help-go-back Stefan Monnier
  1 sibling, 1 reply; 10+ messages in thread
From: Juri Linkov @ 2004-05-08 21:32 UTC (permalink / raw)
  Cc: emacs-devel

Karl Chen <quarl@hkn.eecs.berkeley.edu> writes:
> `Help-go-back' hasn't been working.  Here are two possible fixes.
>
> I think `help-go-back' was kludgey before (that's why it broke) so
> patch #2 is better.

I think the second patch is better, because it doesn't rely on
the location of the [back] button.

> It would also be nice if there were a `help-go-forward' (opposite
> of `help-go-back').  I can write this if there is interest.

It is not possible to implement it with the current `help-xref-stack'
variable.  A new variable could be added that will record the full
Help history.  And it might be as useful as full history of Info nodes
where an additional variable with full history was added recently.

-- 
Juri Linkov
http://www.jurta.org/emacs/

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

* Re: help-go-back
  2004-05-08 21:32 ` help-go-back Juri Linkov
@ 2004-05-08 22:24   ` Karl Chen
  2004-05-09  0:08     ` help-go-back Juri Linkov
  0 siblings, 1 reply; 10+ messages in thread
From: Karl Chen @ 2004-05-08 22:24 UTC (permalink / raw)


>>>>> "Juri" == Juri Linkov <juri@jurta.org> writes:
    >> It would also be nice if there were a `help-go-forward'
    >> (opposite of `help-go-back').  I can write this if there is
    >> interest.
    Juri> 
    Juri> It is not possible to implement it with the current
    Juri> `help-xref-stack' variable.  A new variable could be
    Juri> added that will record the full Help history.  And it
    Juri> might be as useful as full history of Info nodes where
    Juri> an additional variable with full history was added
    Juri> recently.

I was thinking a list of nodes to go forward to (reverse of the
'back' stack).  I'm not sure what you mean re. Info nodes, do you
want to combine the two histories?

-- 
Karl 2004-05-08 15:21

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

* Re: help-go-back
  2004-05-08 22:24   ` help-go-back Karl Chen
@ 2004-05-09  0:08     ` Juri Linkov
  2004-05-09  0:48       ` help-go-back Nick Roberts
  2004-05-09 18:47       ` help-go-back Richard Stallman
  0 siblings, 2 replies; 10+ messages in thread
From: Juri Linkov @ 2004-05-09  0:08 UTC (permalink / raw)
  Cc: emacs-devel

Karl Chen <quarl+dated+1084486903.a9a7a9@nospam.quarl.org> writes:
>>>>>> "Juri" == Juri Linkov <juri@jurta.org> writes:
>     >> It would also be nice if there were a `help-go-forward'
>     >> (opposite of `help-go-back').  I can write this if there is
>     >> interest.
>     Juri> It is not possible to implement it with the current
>     Juri> `help-xref-stack' variable.  A new variable could be
>     Juri> added that will record the full Help history.  And it
>     Juri> might be as useful as full history of Info nodes where
>     Juri> an additional variable with full history was added
>     Juri> recently.
>
> I was thinking a list of nodes to go forward to (reverse of the
> 'back' stack).  I'm not sure what you mean re. Info nodes, do you
> want to combine the two histories?

No, I meant not to combine, but to implement the full Help history list
similar to full Info history list recently implemented, i.e. a list
of all visited Help elements which is displayed in the Help buffer by
a special command, and where older help elements could be revisited by
selecting an element from the full history list.

However, AFAIU, you propose a stack of forward elements, which is
filled by `help-xref-go-back', used by a new `help-xref-go-forward'
command and cleared after the first visit of some other cross-reference.
This may be useful as well, although generally this is a subset of
the full history list.

BTW, since you misunderstood me in such a way that I proposed to
combine the Info and Help histories, it seems it was not accidentally,
because more close integration of Help with Info seems very natural.
Many improvements could be done here.  At least, the Help buffer could
provide automatic buttons with links to Info manuals and call
something like `info-lookup-symbol' or `Info-goto-emacs-command-node'.

-- 
Juri Linkov
http://www.jurta.org/emacs/

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

* Re: help-go-back
  2004-05-09  0:08     ` help-go-back Juri Linkov
@ 2004-05-09  0:48       ` Nick Roberts
  2004-05-09  1:11         ` help-go-back Juri Linkov
  2004-05-09 18:47       ` help-go-back Richard Stallman
  1 sibling, 1 reply; 10+ messages in thread
From: Nick Roberts @ 2004-05-09  0:48 UTC (permalink / raw)
  Cc: Karl Chen, emacs-devel


 > Many improvements could be done here.  At least, the Help buffer could
 > provide automatic buttons with links to Info manuals and call
 > something like `info-lookup-symbol' or `Info-goto-emacs-command-node'.

It already does. Do `C-h f' on vc-directory, for example.

Nick

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

* Re: help-go-back
  2004-05-09  0:48       ` help-go-back Nick Roberts
@ 2004-05-09  1:11         ` Juri Linkov
  2004-05-09  6:52           ` help-go-back Eli Zaretskii
  0 siblings, 1 reply; 10+ messages in thread
From: Juri Linkov @ 2004-05-09  1:11 UTC (permalink / raw)
  Cc: Karl Chen, emacs-devel

Nick Roberts <nick@nick.uklinux.net> writes:
>  > Many improvements could be done here.  At least, the Help buffer could
>  > provide automatic buttons with links to Info manuals and call
>  > something like `info-lookup-symbol' or `Info-goto-emacs-command-node'.
>
> It already does. Do `C-h f' on vc-directory , for example.

Yes, it does, but only for a few functions where the text
"See Info node ..." was added manually.

What I proposed is to add this text automatically in the Help buffer
for every function described in Info manuals.

-- 
Juri Linkov
http://www.jurta.org/emacs/

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

* Re: help-go-back
  2004-05-09  1:11         ` help-go-back Juri Linkov
@ 2004-05-09  6:52           ` Eli Zaretskii
  2004-05-09 11:25             ` help-go-back Juri Linkov
  0 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2004-05-09  6:52 UTC (permalink / raw)
  Cc: nick, quarl+dated+1084486903.a9a7a9, emacs-devel

> From: Juri Linkov <juri@jurta.org>
> Date: Sun, 09 May 2004 04:11:29 +0300
> 
> Yes, it does, but only for a few functions where the text
> "See Info node ..." was added manually.
> 
> What I proposed is to add this text automatically in the Help buffer
> for every function described in Info manuals.

You mean, you want Help to scan the Info manuals before it describes
the Help buffer?  (How else will it know that a function is documented
in some manual?)  Won't this make Help too slow?

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

* Re: help-go-back
  2004-05-09  6:52           ` help-go-back Eli Zaretskii
@ 2004-05-09 11:25             ` Juri Linkov
  0 siblings, 0 replies; 10+ messages in thread
From: Juri Linkov @ 2004-05-09 11:25 UTC (permalink / raw)
  Cc: nick, quarl+dated+1084486903.a9a7a9, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:
>> From: Juri Linkov <juri@jurta.org>
>> Date: Sun, 09 May 2004 04:11:29 +0300
>> 
>> Yes, it does, but only for a few functions where the text
>> "See Info node ..." was added manually.
>> 
>> What I proposed is to add this text automatically in the Help buffer
>> for every function described in Info manuals.
>
> You mean, you want Help to scan the Info manuals before it describes
> the Help buffer?  (How else will it know that a function is documented
> in some manual?)  Won't this make Help too slow?

It can be optimized by caching Info indexes in Lisp structures.

-- 
Juri Linkov
http://www.jurta.org/emacs/

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

* Re: help-go-back
  2004-05-09  0:08     ` help-go-back Juri Linkov
  2004-05-09  0:48       ` help-go-back Nick Roberts
@ 2004-05-09 18:47       ` Richard Stallman
  1 sibling, 0 replies; 10+ messages in thread
From: Richard Stallman @ 2004-05-09 18:47 UTC (permalink / raw)
  Cc: quarl+dated+1084486903.a9a7a9, emacs-devel

    No, I meant not to combine, but to implement the full Help history list
    similar to full Info history list recently implemented, i.e. a list
    of all visited Help elements which is displayed in the Help buffer by
    a special command, and where older help elements could be revisited by
    selecting an element from the full history list.

We don't really need this feature, and based on my experience making
changes in that code, adding this is very likely to be break things.
So let's stop thinking about it.

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

* Re: help-go-back
  2004-05-08 20:44 help-go-back Karl Chen
  2004-05-08 21:32 ` help-go-back Juri Linkov
@ 2004-05-19  0:57 ` Stefan Monnier
  1 sibling, 0 replies; 10+ messages in thread
From: Stefan Monnier @ 2004-05-19  0:57 UTC (permalink / raw)
  Cc: Emacs Developement List

> `Help-go-back' hasn't been working.  Here are two possible fixes.

Thank you.  I've installed your second fix.


        Stefan

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

end of thread, other threads:[~2004-05-19  0:57 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-05-08 20:44 help-go-back Karl Chen
2004-05-08 21:32 ` help-go-back Juri Linkov
2004-05-08 22:24   ` help-go-back Karl Chen
2004-05-09  0:08     ` help-go-back Juri Linkov
2004-05-09  0:48       ` help-go-back Nick Roberts
2004-05-09  1:11         ` help-go-back Juri Linkov
2004-05-09  6:52           ` help-go-back Eli Zaretskii
2004-05-09 11:25             ` help-go-back Juri Linkov
2004-05-09 18:47       ` help-go-back Richard Stallman
2004-05-19  0:57 ` help-go-back Stefan Monnier

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).