Owen Leonard
0d88109018
This follow-up moves the checkboxes into a separate column from the report ID in order to prevent visual inconsistencies. To test, repeat the test plan from the original patch: You must have two or more saved reports to delete. Deletion should work properly when: - Selecting one report for deletion by checking the box. - Selecting more than one report for deletion by checking boxes. - Clicking the old "Delete" link Clicking the delete button should prompt you to confirm. Clicking cancel should cancel. Clicking the delete button when no boxes are checked should trigger an alert asking you to select reports for deletion. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> This works really nicely, passes all tests and the QA script. There is one little thing to note: If you delete a single report using the option from the menu, the line you want to delete is highlighted in red. It would be nice to do that also for deleting multiple reports. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Galen Charlton <gmc@esilibrary.com> |
||
|---|---|---|
| .. | ||
| acquisitions_stats.tt | ||
| bor_issues_top.tt | ||
| borrowers_out.tt | ||
| borrowers_stats.tt | ||
| cat_issues_top.tt | ||
| catalogue_out.tt | ||
| catalogue_stats.tt | ||
| dictionary.tt | ||
| guided_reports_start.tt | ||
| issues_avg_stats.tt | ||
| issues_by_borrower_category.tt | ||
| issues_stats.tt | ||
| itemslost.tt | ||
| itemtypes.tt | ||
| reports-home.tt | ||
| reservereport.tt | ||
| reserves_stats.tt | ||
| serials_stats.tt | ||
| stats_screen.tt | ||