Created attachment 179128 [details] bib edit screen in main In 24.11 and later, the bootstrap update seems to have put a box around subfield tags (see attached screenshot). These values have been editable for a long time, but putting boxes around them makes it really clear that this is possible. But manual editing of those subfield tags is not really the expected way to use this editor and opens one up to numerous cataloging errors. For this sort of unguided cataloging, we expect one to use the advanced editor. I suggest we make those boxes go away again to make this questionable feature less obvious. If we leave the boxes, we really need to fix bug 29253, in which one can use this subfield updating to violate framework constraints.
oops, put the wrong bug number in that comment. Bug 29253 is not relevant here, but bug 39292 is!
If we were going to intentionally show the border in just one of the two editors, I would have picked adding it in the authority editor, where I use the ability to edit subfield letters all the time. For bib records, if you do that much editing of existing subfield codes you really ought to spend an hour learning to use the advanced editor, where you can just type them directly. But I doubt it was intentional, since the lack of border comes from an awkwardly placed border:0; in a style attribute in the template, where it looks more like a mistake than the attempt to obscure a feature that it actually is.
Created attachment 179190 [details] [review] Bug 39293: Remove box around subfield tag in basic editor Prior to the Bootstrap 5 update, the feature of editing subfield codes in the basic editor was kept subtle by putting the code in an <input>, but removing the border around the input, so it required either knowing the feature exists or accidentally clicking on or around the single letter code and seeing it turn into an editable input. This patch removes the border again, because when people accidentally see the feature, they are sometimes not pleased to see it. Test plan: 1. Without the patch, open the cataloging basic editor and switch to the "1" tab with the 100 fields. 2. Note that the codes for the subfields are clearly in editable inputs. 3. Apply patch, reload the editor page. 4. Note that now the codes look like just plain text, until you actually click on or very near the letter, when they become visible as inputs by having the focus border. Sponsored-by: Chetco Community Public Library
Comment on attachment 179190 [details] [review] Bug 39293: Remove box around subfield tag in basic editor Gee, thanks tidy, completely removing addbiblio.tt sounds like a great idea.
Created attachment 179221 [details] [review] Bug 39293: Remove box around subfield tag in basic editor Prior to the Bootstrap 5 update, the feature of editing subfield codes in the basic editor was kept subtle by putting the code in an <input>, but removing the border around the input, so it required either knowing the feature exists or accidentally clicking on or around the single letter code and seeing it turn into an editable input. This patch removes the border again, because when people accidentally see the feature, they are sometimes not pleased to see it. Test plan: 1. Without the patch, open the cataloging basic editor and switch to the "1" tab with the 100 fields. 2. Note that the codes for the subfields are clearly in editable inputs. 3. Apply patch, reload the editor page. 4. Note that now the codes look like just plain text, until you actually click on or very near the letter, when they become visible as inputs by having the focus border. Sponsored-by: Chetco Community Public Library
Created attachment 179231 [details] [review] Bug 39293: Remove box around subfield tag in basic editor Prior to the Bootstrap 5 update, the feature of editing subfield codes in the basic editor was kept subtle by putting the code in an <input>, but removing the border around the input, so it required either knowing the feature exists or accidentally clicking on or around the single letter code and seeing it turn into an editable input. This patch removes the border again, because when people accidentally see the feature, they are sometimes not pleased to see it. Test plan: 1. Without the patch, open the cataloging basic editor and switch to the "1" tab with the 100 fields. 2. Note that the codes for the subfields are clearly in editable inputs. 3. Apply patch, reload the editor page. 4. Note that now the codes look like just plain text, until you actually click on or very near the letter, when they become visible as inputs by having the focus border. Sponsored-by: Chetco Community Public Library Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Created attachment 179500 [details] [review] Bug 39293: Remove box around subfield tag in basic editor Prior to the Bootstrap 5 update, the feature of editing subfield codes in the basic editor was kept subtle by putting the code in an <input>, but removing the border around the input, so it required either knowing the feature exists or accidentally clicking on or around the single letter code and seeing it turn into an editable input. This patch removes the border again, because when people accidentally see the feature, they are sometimes not pleased to see it. Test plan: 1. Without the patch, open the cataloging basic editor and switch to the "1" tab with the 100 fields. 2. Note that the codes for the subfields are clearly in editable inputs. 3. Apply patch, reload the editor page. 4. Note that now the codes look like just plain text, until you actually click on or very near the letter, when they become visible as inputs by having the focus border. Sponsored-by: Chetco Community Public Library Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> Signed-off-by: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
Pushed for 25.05! Well done everyone, thank you!