Closed Bug 1204238 Opened 9 years ago Closed 9 years ago

Migrate tabular report sorting from YUI to jQuery

Categories

(Bugzilla :: Reporting/Charting, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 6.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file)

Attached patch patch, v1Splinter Review
This patch imports the DataTables plugin for jQuery, see https://datatables.net/.
Attachment #8660324 - Flags: review?(gerv)
Comment on attachment 8660324 [details] [diff] [review]
patch, v1

Review of attachment 8660324 [details] [diff] [review]:
-----------------------------------------------------------------

I've found one problem - if you create a data table report and then resize the browser window, the headings along the top do not stay aligned with the columns they refer to. (Let me know if you need STR or a screenshot.) If that's a bug we have, we need to fix it. If it's a datatables bug, it needs to be reported, and once that's done, r=gerv.

Gerv
Attachment #8660324 - Flags: review?(gerv) → review+
(In reply to Gervase Markham [:gerv] from comment #1)
> I've found one problem - if you create a data table report and then resize
> the browser window, the headings along the top do not stay aligned with the
> columns they refer to.

Yes, I saw that too. This is not a bug on our side, AFAIK, but rather an upstream bug. Note that you don't resize your window very often, so the impact is very low. And if you reload the page, the UI is fine again.
Well, it must be reasonably common, as we both saw it :-) But OK.

Gerv
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
   ad1dd06..b3abb29  master -> master
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 760197
Blocks: 1232716
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: