Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARIA Practices Guide breaks search #357

Closed
yatil opened this issue Aug 10, 2023 · 4 comments · Fixed by #362
Closed

ARIA Practices Guide breaks search #357

yatil opened this issue Aug 10, 2023 · 4 comments · Fixed by #362

Comments

@yatil
Copy link
Contributor

yatil commented Aug 10, 2023

When searching for a phrase present in the ARIA Practices Guide documentation, I get many entries that do not contain a link to the search result. The reason is that the search uses front matter information to put these results together. The APG uses HTML documents that do not follow the template.

Example: https://www.w3.org/WAI/search/?q=landmarks

How to fix:

  1. Make APG follow the template and add expected metadata
  2. Remove APG from the search index.
yatil added a commit to yatil/wai-website that referenced this issue Aug 10, 2023
@yatil
Copy link
Contributor Author

yatil commented Aug 11, 2023

Attn: @iadawn & @shawna-slh

@yatil
Copy link
Contributor Author

yatil commented Oct 3, 2023

👋

@mcking65
Copy link

mcking65 commented Oct 4, 2023

See alternative solution in #362, which is a solution that targets the 12 problematic pages in the APG. The APG Task Force will resolve the issues with the problematic pages so that we can later undo their exclusion from search.

@remibetin
Copy link
Member

@yatil Thanks for raising the issue.

I have just merged and deployed #362 temporary fix to exclude the concerned pages.

See https://www.w3.org/WAI/search/?q=landmarks

cc @mcking65 @shawna-slh @howard-e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants