Continuing the work of Bug 32571, the biblio and authority merge pages should be updated to use the new WRAPPER for tabs.
Created attachment 148016 [details] [review] Bug 33181: Use template wrapper for tabs on record merge pages This patch updates the include file which is used by both the bibliographic and authority recrd merge pages. The markup is updated to use the new tab WRAPPER directives. To test, apply the patch and perform a catalog search which will return more than one record. - Select two or more records and choose "Merge records" from the edit menu at the top of the search results. - On the following page you should see one tab for each record you selected from the search results. - Clicking each tab should show you the correct bibliographic details. Perform the same tests from the authority search results page.
Owen, can this one be rebased?
Created attachment 148488 [details] [review] Bug 33181: Use template wrapper for tabs on record merge pages This patch updates the include file which is used by both the bibliographic and authority recrd merge pages. The markup is updated to use the new tab WRAPPER directives. To test, apply the patch and perform a catalog search which will return more than one record. - Select two or more records and choose "Merge records" from the edit menu at the top of the search results. - On the following page you should see one tab for each record you selected from the search results. - Clicking each tab should show you the correct bibliographic details. Perform the same tests from the authority search results page.
Created attachment 148912 [details] [review] Bug 33181: Use template wrapper for tabs on record merge pages This patch updates the include file which is used by both the bibliographic and authority recrd merge pages. The markup is updated to use the new tab WRAPPER directives. To test, apply the patch and perform a catalog search which will return more than one record. - Select two or more records and choose "Merge records" from the edit menu at the top of the search results. - On the following page you should see one tab for each record you selected from the search results. - Clicking each tab should show you the correct bibliographic details. Perform the same tests from the authority search results page. Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com>
Created attachment 149324 [details] [review] Bug 33181: Use template wrapper for tabs on record merge pages This patch updates the include file which is used by both the bibliographic and authority recrd merge pages. The markup is updated to use the new tab WRAPPER directives. To test, apply the patch and perform a catalog search which will return more than one record. - Select two or more records and choose "Merge records" from the edit menu at the top of the search results. - On the following page you should see one tab for each record you selected from the search results. - Clicking each tab should show you the correct bibliographic details. Perform the same tests from the authority search results page. Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to stable for 22.11.x