Replace "style=display:none" with HTML data tags:

When storing metadata in an HTML node, you can use data attributes, which is
more declarative, and doesn't require rendering an invisible node.

See: https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes
This commit is contained in:
dosisod 2022-07-01 19:15:30 -07:00
parent 550fdf24f2
commit 4f17b3d901
2 changed files with 3 additions and 3 deletions

View File

@ -35,8 +35,8 @@ document.addEventListener("DOMContentLoaded", () => {
// for each recipe hide all but matched
recipes.forEach(recipe => {
const recipeName = recipe.textContent.toLowerCase();
const isMatch = searchTerms.every(term => recipeName.includes(term));
const searchString = `${recipe.textContent} ${recipe.dataset.tags}`.toLowerCase();
const isMatch = searchTerms.every(term => searchString.includes(term));
recipe.hidden = !isMatch;
recipe.classList.toggle("matched-recipe", hasFilter && isMatch);

View File

@ -1,5 +1,5 @@
<ul id=artlist>
{{range.Site.RegularPages}}
<li><a href="{{.Permalink}}">{{.Title}}<span style="display:none">{{.Params.Tags}}</span></a></li>
<li data-tags="{{.Params.Tags}}"><a href="{{.Permalink}}">{{.Title}}</a></li>
{{end}}
</ul>