I noticed a couple of issues with the Bootstrap OPAC lists interface: - The old breadcrumbs from the prog theme are still present on the List edit screen. - New/Edit/Delete controls are missing from the list view if the list is empty.
Created attachment 25138 [details] [review] Bug 11709 - Bootstrap OPAC lists interface requires some revisions This patch makes some revisions to the lists template: - Removed the old prog theme breadcrumbs from the List edit screen. - Added New/Edit/Delete controls to the list view in cases where the list is empty. - Corrected some breadcrumb logic. To test, apply the patch and edit an existing list. You should see a heading "Editing [list name]". If necessary create a list with zero items on it. View the list. You should see New/Edit/Delete links above the message about the list being empty.
Created attachment 25353 [details] [review] [Signed-off] Bug 11709 - Bootstrap OPAC lists interface requires some revisions This patch makes some revisions to the lists template: - Removed the old prog theme breadcrumbs from the List edit screen. - Added New/Edit/Delete controls to the list view in cases where the list is empty. - Corrected some breadcrumb logic. To test, apply the patch and edit an existing list. You should see a heading "Editing [list name]". If necessary create a list with zero items on it. View the list. You should see New/Edit/Delete links above the message about the list being empty. Patch behaves as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 25373 [details] [review] Bug 11709 - Bootstrap OPAC lists interface requires some revisions This patch makes some revisions to the lists template: - Removed the old prog theme breadcrumbs from the List edit screen. - Added New/Edit/Delete controls to the list view in cases where the list is empty. - Corrected some breadcrumb logic. To test, apply the patch and edit an existing list. You should see a heading "Editing [list name]". If necessary create a list with zero items on it. View the list. You should see New/Edit/Delete links above the message about the list being empty. Patch behaves as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Pushed to master. Thanks, Owen!
Pushed to 3.14.x, will be in 3.14.08