#12811 closed enhancement (fixed)
BatchModify support for table footer
| Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.2.2 |
| Component: | ticket system | Version: | |
| Severity: | normal | Keywords: | batch-modify |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: |
Query result headings have the class |
||
| Internal Changes: | |||
Description
SumFields adds a row at the end of each query results table. Reported in th:#13180, the row has a checkbox added to it, which when selected results in an exception for a batch modify request.
I'm changing SumFields to instead add the row in a tfoot. With some minor changes to batch modify a select box won't be added to a tfoot row.
Attachments (1)
Change History (6)
by , 7 years ago
| Attachment: | Screen Shot 2017-05-17 at 12.01.55.png added |
|---|
comment:1 by , 7 years ago
comment:2 by , 7 years ago
| API Changes: | modified (diff) |
|---|
comment:3 by , 7 years ago
The proposed changes don't work for the case of grouped results. It seems only one tfoot is allowed per table, and it must come before the tbody elements. Instead, I propose to just classify the tbody elements as either headers or results: [37481a733/rjollos.git].
comment:4 by , 7 years ago
| API Changes: | modified (diff) |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Committed to 1.2-stable in [15919,15926], merged to trunk in [15920,15927].
comment:5 by , 7 years ago
| API Changes: | modified (diff) |
|---|
Proposed changes in [48594d2eb/rjollos.git].