368893 - Caret right-aligned in initially focused textboxes

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Steve England [:stevee]]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/20070131 Minefield/3.0a2pre ID:2007013108 [cairo]

Reported by Littlemutt at the mz forums; http://forums.mozillazine.org/viewtopic.php?p=2725745#2725745

1. New profile
2. about:config
3. Examine the 'Filter' textarea. The cursor starts off on the right hand side of this element. When you give it focus, it appears over at the left hand side as expected.

Comment 1

[Peter S.]

I see this after upgrading from 2007013004 to 2007013104 in the same profile.

Comment 2

[Steve England [:stevee]]

Regression range from comment 1
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=PhoenixTinderbox&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-01-30+03&maxdate=2007-01-31+05&cvsroot=%2Fcvsroot

Comment 3

[Dave Townsend [:mossop]]

This is visible on Mactel

Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a2pre) Gecko/20070131 Minefield/3.0a2pre ID:2007013104 [cairo]

Comment 4

[Steve England [:stevee]]

Left aligned (expected)
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/2007013011 Minefield/3.0a2pre

Right aligned (not expected)
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/2007013013 Minefield/3.0a2pre

Range
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=PhoenixTinderbox&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-01-30+10&maxdate=2007-01-30+14&cvsroot=%2Fcvsroot

Bug 363198?

Comment 5

[:Gavin Sharp [email: [e-mail]]]

See also bug 368861.

Comment 6

[Nian Liu(n/a in a long time)]

i can still see the bug when backout 363198 in my local workspace

Comment 7

[Dave Townsend [:mossop]]

(In reply to comment #6)
> i can still see the bug when backout 363198 in my local workspace
> 

What did you actually backout here? I just backed out the two patches for that bug listed in the regression range and it solved it for me.

Comment 8

[drew1jg]

Just thought I should add that this occurs not only in about:config, but in many sites that start with focus in a text box.

Comment 10

[Mike Shaver (:shaver emeritus)]

I see this in gmail without the initial focus thing, and pressing "up" past the beginning/top of the textarea results in be having a non-blinking "caret" to the _right_ (outside) of the scrollbar.  Pressing down after that gets me a caret at the very _bottom_ of the textarea, which is also quite confusing.

Apparently that's this bug?  I dunno, tbh, but I wanted to comment somewhere!

Comment 11

[Ryan Jones-Ward [:sciguyryan]]

(In reply to comment #10)
> I see this in gmail without the initial focus thing, and pressing "up" past the
> beginning/top of the textarea results in be having a non-blinking "caret" to
> the _right_ (outside) of the scrollbar.  Pressing down after that gets me a
> caret at the very _bottom_ of the textarea, which is also quite confusing.
> 
> Apparently that's this bug?  I dunno, tbh, but I wanted to comment somewhere!
> 

Its related for sure because we have the characteristic caret the same height as the textarea. So seems like the same bug to me (but not initially to the right which is different).

Comment 12

[Nian Liu(n/a in a long time)]

(In reply to comment #7)
> (In reply to comment #6)
> > i can still see the bug when backout 363198 in my local workspace
> > 
> 
> What did you actually backout here? I just backed out the two patches for that
> bug listed in the regression range and it solved it for me.
> 

what do you mean two patches? patch for bug 363198 is one file modified.

Comment 13

[Jesse Ruderman]

Bonsai shows it as two checkins:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&hours=2&date=explicit&mindate=2007-01-30+11%3A29&maxdate=2007-01-30+11%3A36&cvsroot=%2Fcvsroot

Comment 14

[Nian Liu(n/a in a long time)]

(In reply to comment #13)
> Bonsai shows it as two checkins:
> http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&hours=2&date=explicit&mindate=2007-01-30+11%3A29&maxdate=2007-01-30+11%3A36&cvsroot=%2Fcvsroot
> 

thanks for the explanation. i can't reprdocueo on my local workspace. feel free to reopen it when you're sure it's the regression

Comment 16

[Dave Townsend [:mossop]]

Not sure how much more sure I can get. With a clean build on my machine I can reproduce the issue (right aligned caret on textboxes and the caret growing to the full height in empty textareas when pressing the up arrow). Reverting the patch from bug 363198 and clobbering my build and I can't reproduce at all.

Comment 17

[Uri Bernstein]

The "two patches" issue was due to me checking in the wrong version of the patch, initially. Sorry about that.

I'll try to have a look at this during the weekend.

Comment 18

[Uri Bernstein]

Attachment: patch

Always get numChildren.

I should have caught this when reviewing.

Comment 19

[Nian Liu(n/a in a long time)]

Comment on attachment 253749 [details] [diff] [review]
patch

an old bug brought out by a new patch. i like the way

Comment 20

[Uri Bernstein]

(In reply to comment #19)
> (From update of attachment 253749 [details] [diff] [review])
> an old bug brought out by a new patch. i like the way
> 

It's not really an old bug. Before the patch, there was no need to initialize numChildren outside the |else| block, because it was only referred to inside that block.

Comment 21

[Gábor Stefanik]

This bug is about the caret (text insertion point), not the cursor (mouse pointer), so changing summary accordingly.

Comment 22

[Uri Bernstein]

Checked in:
Checking in mozilla/layout/generic/nsSelection.cpp;
/cvsroot/mozilla/layout/generic/nsSelection.cpp,v  <--  nsSelection.cpp
new revision: 3.277; previous revision: 3.276
done

Comment 24

[Gábor Stefanik]

Oh wow, great. Spambots using Bugzilla for link farming. Admins, please do something!