Download
We're evolving to serve you better! This current forum has transitioned to read-only mode. For new discussions, support, and engagement, we've moved to GitHub Discussions.

Search input label is missing

  • #2818
    Avatar photo[anonymous]

    Hi there,

    It’s no biggies but it should be fixed on all themes. Search Input element does not have associated labels.

    Read on: https://web.dev/label/

    #2819
    Avatar photoBob

    Hi Yoav,

    In cases where the label breaks styles or when we use only search icon, instead of a label, we can add the aria-label to the search input and if I remember well almost in all themes I used this attribute.

    #2820
    Avatar photo[anonymous]

    Yeah, I think the aria-label should improve the overall score. The Documentation theme doesn’t have it.

    #2822
    Avatar photoBob

    It has, and the lighthouse does not show any issue with that.

    #2824
    Avatar photo[anonymous]

    Weird, I cannot see it the latest version (1.4.0.0) for some reasons, even after looking in the .hbs files to be certain.

    #2825
    Avatar photoBob

    What search engine do you use, Google or DocSearch?

    #2826
    Avatar photo[anonymous]

    Google’s Custom Search. I do see the tag it for Algolia indeed.

    Edit: I’ve also found another, bigger, problem: https://forum.getpublii.com/topic/amp-validation-errors/

    #2827
    Avatar photoBob

    Yeap, I forgot to add the aria-label to the Google search input, will do it the next theme release.

    #2828
    Avatar photo[anonymous]

    Yeah, no worries. Thanks 🙂

    Can we have also a look at: https://forum.getpublii.com/topic/amp-validation-errors/

    #2829
    Avatar photoBob
    [anonymous] wrote:

    Can we have also a look at: https://forum.getpublii.com/topic/amp-validation-errors/

    Yes, but the fix must be applied to the app, so I wouldn’t expect it too soon.