https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40092 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |aleisha@catalyst.net.nz |y.org | --- Comment #11 from Aleisha Amohia <aleisha@catalyst.net.nz> --- (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.