The 'clear filter' button is permanently disabled on the patron search results page (member.pl). It shouldn't be.
Created attachment 134061 [details] [review] Bug 30607: Don't overload our default initComplete We want to keep the default initComplete call to add/remove the 'disabled' class on the 'Clear filters' link. Test plan: Search for patrons, add something to the general DT search and confirm that the 'Clear filters' link can be clicked (ie. is not disabled) when the input field is not empty.
Is there a more elegant way?
Created attachment 134450 [details] [review] Bug 30607: Don't overload our default initComplete We want to keep the default initComplete call to add/remove the 'disabled' class on the 'Clear filters' link. Test plan: Search for patrons, add something to the general DT search and confirm that the 'Clear filters' link can be clicked (ie. is not disabled) when the input field is not empty. Signed-off-by: David Nind <david@davidnind.com>
QAing
(In reply to Jonathan Druart from comment #2) > Is there a more elegant way? Good question ;)
Created attachment 134695 [details] [review] Bug 30607: Don't overload our default initComplete We want to keep the default initComplete call to add/remove the 'disabled' class on the 'Clear filters' link. Test plan: Search for patrons, add something to the general DT search and confirm that the 'Clear filters' link can be clicked (ie. is not disabled) when the input field is not empty. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Agree that it does not look very elegant.
I couldn't come up with a more elegant way.. been mulling it over for a few days and I came up empty.. so I fully agree with the PQA here :)
Pushed to master for 22.05, thanks to everybody involved [U+1F984]