Clicking the plus sign next to a repeatable subfield no longer clones the subfield. Also, clicking the minus sign will not remove the subfield initially, but if you click the plus sign, then click the minus sign, it will. Owen & wizzyrea were able to confirm the first half of that, at least. The plus & minus functions worked as recently as 3.01.00.061.
There are a few changes to addbiblio.tmpl since 1/2010 that might be related.
When I tried it in Chrome in developer mode, I got this error:
Uncaught TypeError: Cannot call method 'getAttribute' of undefined
Owen's patch for bug 4868 fixes this.
On further testing, the patch for 4868, makes it better, but there's still some odd behavior. You can clone the subfield, but sometimes after clicking delete it won't let you clone the subfield again. Also the delete won't always delete the subfield; it only seems to work once for me, so if you clone the subfield twice, and then try to delete them, it will only delete one but not both.
Created attachment 2389 [details] [review]
add existence test on link after subfield
When the clone/unclone plus/minus were replaced with images the <span> tags were inadvertantly deleted causing the js to loose its ability to identify the clone/unclone links and properly udpate the indexes.
The resulting incorrect indexes caused various weirdnesses to result when one attempted to clone/unclone subfields.
The solution is to add classes to the two links and update the js to identify the links based on class.
Thanks to Owen for his help in chasing this down.
The patch is available here:
Be sure to pick up the associated tidy patch before it as well.
The supplied patch:
does not work for IE 7. When clicking on clone-subfield.png or delete-subfield.png the MARC editor site jumps to the bottom, but fields don't collapse/uncollapse.
Changing status back to assigned
(In reply to comment #6)
> The supplied patch:
> does not work for IE 7. When clicking on clone-subfield.png or
> delete-subfield.png the MARC editor site jumps to the bottom, but fields don't
Testing in IE 7.0.5730.13 shows that deleting subfields works as expected, but cloning subfields does not.
This is probably due to some quark with IE.
Since we basically state that FF is the browser supported in the staff client, I'm not sure how much of a priority this will be. However, I'm adding Owen to the cc list in case the fix is something obvious.
This is already in master