https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40092

Aleisha Amohia <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         QA Contact|[email protected] |[email protected]
                   |y.org                       |

--- Comment #11 from Aleisha Amohia <[email protected]> ---
(In reply to Andrii Nugged from comment #7)
> Notes for QA phase: we need to decide if `.classList.add(notFilledClass);`
> is universal and do not interfere/future proof (looks like). If so, then
> remove another proposed but commented out solution `// elt.setAttribute...`.

Thank you Andrii. I decided to use the 'safer' commented out solution because
then we're clear on exactly which classes should be present. This is a symptom
of a bug which happens earlier in the process, where the framework_plugin class
isn't passed back to the template as it should, probably in a Perl script,
which we would be able to add a test for... but since this is a major bug,
let's get this fix in and work out how to programmatically ensure the right
classes are passed through later.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to