On Fri, Nov 18, 2022 at 05:40:51PM -0800, Per Bothner wrote:
> On 11/18/22 14:43, Gavin Smith wrote:
> > Unfortunately, it has various obvious problems. Most significantly, the
> > index search does not work.
>
> The problem is line 1459 in info.js:
>
> var index_links = document.querySelectorAll ("td[valign=top] a");
> The <td> elements in index no longer have the valign="top" attribute,
> so this query fails to find anything. Instead we can do:
>
> var index_links = document.querySelectorAll ("td a");
>
> Being a bit stricter in the search seems to make sense to me.
> Maybe something like:
> var index_links = document.querySelectorAll ("div.printindex td a");
>
> I don't know if this is the right thing - i.e. if it matches all the indices
> we need.
I think that it should be even stricter, otherwise you'll also get the
section names associated with the indices. I think that it should be
td.printindex-index-entry a
The div.printindex is ok, but should be redundant with
td.printindex-index-entry.
There could be more to do in relation to @subentry implementaiton, but
I'll do a separate mail on that subject.
--
Pat
