On 2009-12-29, Patrick McCarty wrote:
> On Mon, Dec 28, 2009 at 2:26 PM, Graham Percival
> <gra...@percival-music.ca> wrote:
> > The below patch stops texi2html from creating two anchors, making the
> > webpage navigation much more comfortable. However, it also makes the
> > TOC navbars disappear.
> >
> > diff --git a/Documentation/lilypond-texi2html.init
> > b/Documentation/lilypond-texi
> > index 1ff5d8a..2ca1a67 100644
> > --- a/Documentation/lilypond-texi2html.init
> > +++ b/Documentation/lilypond-texi2html.init
> > @@ -733,7 +733,10 @@ sub lilypond_element_target_name($$$)
> > # TODO: Once texi2html correctly prints out the target and not the id for
> > # the sections, change this back to ($id, $target)
> > $target = lc($target);
> > - return ($target, $target);
> > +
> > +# removes duplicate anchor generation
> > + return ($target, "");
> > +# return ($target, $target);
> > }
>
> The HTML for the second-layer TOC is missing after applying this
> patch, so the CSS is not broken, just the HTML. div#tocframe is still
> generated, but only for the top-level TOC.
>
> Also, it's only an accident that this would make the webpage
> navigation more comfortable, since the real solution is to remove the
> anchors from the hyperlink URLs when the texinfo is split by section
> (I'm working on it). With this patch, the hyperlink anchors point to
> a non-existent anchors; while harmless, it's still annoying. :-)
After an hour digging through the texi2html source, I was not able to
figure it out, so I'll come back this issue in a couple of days.
Though, as an experiment, I tried nulling the first return value
instead of the second, and the second-layer TOC did not disappear. I
don't know if this causes problems elsewhere though. See the diff
below.
Thanks,
Patrick
diff --git a/Documentation/lilypond-texi2html.init
b/Documentation/lilypond-texi2html.init
index 1ff5d8a..2906347 100644
--- a/Documentation/lilypond-texi2html.init
+++ b/Documentation/lilypond-texi2html.init
@@ -733,7 +733,10 @@ sub lilypond_element_target_name($$$)
# TODO: Once texi2html correctly prints out the target and not the id for
# the sections, change this back to ($id, $target)
$target = lc($target);
- return ($target, $target);
+
+ # remove duplicate anchor generation
+ return ("", $target);
+ #return ($target, $target);
}
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel
