"Show All" fails if search starts with capital letter #8
Comments
mshenfield
added a commit
to mshenfield/lending-library
that referenced
this issue
Mar 22, 2017
This replaces some of the toggling with a CSS class on the tools container, that when set, along with the `not-available` class on each item, set's their display property to `none`. When the button is pressed, this just toggles the container class as well. This also updates the filtering logic to work with __SHOW AVAILABLE__. jQuery's `show` method adds inline style `display: block;`, overriding any previous logic. This again replaces jQuery's `show`/`hide` with a class, `filtered`, that is hidden with CSS. A lot of jQuery code re-running filters, etc. get to be deleted! I'm not 100% sure splitting the display logic between the CSS/JS is a great idea though. Closes jlord#8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Search "Ci" (with a capital "C"):
Clicking "Show Available" clears claimed items:
Clicking "Show All" shows nothing:
Searching with "ci" (lower case "c") "Show All" works as expected:
The text was updated successfully, but these errors were encountered: