REGRESSION (NativeTextField): New text fields don't scroll to the beginning when losing focus

• Component: Forms
• Status: VERIFIED FIXED

New text fields don't scroll back to the beginning when they are unfocused.

To reproduce: enter long text in a text field, making it scroll to the right (to show the end of the text), then click outside the text field (to unfocus it).

Expected result: the text field should scroll to the left, showing the beginning of the text, when it's unfocused.

Actual result: the text field doesn't scroll back when it's unfocused.

[Darin Adler]

I've got a fix for this one.

[Darin Adler]

Created attachment 7437 [details]
patch with detailed change log and a layout test

[Justin Garcia]

Comment on attachment 7437 [details]
patch with detailed change log and a layout test

r=me

RTL fields should scroll all the way to the right. The patch made them scroll to the left :-\

[Darin Adler]

(In reply to comment #4)
> RTL fields should scroll all the way to the right. The patch made them scroll
> to the left :-\

I have to admit I even thought of that, but I wasn't sure how to code that correctly.

[Darin Adler]

I think I have the code right to handle RTL correctly now, but only for the "blur" case, not for text fields that never had focus in the first place.

[Darin Adler]

By the way, Firefox does not have this behavior for text fields. You could even consider this an anomaly of AppKit text fields. The reason they scroll to the beginning when they lose focus is that the base field has no scrolling capability, only the field editor does.

So I'm not entirely sure it was a good idea for me to have fixed this!

[Darin Adler]

Created attachment 7456 [details]
patch to handle RTL case, including adding it to the test case

[Maciej Stachowiak]

These are all text field regressions so they should all be P1.