Code review comment for lp:~jcsackett/juju-gui/scroll-to-top-on-change

Revision history for this message
j.c.sackett (jcsackett) wrote :

*** Submitted:

Resolves views rendering in partly scrolled state

This branch resolves two bugs wherein a view rendered scrolled partway
down.
This appears to be an issue with browser behavior rather than JS, as
investigation showed no problem when debuggers were used to check state
as the
views render.

To get around that, this branch simply grabs the top-most visible
element of
the two views and ensures they're scrolled into view.

R=rharding, jeff.pihach
CC=
https://codereview.appspot.com/9088043

https://codereview.appspot.com/9088043/

« Back to merge proposal