Yea, I think I was seeing the scroll shift slightly due to this.  I just didn't have time to go back and fix it today.  Thanks!

On Nov 26, 2013, at 7:27 PM, Andrew Sutherland <notifications@github.com> wrote:

> In apps/email/js/cards/message_list.js:
> 
> > @@ -743,6 +743,17 @@ MessageListCard.prototype = {
> >      this._snippetRequestPending = false;
> >    },
> >  
> > +  // the distance between items. It is expected to remain fairly constant
> > +  // throughout the list so we only need to calculate it once.
> > +  _getDistance: function() {
> > +    var items = this.messagesSlice.items;
> > +    if (!this._distanceBetweenMessages && items.length) {
> > +      this._distanceBetweenMessages =
> > +        items[0].element.getBoundingClientRect().height;
> I think this won't compensate for inter-element margins (getBoundingClientRect is only border-box not margin-box) and you'll want to calculate the distance between the tops of 2 (or similar) instead of just the height. MSDN has a neat picture at http://msdn.microsoft.com/en-us/library/hh781509%28v=vs.85%29.aspx
> 
> —
> Reply to this email directly or view it on GitHub.