[Bug 41439] New: Column filtering on the circ rules table
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Bug ID: 41439 Summary: Column filtering on the circ rules table Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: System Administration Assignee: koha-bugs@lists.koha-community.org Reporter: martin.renvoize@openfifth.co.uk QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com The Circulation Rules table is obscenely wide at this point. We've tried a number of approaches to a new UI in this area, and so far all have got stuck. I'm proposing a smaller impact resolution of just having a set of column groups for display to allow limiting the width of the table to only those fields of interest at the moment you're working. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@openfifth.c |ity.org |o.uk -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #1 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 190455 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190455&action=edit Bug 41439: Add column visibility toggles to circulation rules table The circulation rules table has become excessively wide with 40+ columns, making it difficult to navigate and use. This patch adds column visibility toggle buttons that allow users to focus on specific rule categories. Changes: - Add toggle buttons for: Checkouts, Due dates, Fines, Renewals, Holds, Article requests, Recalls, and Show all - Group columns into logical categories based on their function - Default to showing only Checkout columns on page load for a cleaner initial view - Highlight the active button to indicate current view - Disable search functionality (not appropriate for this table) - Calculate column indices dynamically based on enabled system preferences (UnseenRenewals, ArticleRequests, UseRecalls) Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Observe the table now shows only checkout-related columns by default with the "Checkouts" button highlighted 3. Click different category buttons (Fines, Renewals, Holds, etc.) and verify only relevant columns are shown 4. Verify the active button is highlighted in blue 5. Click "Show all" to display all columns 6. Verify the layout works with different system preference combinations (enable/disable ArticleRequests, UseRecalls, UnseenRenewals) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #2 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 190456 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190456&action=edit Proof of concept -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #3 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- This is just a proof of concept for now and will need testing to ensure it fully works with form submission etc.. I've only quickly trials the 'look'. Setting to Needs Signoff to trigger some feedback.. whilst I do want some feedback and any suggestions of other approaches, lets try to stick to simple solutions for now to limit the scope.. I still think longer term we'll want to re-write this page in Vue with a possibly very different overall UI but that's for another bug. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190455|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #4 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 190459 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190459&action=edit Bug 41439: Add column visibility toggles to circulation rules table The circulation rules table has become excessively wide with 40+ columns, making it difficult to navigate and use. This patch adds column visibility toggle buttons that allow users to focus on specific rule categories. Changes: - Add data-colname attributes to all table headers and footers for robust column identification - Add toggle buttons for: Checkouts, Due dates, Fines, Renewals, Holds, Article requests, Recalls, and Show all - Group columns into logical categories based on their function - Use column name selectors instead of fragile index-based selection for maintainability - Default to showing only Checkout columns on page load for a cleaner initial view - Highlight the active button to indicate current view - Disable search functionality (not appropriate for this table) - Handle dynamic column visibility based on system preferences (UnseenRenewals, ArticleRequests, UseRecalls) Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Observe the table now shows only checkout-related columns by default with the "Checkouts" button highlighted 3. Click different category buttons (Fines, Renewals, Holds, etc.) and verify only relevant columns are shown 4. Verify the active button is highlighted in blue 5. Click "Show all" to display all columns 6. Verify the layout works with different system preference combinations (enable/disable ArticleRequests, UseRecalls, UnseenRenewals) 7. Verify column visibility persists correctly when system preferences are changed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Michaela Sieber <michaela.sieber@kit.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |clemens.tubach@kit.edu, | |michaela.sieber@kit.edu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=33312 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com, | |george@nekls.org --- Comment #5 from David Nind <david@davidnind.com> --- Just to say, I like it! I've added George Williams to the bug - I recall seeing something similar done with JQuery on TTEOTV or koha-US special interest group videos over the last few years. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 OpenFifth Sandboxes <sandboxes@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190459|0 |1 is obsolete| | --- Comment #6 from OpenFifth Sandboxes <sandboxes@openfifth.co.uk> --- Created attachment 190546 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190546&action=edit Bug 41439: Add column visibility toggles to circulation rules table The circulation rules table has become excessively wide with 40+ columns, making it difficult to navigate and use. This patch adds column visibility toggle buttons that allow users to focus on specific rule categories. Changes: - Add data-colname attributes to all table headers and footers for robust column identification - Add toggle buttons for: Checkouts, Due dates, Fines, Renewals, Holds, Article requests, Recalls, and Show all - Group columns into logical categories based on their function - Use column name selectors instead of fragile index-based selection for maintainability - Default to showing only Checkout columns on page load for a cleaner initial view - Highlight the active button to indicate current view - Disable search functionality (not appropriate for this table) - Handle dynamic column visibility based on system preferences (UnseenRenewals, ArticleRequests, UseRecalls) Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Observe the table now shows only checkout-related columns by default with the "Checkouts" button highlighted 3. Click different category buttons (Fines, Renewals, Holds, etc.) and verify only relevant columns are shown 4. Verify the active button is highlighted in blue 5. Click "Show all" to display all columns 6. Verify the layout works with different system preference combinations (enable/disable ArticleRequests, UseRecalls, UnseenRenewals) 7. Verify column visibility persists correctly when system preferences are changed Signed-off-by: georgewilliams <george@nekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 George Williams (NEKLS) <george@nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #7 from George Williams (NEKLS) <george@nekls.org> --- Outstanding! Thanks for this improvement. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #8 from Owen Leonard <oleonard@myacpl.org> --- In my test I couldn't submit edits in two different tabs. If I add a value under "checkouts" and then one under "holds" and submit, only the active tab's changes are saved. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #9 from George Williams (NEKLS) <george@nekls.org> --- Yes, I see what Owen is saying also. In my first test, I only tested adding rules, not modifying an existing rule. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 carthur@slolibrary.org <carthur@slolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |carthur@slolibrary.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #10 from carthur@slolibrary.org <carthur@slolibrary.org> --- +1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #11 from David Nind <david@davidnind.com> --- I quickly demoed this at the koha-US Customization Special Interest Group today. Everyone was very positive about it, including a +100! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #12 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 190646 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190646&action=edit Bug 41439: Fix form submission for hidden column inputs When using the column visibility toggles, only inputs in the currently visible columns were being submitted with the form. This meant that if a user edited a value in "Checkouts" view, then switched to "Holds" view to edit another value, only the holds values would be saved. This patch adds a form submit handler that temporarily shows all columns before form submission, ensuring all input values are included regardless of which view is currently active. Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Click "Checkouts" button 3. Enter a value in "Current checkouts allowed" field 4. Click "Holds" button 5. Enter a value in "Holds allowed (total)" field 6. Click Save 7. Verify both values are saved (previously only the holds value would be saved) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #13 from David Nind <david@davidnind.com> --- Wasn't sure if this as ready for re-testing, but I had a go. It is still not correctly saving things - for non-visible groups it is resetting to the empty value: 1. Enable UseRecalls and ArticleRequests. 2. For the checkouts section, edit the "All All" rule, change current checkouts allowed from unlimited to 50, click save. 3. Click the holds button, and edit the "All All" rule, change holds allowed (total) from unlimited to 50, click save. 4. Repeat steps 2 and 3, except this time don't save each time, save once both changes are made. 5. Up until now, everything works as expected and things are all updated correctly. 6. Edit the holds rule by itself, change 50 to 25, and click save. 7. In the checkouts section, current checkouts allowed is changed from 50 to unlimited. Presumably because the changed value for the filed is empty, so it resets to unlimted. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Katie Bliss <kebliss@dmpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kebliss@dmpl.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Danielle M. <dmeininger591@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dmeininger591@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 mteal@greenvillelibrary.org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mteal@greenvillelibrary.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Ashley Hernandez <ahernandez@ckls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ahernandez@ckls.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Michaela Sieber <michaela.sieber@kit.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=15522 Keywords| |Hackfest CC| |laurence.rault@biblibre.com --- Comment #14 from Michaela Sieber <michaela.sieber@kit.edu> --- There was already Feedback in this Mattermost discussion: https://chat.koha-community.org/koha-community/pl/6m9r6p4ifjfjdcg94d8tn8nepw Topic for Hackfest 2026 in Marseille -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Donna <bwsdonna@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bwsdonna@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Kendall Purser <bcldteam.kendall@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bcldteam.kendall@gmail.com URL| |https://github.com/orwek/Ko | |ha-Bits/blob/main/IntranetU | |serJS.js#L92 --- Comment #15 from Kendall Purser <bcldteam.kendall@gmail.com> --- I made the following for our library using JavaScript/Jquery: https://github.com/orwek/Koha-Bits/blob/main/IntranetUserJS.js#L92 It holds on to information between tabs because I am only hiding the columns in the view, not rewriting the table each time. The commend underneath this snippet gives the column numbers so it is easy to customize or add more buttons. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190546|0 |1 is obsolete| | Attachment #190646|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #16 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 195953 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195953&action=edit Bug 41439: Add column visibility toggles to circulation rules table The circulation rules table has become excessively wide with 40+ columns, making it difficult to navigate and use. This patch adds column visibility toggle buttons that allow users to focus on specific rule categories. Changes: - Add data-colname attributes to all table headers and footers for robust column identification - Add toggle buttons for: Checkouts, Due dates, Fines, Renewals, Holds, Article requests, Recalls, and Show all - Group columns into logical categories based on their function - Use column name selectors instead of fragile index-based selection for maintainability - Default to showing only Checkout columns on page load for a cleaner initial view - Highlight the active button to indicate current view - Disable search functionality (not appropriate for this table) - Handle dynamic column visibility based on system preferences (UnseenRenewals, ArticleRequests, UseRecalls) Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Observe the table now shows only checkout-related columns by default with the "Checkouts" button highlighted 3. Click different category buttons (Fines, Renewals, Holds, etc.) and verify only relevant columns are shown 4. Verify the active button is highlighted in blue 5. Click "Show all" to display all columns 6. Verify the layout works with different system preference combinations (enable/disable ArticleRequests, UseRecalls, UnseenRenewals) 7. Verify column visibility persists correctly when system preferences are changed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #17 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 195954 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195954&action=edit Bug 41439: Fix form submission for hidden column inputs When using the column visibility toggles, only inputs in the currently visible columns were being submitted with the form. This meant that if a user edited a value in "Checkouts" view, then switched to "Holds" view to edit another value, only the holds values would be saved. This patch adds a form submit handler that temporarily shows all columns before form submission, ensuring all input values are included regardless of which view is currently active. Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Click "Checkouts" button 3. Enter a value in "Current checkouts allowed" field 4. Click "Holds" button 5. Enter a value in "Holds allowed (total)" field 6. Click Save 7. Verify both values are saved (previously only the holds value would be saved) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #18 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 195955 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195955&action=edit Bug 41439: Fix hidden column inputs resetting values on save When editing a rule while a column group is hidden, DataTables removes hidden <td> elements from the DOM. On form submit, restoring visibility with columns().visible(true) re-inserts those cells with empty inputs, causing the corresponding fields to be saved as empty (e.g. unlimited). Before making all columns visible, repopulate hidden column inputs from the highlighted display row using DataTables' cached cell nodes. Mirrors the same special-case logic used when clicking Edit (unlimited fields, hardduedate, no_auto_renewal_after_hard_limit, cap_fine_to_replacement_price). Also add CSP nonce to the <style> element to fix QA forbidden_patterns. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #19 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 195956 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195956&action=edit Owens tabs for consistency? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195953|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195954|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195955|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #20 from Emmanuel Bétemps <e.betemps@gmail.com> --- Created attachment 196017 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196017&action=edit Bug 41439: Add column visibility toggles to circulation rules table The circulation rules table has become excessively wide with 40+ columns, making it difficult to navigate and use. This patch adds column visibility toggle buttons that allow users to focus on specific rule categories. Changes: - Add data-colname attributes to all table headers and footers for robust column identification - Add toggle buttons for: Checkouts, Due dates, Fines, Renewals, Holds, Article requests, Recalls, and Show all - Group columns into logical categories based on their function - Use column name selectors instead of fragile index-based selection for maintainability - Default to showing only Checkout columns on page load for a cleaner initial view - Highlight the active button to indicate current view - Disable search functionality (not appropriate for this table) - Handle dynamic column visibility based on system preferences (UnseenRenewals, ArticleRequests, UseRecalls) Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Observe the table now shows only checkout-related columns by default with the "Checkouts" button highlighted 3. Click different category buttons (Fines, Renewals, Holds, etc.) and verify only relevant columns are shown 4. Verify the active button is highlighted in blue 5. Click "Show all" to display all columns 6. Verify the layout works with different system preference combinations (enable/disable ArticleRequests, UseRecalls, UnseenRenewals) 7. Verify column visibility persists correctly when system preferences are changed Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #21 from Emmanuel Bétemps <e.betemps@gmail.com> --- Created attachment 196018 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196018&action=edit Bug 41439: Fix form submission for hidden column inputs When using the column visibility toggles, only inputs in the currently visible columns were being submitted with the form. This meant that if a user edited a value in "Checkouts" view, then switched to "Holds" view to edit another value, only the holds values would be saved. This patch adds a form submit handler that temporarily shows all columns before form submission, ensuring all input values are included regardless of which view is currently active. Test plan: 1. Navigate to Administration > Circulation and fine rules 2. Click "Checkouts" button 3. Enter a value in "Current checkouts allowed" field 4. Click "Holds" button 5. Enter a value in "Holds allowed (total)" field 6. Click Save 7. Verify both values are saved (previously only the holds value would be saved) Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #22 from Emmanuel Bétemps <e.betemps@gmail.com> --- Created attachment 196019 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196019&action=edit Bug 41439: Fix hidden column inputs resetting values on save When editing a rule while a column group is hidden, DataTables removes hidden <td> elements from the DOM. On form submit, restoring visibility with columns().visible(true) re-inserts those cells with empty inputs, causing the corresponding fields to be saved as empty (e.g. unlimited). Before making all columns visible, repopulate hidden column inputs from the highlighted display row using DataTables' cached cell nodes. Mirrors the same special-case logic used when clicking Edit (unlimited fields, hardduedate, no_auto_renewal_after_hard_limit, cap_fine_to_replacement_price). Also add CSP nonce to the <style> element to fix QA forbidden_patterns. Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |e.betemps@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 Gamo <moise.gachiniard@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |moise.gachiniard@biblibre.c | |om --- Comment #23 from Gamo <moise.gachiniard@biblibre.com> --- Just tested it, with current latest version. Seems really great and would be way much easier for customers ! Still, I suggest the use of Owens tabs. It would be more user friendly -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #24 from Gamo <moise.gachiniard@biblibre.com> --- https://sandboxes.biblibre.eu/signoff/test ko but for the 3 current patch : Signed-off-by: Moïse Gachiniard <moise.gachiniard@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=39127 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #25 from David Cook <dcook@prosentient.com.au> --- I like the sound of this. Going to make a note to revisit this. Especially since it would allow us to move on from having the same conversation across multiple bug reports like bug 39127 and bug 15522 and I'm sure heaps of others. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #26 from David Cook <dcook@prosentient.com.au> --- I think the bug title could probably be phrased better, but I can't be fussed at this point. Hoping to look at this sometime this week... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #27 from David Cook <dcook@prosentient.com.au> --- I'm liking this overall, although I'm not 100% sure about the way the buttons display about the table. It looks a bit weird having two separate control DIVs with a gap between them. It also looks a bit weird having them all be right aligned I think. -- I reckon that a lot of this Javascript could actually be placed in a separate .js file. I think the mix of template toolkit and JS will be at odds with Jonathan's latest script tidy work. I think we should get into the habit of putting the styles into separate files as well. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41439 --- Comment #28 from Jonathan Druart <jonathan.druart@gmail.com> --- 1. th in thead should not be added to the DOM conditionally, it breaks the column visibility feature, see bug 41604. I can rebase on top of it if you want. 2. This morning I also worked on another DT bug (bug 42910), and the following line would be problematic: dtApi.columns().visible(true); "programmatic-change" would need to be set. 3. TT tags within script tags are no longer allowed 4. I would move CSS rules to a .css file I can implement 1 and 2 but we will have to wait. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org