Re: info.js demo on Texinfo manual updated on website

[Patrice Dumas]

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