When creating a new page with the additional content page, there is a possibility of adding an 'appears in position' to the page. I think this is a leftover from the news/html customizations tool where you could have many entries in the same location and they would be ordered by 'appears in position'. However, I don't think it is useful in the case of pages and should probably be removed.
Created attachment 146656 [details] [review] Bug 32021: Only show 'appear in position' for News The 'Appear in position' option isn't relevant for HTML customisations or Pages. The option is only used by the News tool, so we should hide it when not adding or modifying a News item. To test: 1) Add a new HTML customisation or Page 2) Confirm the 'Appear in position' option does not show 3) Add a new News item 4) Confirm the 'Appear in position' option DOES show Sponsored-by: Catalyst IT
Hi Aleisha, We can have more than one HTML customization per display location. I just tried it without the patch and the 'appear in position' do change the position. 1. Go to Tools > HTML customizations 2. Click 'New entry' 3. Fill out the top form - Display location: OpacNavRight - Library: All libraries - Publication date: set to today - Expiration date: leave empty - Appear in position: 1 4. Enter text in the title and content for the default language - Title: OpacNavRight 1 - Content: OpacNavRight 1 5. Click 'Save' 6. Redo steps 2 to 5, entering the following information for the second entry - Display location: OpacNavRight - Library: All libraries - Publication date: set to today - Expiration date: leave empty - Appear in position: 2 - Title: OpacNavRight 2 - Content: OpacNavRight 2 7. Go to the OPAC --> On the right, there should be OpacNavRight 1 followed by OpacNavRight 2 8. Edit the OpacNavRight 1 entry, change the Appear in position for 2 and save 9. Edit the OpacNavRight 2 entry, change the Appear in position for 1 and save 10. Go to the OPAC --> On the right, there should be OpacNavRight 2 followed by OpacNavRight 1 So this bug is really only for pages. Thanks! Caroline
Created attachment 146704 [details] [review] Bug 32021: Don't show 'appear in position' for Pages The 'Appear in position' option isn't relevant for Pages, so we should hide it when not adding or modifying a Page. To test: 1) Add a new Page 2) Confirm the 'Appear in position' option does not show 3) Add a new News item or HTML customisation 4) Confirm the 'Appear in position' option DOES show Sponsored-by: Catalyst IT
Created attachment 146734 [details] [review] Bug 32021: Don't show 'appear in position' for Pages The 'Appear in position' option isn't relevant for Pages, so we should hide it when not adding or modifying a Page. To test: 1) Add a new Page 2) Confirm the 'Appear in position' option does not show 3) Add a new News item or HTML customisation 4) Confirm the 'Appear in position' option DOES show Sponsored-by: Catalyst IT Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Thanks Aleisha!
For release maintainers, please mark as Needs documenting once you're done. I'd like to correct the manual
Created attachment 146903 [details] [review] Bug 32021: Don't show 'appear in position' for Pages The 'Appear in position' option isn't relevant for Pages, so we should hide it when not adding or modifying a Page. To test: 1) Add a new Page 2) Confirm the 'Appear in position' option does not show 3) Add a new News item or HTML customisation 4) Confirm the 'Appear in position' option DOES show Sponsored-by: Catalyst IT Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 23.05. Nice work everyone, thanks!
Enhancement - not backporting to 22.11.x Nice work everyone!