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

Mauro Carvalho Chehab
2026-05-05T13:00:14Z

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 = `
> > + Filter:
> > +
>
> 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.,
>
> Filter: (subsystem or property, case-sensitive)
>
> > +
> > + `;

Makes sense. Changed the logic to:

filt_div.innerHTML = `
Filter:

subsystem or property (case-sensitive)

`;

--
Thanks,
Mauro