when on a list and choosing to tag multiple items the tags are not added and there is no error.
Created attachment 6600 [details] [review] Proposed fix Incorrect check for TagsInputOnList was preventing the right JavaScript from being triggered, and markup was nested inside the wrong logic for the confirmation message to be displayed consistently.
Created attachment 6602 [details] [review] [SIGNED-OFF] Bug 7226 - can't add tags on list Incorrect check for TagsInputOnList was preventing the right JavaScript from being triggered, and markup was nested inside the wrong logic for the confirmation message to be displayed consistently. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Created attachment 6603 [details] [review] [SIGNED-OFF] Bug 7280 - can't place hold without selecting on list JavaScript was looking for clicks on links with class "hold," and incorrectly triggering the check for checked checkboxes. Making the selection stricter lets the JS function as it should while allowing people to place individual holds directly. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment on attachment 6603 [details] [review] [SIGNED-OFF] Bug 7280 - can't place hold without selecting on list Attached wrong.
QA comment: small patch, template change only prove xt/author/valid-templates.t xt/author/valid-templates.t .. ok prove xt/author/translatable-templates.t xt/author/translatable-templates.t .. ok passed QA Patch pushed, please test