MARC Batch Editor summary table html problems
Bug #1545226 reported by
Josh Stompro
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned |
Bug Description
EG 2.8.4
The table definition for the summary of results has an incorrect html table definition, which makes the output look strange.
For the first row, <th> should be <tr> and the <td>'s should be <th>'s.
<table style="width:100%; margin-top:100px;">
<th>
</th>
<tr>
<td id='success_
Josh
Changed in evergreen: | |
assignee: | nobody → Jennifer Pringle (jpringle-u) |
Changed in evergreen: | |
assignee: | Jennifer Pringle (jpringle-u) → Galen Charlton (gmc) |
Changed in evergreen: | |
assignee: | Galen Charlton (gmc) → nobody |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Working Branch at: user/stompro/ lp1545226_ marc_batch_ editor_ status git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ stompro/ lp1545226_ marc_batch_ editor_ status
http://
I found a few more issues once I started looking into it.
- Fixed table html
- Changed table so it isn't trying to fill the whole screen.
- Added borders and other styling.
- Added color for failed records (red) and for completion (green).
- Added a row with total number of records.
- Changed progress bar so it shows the progress. It was plumbed to show the progress, but then marked as indeterminate, so no progress was shown.
To test.
1. On unmodified system use the marc batch editor to update some records. Make note of how the results screen looks.
2. After fix is applied try the same thing and see what the results screen looks like.
Josh