This was originally part of bug 619223 but split to make things easier. We should prevent this situation: http://fredericiana.com/wp-content/uploads/2010/12/html5-feedback-forms-1.jpg

Comment on attachment 502257 [details] [diff] [review] Patch v1 >+ let style = element.ownerDocument.defaultView.getComputedStyle(element, null); This should be inside the else block. >+ } else { >+ offset = parseInt(style.paddingLeft) + parseInt(style.borderLeftWidth); >+ position = "after_start"; >+ } Do you need to use the right padding and border for rtl? Unfortunately, it doesn't look like paddingStart/borderStartWidth is implemented for getComputedStyle.

(In reply to comment #2) > Comment on attachment 502257 [details] [diff] [review] > Patch v1 > > >+ let style = element.ownerDocument.defaultView.getComputedStyle(element, null); > > This should be inside the else block. > > >+ } else { > >+ offset = parseInt(style.paddingLeft) + parseInt(style.borderLeftWidth); > >+ position = "after_start"; > >+ } > > Do you need to use the right padding and border for rtl? Unfortunately, it > doesn't look like paddingStart/borderStartWidth is implemented for > getComputedStyle. Indeed, I should change that.

Comment on attachment 502257 [details] [diff] [review] Patch v1 a=beltzner

Pushed: http://hg.mozilla.org/mozilla-central/rev/d910f58e6c9c