https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #1 from David Nind <david@davidnind.com> --- Can confirm that this also occurs on master (using Firefox, Google Chrome and Chromium on Ubuntu 18.04). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|20.11 |master Assignee|koha-bugs@lists.koha-commun |oleonard@myacpl.org |ity.org | --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- (In reply to Jonathan Druart from comment #1 on Bug 27838) function SetSubfieldNumber(subfield_name, nb) {

And I bet the culprit is: 52 var field_start = whichfield.parentNode.parentNode.parentNode.parentNode;

...which is exactly what makes that kind of selector so fragile!! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 117544 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117544&action=edit Bug 27737: Tag editor for authority lookup broken in authority editor This patch corrects the selector used for targeting the correct fields during an authority linking operation. To test, apply the patch and go to Authorities in the staff client. - Create or edit an authority record. - Switch to tab 5 and if necessary click one of the tag names (e.g. "SEE ALSO FROM TRACING--PERSONAL NAME") to expand the subfields. - Click the "..." plugin link next to subfield a ("Personal name" in this example). - In the popup window, search for an authority record. - Click "Choose" next to one of the results. - The popup window should close and the authority editor should contain data from the record you chose, including a value in $9 and $a. - Test with various records to confirm that data in other fields is copied correctly, for instanct $d, dates associated with a name, $t title of a work. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #117544|0 |1 is obsolete| | --- Comment #5 from David Nind <david@davidnind.com> --- Created attachment 117545 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117545&action=edit Bug 27737: Tag editor for authority lookup broken in authority editor This patch corrects the selector used for targeting the correct fields during an authority linking operation. To test, apply the patch and go to Authorities in the staff client. - Create or edit an authority record. - Switch to tab 5 and if necessary click one of the tag names (e.g. "SEE ALSO FROM TRACING--PERSONAL NAME") to expand the subfields. - Click the "..." plugin link next to subfield a ("Personal name" in this example). - In the popup window, search for an authority record. - Click "Choose" next to one of the results. - The popup window should close and the authority editor should contain data from the record you chose, including a value in $9 and $a. - Test with various records to confirm that data in other fields is copied correctly, for instanct $d, dates associated with a name, $t title of a work. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_20_11_candidate -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #7 from Primo Kohák <primokohak@seznam.cz> --- I confirm the bug for Koha 20.11 on Ubuntu 20.04.1. For us, it is a critical error, almost a game stopper. I tried to reflect it by my vote above. We are thinking about downgrading to Koha 20.05. We are not sure, how difficult it is. We would be grateful for an advice or an opinion. The question is: What to expect about the time needed for the "trivial patch" expected above. We may be able to provide some small money for the solution. What are the other ways to support the resolution of this bug? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|major |critical Patch complexity|Trivial patch |Small patch --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Upping severity. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #11 from Primo Kohák <primokohak@seznam.cz> --- Created attachment 117862 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117862&action=edit After workaround suggested by Hugo -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #13 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Primo Kohák from comment #7)

I confirm the bug for Koha 20.11 on Ubuntu 20.04.1. For us, it is a critical error, almost a game stopper. I tried to reflect it by my vote above. We are thinking about downgrading to Koha 20.05. We are not sure, how difficult it is. We would be grateful for an advice or an opinion. The question is: What to expect about the time needed for the "trivial patch" expected above. We may be able to provide some small money for the solution. What are the other ways to support the resolution of this bug?

This is likely to get into the next maintenance release - I would strongly recommend against trying a downgrade. There is no process for this and you'd have to manually reverse database changes etc. - dangerous. Easier to use the patch from here and maybe implement that in your installation. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #117545|0 |1 is obsolete| | --- Comment #14 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 117872 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117872&action=edit Bug 27737: Tag editor for authority lookup broken in authority editor This patch changes the markup structure for the authorities editor so that it better matches that of the basic bibliographic record editor. This allows the authority-linking JavaScript to correctly target fields on both pages. To test, apply the patch and go to Authorities in the staff client. - Create or edit an authority record. - Switch to tab 5 and if necessary click one of the tag names (e.g. "SEE ALSO FROM TRACING--PERSONAL NAME") to expand the subfields. - Click the "..." plugin link next to subfield a ("Personal name" in this example). - In the popup window, search for an authority record. - Click "Choose" next to one of the results. - The popup window should close and the authority editor should contain data from the record you chose, including a value in $9 and $a. - Test with various records to confirm that data in other fields is copied correctly, for instanct $d, dates associated with a name, $t title of a work. - Test other JavaScript-driven interactions in the editor: - Duplicate repeatable tag - Remove repeatable tag - Empty tag - Reorder sortable tags (e.g. multiple 500 tags) - Reorder sortable subfields - Duplicate repeatable subfield - Remove repeatable subfield - Empty repeatable subfield -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #16 from Primo Kohák <primokohak@seznam.cz> --- (In reply to Katrin Fischer from comment #13)

I would strongly recommend against trying a downgrade. Easier to use the patch from here and maybe implement that in your installation.

Thank you for your advice. We shall follow it. (In reply to Owen Leonard from comment #14)

Created attachment 117872

Thank you for the patch. (In reply to Phil Ringnalda from comment #15)

That restores all the functionality, but the CSS changes drop the specificity of the rules that were #cat_addbiblio .whatever, so you lose the flex and width: 100% on the inputs and wind up with 50 or 60em, depending on the min-width.

Not sure, what that means for us. Perhaps, I shall wait a little bit with the implementation of the patch. It will be my first time doing a patch anyway. I will start with the man page (https://man.archlinux.org/man/patch.1). -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Katrin Fischer <katrin.fischer@bsz-bw.de> 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=27737 Phil Ringnalda <phil@chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #20 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Can we get a signoff on this one please? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #22 from Peter Vashchuk <stalkernoid@gmail.com> --- It seems like the problem of input fields overlaying the field descriptions is still here, see the PNG attachment: https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118115 Though the copy/delete icons are displayed more correctly now. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #117884|0 |1 is obsolete| | --- Comment #23 from David Nind <david@davidnind.com> --- Created attachment 118116 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118116&action=edit Bug 27737: Tag editor for authority lookup broken in authority editor This patch changes the markup structure for the authorities editor so that it better matches that of the basic bibliographic record editor. This allows the authority-linking JavaScript to correctly target fields on both pages. To test, apply the patch and go to Authorities in the staff client. - Create or edit an authority record. - Switch to tab 5 and if necessary click one of the tag names (e.g. "SEE ALSO FROM TRACING--PERSONAL NAME") to expand the subfields. - Click the "..." plugin link next to subfield a ("Personal name" in this example). - In the popup window, search for an authority record. - Click "Choose" next to one of the results. - The popup window should close and the authority editor should contain data from the record you chose, including a value in $9 and $a. - Test with various records to confirm that data in other fields is copied correctly, for instanct $d, dates associated with a name, $t title of a work. - Test other JavaScript-driven interactions in the editor: - Duplicate repeatable tag - Remove repeatable tag - Empty tag - Reorder sortable tags (e.g. multiple 500 tags) - Reorder sortable subfields - Duplicate repeatable subfield - Remove repeatable subfield - Empty repeatable subfield Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #25 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Peter Vashchuk from comment #22)

It seems like the problem of input fields overlaying the field descriptions is still here, see the PNG attachment: https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118115

Though the copy/delete icons are displayed more correctly now.

What I am seeing on master+those 2 patches: https://snipboard.io/sjlpMQ.jpg Looks nice. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 --- Comment #27 from Phil Ringnalda <phil@chetcolibrary.org> --- No idea how to produce the result in Peter's screenshot, unless it's cached HTML and uncached CSS. That appearance says that #authoritytabs .subfield_line and #authoritytabs .field_marceditor apply, but #authoritytabs .subfield label does not, implying that the <label> isn't a descendant of something class="subfield", which it used to not be, but is with the first patch here. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Peter Vashchuk <stalkernoid@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #118174|0 |1 is obsolete| | --- Comment #29 from Peter Vashchuk <stalkernoid@gmail.com> --- Created attachment 118179 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118179&action=edit Bug 27737: (follow-up) Minimal CSS changes to avoid regressions To keep the specificity of the rules we want to have apply to both the biblio editor and also the authorities editor, we need to add the ID for the authorities editor, rather than remove the ID for the biblio editor. Also, the removal of the selectors for #authoritiestabs descendents was regressing the display of authority record details, which is where that plural authoritiestabs element lives. Test plan: 1. With neither patch from this bug applied, Cataloging - New record - Default framework. 2. Shrink and widen your browser window, noticing how the input fields flex to remain the full width, keeping the controls to duplicate and delete a subfield out directly to the right of the input. 3. Authorities - search for Perl (or any term that will match a record), click the linked authority term to open the Authority detail display 4. Note how the field number, field description, and subfield descriptions are all different colors, and subfield codes are on the same line as the subfield contents. 5. Apply both patches from this bug. 6. Cataloging - New record - Default framework 7. Verify that the input fields still behave the same as you widen and shrink your browser window 8. Authorities - New authority - Chronological Term 9. Verify that the input fields now behave like the ones in the biblio editor, shrinking and widening to fill the width of the page, keeping the controls to duplicate and delete subfields immediately to the right of the field 10. Authorities - search for Perl, click the linked authority term to open the Authority detail display 11. Verify that the tag number, field description and subfield descriptions are still properly colored and subfield codes are still on the same line as the subfield contents. Signed-off-by: David Nind <david@davidnind.com> Trivial fixup for a copy-paste failure, missed the y on subfield_loop_mandator Signed-off-by: Petro Vashchuk <stalkernoid@gmail.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #118116|0 |1 is obsolete| | --- Comment #30 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 118187 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118187&action=edit Bug 27737: Tag editor for authority lookup broken in authority editor This patch changes the markup structure for the authorities editor so that it better matches that of the basic bibliographic record editor. This allows the authority-linking JavaScript to correctly target fields on both pages. To test, apply the patch and go to Authorities in the staff client. - Create or edit an authority record. - Switch to tab 5 and if necessary click one of the tag names (e.g. "SEE ALSO FROM TRACING--PERSONAL NAME") to expand the subfields. - Click the "..." plugin link next to subfield a ("Personal name" in this example). - In the popup window, search for an authority record. - Click "Choose" next to one of the results. - The popup window should close and the authority editor should contain data from the record you chose, including a value in $9 and $a. - Test with various records to confirm that data in other fields is copied correctly, for instanct $d, dates associated with a name, $t title of a work. - Test other JavaScript-driven interactions in the editor: - Duplicate repeatable tag - Remove repeatable tag - Empty tag - Reorder sortable tags (e.g. multiple 500 tags) - Reorder sortable subfields - Duplicate repeatable subfield - Remove repeatable subfield - Empty repeatable subfield Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> https://bugs.koha-community.org/show_bug.cgi?id=22737 Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |21.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m Status|Pushed to master |Pushed to stable Version(s)|21.05.00 |21.05.00,20.11.04 released in| | --- Comment #33 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 20.11.x for 20.11.04 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27737 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patch changes the release notes| |markup structure for the | |authorities editor so that | |it better matches that of | |the basic bibliographic | |record editor. This allows | |the authority-linking | |JavaScript to correctly | |target fields on both | |pages. -- You are receiving this mail because: You are watching all bug changes.