Messages in this thread |  | | Date | Mon, 4 May 2026 14:12:50 -0700 | | Subject | Re: [PATCH 9/9] docs: maintainers: add a filtering javascript | | From | Randy Dunlap <> |
| |
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> > + `;
-- ~Randy
|  |