[wp-trac] [WordPress Trac] #24056: Revisions: UI a bit unusable when you have ALOT of revisions
WordPress Trac
noreply at wordpress.org
Wed May 15 18:55:59 UTC 2013
#24056: Revisions: UI a bit unusable when you have ALOT of revisions
-----------------------------+--------------------
Reporter: lancewillett | Owner:
Type: defect (bug) | Status: new
Priority: highest omg bbq | Milestone: 3.6
Component: Revisions | Version: trunk
Severity: blocker | Resolution:
Keywords: needs-patch |
-----------------------------+--------------------
Comment (by adamsilverstein):
Replying to [comment:9 a.hoereth]:
> Replying to [comment:8 adamsilverstein]:
> > Replying to [comment:7 ocean90]:
> > > This definitely needs to be done. It's currently unusable for > 50
revisions. a.hoereth is working on a new patch.
> >
> > why is it unusable, because of time for page to load? prioritizing the
load for the selected revision would be very helpful, still not sure how
paging will work.
> demo for
[https://core.trac.wordpress.org/attachment/ticket/24056/24056-prioritize.patch
24056-prioritize.patch]: http://www.screenr.com/gpI7
very nice work, this will really help with a large number of revisions...
i like your approach to the load prioritizing. the screencast show its
working perfectly in one handle mode.
in two handle mode it is very close, still slightly off: when the left
handle stops, only the comparisons for dragging the right handle are
reloaded - the left handle comparisons haven't changed because the right
handle hadn't moved, so they aren't reloaded.
therefore, i would expect ticks around the right handle position to load
when stopping the left handle and conversely the ticks around the left
handle to load when the right handle stops; in your patch, comparisons
around _both_ handles are loaded, it should be the left or right depending
on which handle just stopped.
hope that makes sense!
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24056#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list