While testing bug 33909, I had to look at all the pages in the administration module. In doing so, I found some inconsistencies between the various pages. For the Additional fields page, here is what should be changed to make it consistent with the other pages in the module: Page title - Change 'Manage additional fields' to 'Additional fields' - Add level by table (like breadcrumbs) - Change 'Add additional field' to "New additional field for 'table'" Breadcrumb - Change 'Manage additional fields' to 'Additional fields' - Change "Modify fields for 'table'" to "Additional fields for 'table'" - Change 'Add field' to "New additional field for 'table'" - Change "Modify field 'name'" to "Modify additional field 'name'" Page header - Change 'Add field' to "New additional field for 'table'" - Change 'Modify field' to "Modify additional field 'name'" Change 'Create field' button to 'New field'. No special page to confirm deletion, only a browser prompt.
Created attachment 155319 [details] [review] Bug 34411: Inconsistencies in Additional fields page titles, breadcrumbs, and header This patch makes changes to the additional fields administration template in order to make it more consistent with other pages in Koha. To test, apply the patch and go to Administration -> Additional fields. View each variation of the page to confirm that breadcrumb navigation, page title, and page headings are consistent with each other. - Main list - Table view (e.g. aqbasket, aqorders, etc) - New field - Edit field
Created attachment 155491 [details] [review] Bug 34411: Inconsistencies in Additional fields page titles, breadcrumbs, and header This patch makes changes to the additional fields administration template in order to make it more consistent with other pages in Koha. To test, apply the patch and go to Administration -> Additional fields. View each variation of the page to confirm that breadcrumb navigation, page title, and page headings are consistent with each other. - Main list - Table view (e.g. aqbasket, aqorders, etc) - New field - Edit field Signed-off-by: Salah Ghedda <salah.ghedda@inLibro.com>
Created attachment 155492 [details] [review] Bug 34411: Inconsistencies in Additional fields page titles, breadcrumbs, and header This patch makes changes to the additional fields administration template in order to make it more consistent with other pages in Koha. To test, apply the patch and go to Administration -> Additional fields. View each variation of the page to confirm that breadcrumb navigation, page title, and page headings are consistent with each other. - Main list - Table view (e.g. aqbasket, aqorders, etc) - New field - Edit field Signed-off-by: Salah Ghedda <salah.ghedda@inLibro.com> Signed Off
Created attachment 155745 [details] [review] Bug 34411: Inconsistencies in Additional fields page titles, breadcrumbs, and header This patch makes changes to the additional fields administration template in order to make it more consistent with other pages in Koha. To test, apply the patch and go to Administration -> Additional fields. View each variation of the page to confirm that breadcrumb navigation, page title, and page headings are consistent with each other. - Main list - Table view (e.g. aqbasket, aqorders, etc) - New field - Edit field Signed-off-by: Salah Ghedda <salah.ghedda@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 23.11. Nice work everyone, thanks!
Like others, not pushed to 23.05.x