Re: [PATCH 9/9] docs: maintainers: add a filtering javascript

[Mauro Carvalho Chehab <mchehab+huawei@kernel.org>]

On Mon, 4 May 2026 14:12:50 -0700
Randy Dunlap <rdunlap@infradead.org> wrote:

> On 5/4/26 8:51 AM, Mauro Carvalho Chehab wrote:
> > The maintainers table is big. Add a javascript to allow filtering
> > it. Such script is only added at the page which contains the
> > maintainers-include tag.
> > 
> > I opted to keep the search case-sensitive, as, this way,
> > upper case searches at subsystem.
> > 
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> > ---
> >  Documentation/sphinx/maintainers_include.py | 77 +++++++++++++++++++--
> >  1 file changed, 71 insertions(+), 6 deletions(-)
> > 
> > diff --git a/Documentation/sphinx/maintainers_include.py b/Documentation/sphinx/maintainers_include.py
> > index bbdadf2aa4f3..f85298627da2 100755
> > --- a/Documentation/sphinx/maintainers_include.py
> > +++ b/Documentation/sphinx/maintainers_include.py  
> 
> > +  function addInput() {
> > +    const table = document.getElementById("maintainers-table");
> > +    if (!table) return;
> > +    let input = document.getElementById("filter-table");
> > +    if (!input) {
> > +      const filt_div = document.createElement('div');
> > +      filt_div.innerHTML = `
> > +        <p>Filter:
> > +          <input type="search" id="filter-table" placeholder="subsystem or property (case-sensitive)" />  
> 
> The "placeholder" string does not fit inside the prompt/entry block so it is
> truncated with no way to tell what the missing part is AFAICT.
> Maybe include that after "Filter:"? E.g.,
> 
>            <p>Filter: (subsystem or property, case-sensitive)
> 
> > +        </p>
> > +      `;  

Makes sense. Changed the logic to:

      filt_div.innerHTML = `
        <p>Filter:
          <input type="search" id="filter-table" placeholder="search string"/>
          subsystem or property (case-sensitive)
        </p>
      `;

-- 
Thanks,
Mauro