Closed
Bug 591569
Opened 15 years ago
Closed 15 years ago
Reduce size of themes page
Categories
(Input :: General, defect, P1)
Input
General
Tracking
(Not tracked)
VERIFIED
FIXED
1.8
People
(Reporter: morgamic, Assigned: davedash)
References
()
Details
Attachments
(1 file)
|
802.77 KB,
image/png
|
Details |
Screenshot, comparing performance
The themes page doesn't have pagination and spits out a ridiculous amount of information. The data set should be smaller (10-20) with pages much like other parts of the site where you can scroll ahead/back in time.
The page is also slow as a result of the sheer size of it:
http://www.webpagetest.org/result/100828_3QGV/1/details/
Just loading the text/html takes 10 seconds for a new client.
|
||
Comment 1•15 years ago
|
||
Oops ;) Dave, will you take a look? Just copy the pagination code from any of the other views.
Assignee: nobody → dd
|
Assignee | |
Comment 2•15 years ago
|
||
(In reply to comment #0)
> The themes page doesn't have pagination and spits out a ridiculous amount of
> information. The data set should be smaller (10-20) with pages much like other
> parts of the site where you can scroll ahead/back in time.
>
> The page is also slow as a result of the sheer size of it:
> http://www.webpagetest.org/result/100828_3QGV/1/details/
>
> Just loading the text/html takes 10 seconds for a new client.
/me didn't expect to have more than 20 items to begin with.
Good catch.
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 3•15 years ago
|
||
r? fwenzel
http://github.com/fwenzel/reporter/pull/2
Target Milestone: --- → 1.8
|
||
Updated•15 years ago
|
Priority: -- → P1
|
|
||
Updated•15 years ago
|
Whiteboard: [r?fwenzel]
|
|
Assignee | |
Comment 4•15 years ago
|
||
This is committed, but we can't look at it unless we have data in staging...
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Comment 5•15 years ago
|
||
Verified FIXED: http://www.webpagetest.org/result/100914_4YZK/1/details/
Status: RESOLVED → VERIFIED
|
|
||
Comment 6•15 years ago
|
||
|
Reporter | |
Comment 8•15 years ago
|
||
Was this pushed out?
|
|
||
Comment 9•15 years ago
|
||
Not yet, it's part of a Thursday night push for the general 1.8 milestone.
|
|
Reporter | |
Comment 10•15 years ago
|
||
Awesome!
|
||
Updated•14 years ago
|
Component: Input → General
Product: Webtools → Input
You need to log in
before you can comment on or make changes to this bug.
Description
•