[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 19:17:52 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 a.hoereth):
Replying to [comment:10 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
>
[...]
> 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.
>
I can't really see the difference in loading behavior. For me it reloads
all tickmarks without my patch as well: http://www.screenr.com/vAI7 (gets
interesting at 1:40+)
No matter which handle is dragged, all tickmarks are (visually) reloaded.
But: We do not clean up data, so backbone reuses already available diffs
while still visually reloading them (internal cid stays the same) - I
guess ;)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24056#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list