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

[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@xxxxxxxxxx>
> ---
>  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