If you load the advanced cataloging editor with a record that has more than 25 lines not all the lines will load initially. Once you add focus to the CodeMirror instance those lines do load but they should all load intially. To recreate: 1. Have advanced editor turned on. 2. Load a record with many lines and see that not all the lines appear until you click on the visible ones.
Created attachment 150476 [details] [review] Bug 32253: Load all CodeMirror lines initially when using advanced editor To test: 1. Have advanced editor turned on. 2. Load a record with many lines and see that not all the lines appear until you click on the visible ones. 3. Apply patch and try again 4. All lines should load.
Created attachment 150483 [details] [review] Bug 32253: Load all CodeMirror lines initially when using advanced editor To test: 1. Have advanced editor turned on. 2. Load a record with many lines and see that not all the lines appear until you click on the visible ones. 3. Apply patch and try again 4. All lines should load. Signed-off-by: David Nind <david@davidnind.com>
Testing notes (using KTD): 1. Used biblionumber = 248 - this has more than 25 lines in the advanced editor.
Created attachment 150536 [details] [review] Bug 32253: Load all CodeMirror lines initially when using advanced editor To test: 1. Have advanced editor turned on. 2. Load a record with many lines and see that not all the lines appear until you click on the visible ones. 3. Apply patch and try again 4. All lines should load. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Small fix, great effect, thanks!
Pushed to master for 23.05. Nice work everyone, thanks!
Many hands makes light work, thankyou everyone! Pushed to 22.11.x for the next release
Backported to 22.05.x for upcoming 22.05.13