Skip to content

Commit

Permalink
Merge pull request #191 from Honny1/add-clear-button-to-search-bar
Browse files Browse the repository at this point in the history
Implement clear button for search bar
  • Loading branch information
evgenyz committed Jun 21, 2023
2 parents 196139a + a3b5acf commit 4b347e4
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -128,4 +128,9 @@ const selector = ".pf-c-clipboard-copy__text";
document.querySelectorAll("#copy").forEach(el => el.addEventListener('click', () => {
const el_with_text = el.parentElement.parentElement.parentElement.querySelector(selector);
navigator.clipboard.writeText(el_with_text.textContent);
}));
}));

function clear_input(self) { // eslint-disable-line no-unused-vars
self.previousSibling.value= "";
search_rule(self.previousSibling);
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@
<div class="pf-c-input-group">
<input class="pf-c-form-control pf-m-search" oninput="search_rule(this)" type="search"
placeholder="Search rule" id="input-search-rule" />
<button class="pf-c-button pf-m-control" type="button" aria-label="Clear" onclick="clear_input(this)">
<i class="fas fa-times fa-fw" aria-hidden="true"></i>
</button>
<button class="pf-c-button pf-m-control" type="button" aria-expanded="true" aria-label="Advanced options"
onclick="show_advanced_options(this);">
<i class="fas fa-caret-right" aria-hidden="true"></i>
Expand Down

0 comments on commit 4b347e4

Please sign in to comment.