Closed
Bug 386820
Opened 17 years ago
Closed 17 years ago
Gmail , compose mail, cursor missing in textfield
Categories
(Core :: DOM: Events, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: Peter6, Assigned: smaug)
References
Details
(Keywords: regression)
Attachments
(2 files)
1.38 KB,
patch
|
Details | Diff | Splinter Review | |
3.26 KB,
patch
|
peterv
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007070400 Minefield/3.0a7pre ID:2007070400
repro:
Open FF
Open Gmail
Press compose Mail and place the cursor in the textfield
result:
No cursor visable
regressionwindow:
works in 20070703_0613_firefox-3.0a7pre.en-US.win32
fails in 20070703_0644_firefox-3.0a7pre.en-US.win32
http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=1183468380&maxdate=1183470239
-> Bug 238987
Reporter | ||
Updated•17 years ago
|
Flags: blocking1.9?
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → Olli.Pettay
Assignee | ||
Comment 1•17 years ago
|
||
Cursor is visible when plaintext editing is used.
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•17 years ago
|
||
(In reply to comment #1)
> Cursor is visible when plaintext editing is used.
>
Likewise when you reply a mail with rich formatting
Assignee | ||
Comment 3•17 years ago
|
||
Assignee | ||
Comment 4•17 years ago
|
||
Comment on attachment 270883 [details] [diff] [review]
possible
...patch.
But need to think a bit if this breaks something else.
Attachment #270883 -
Attachment is patch: true
Attachment #270883 -
Attachment mime type: text/x-patch → text/plain
Comment 5•17 years ago
|
||
No cursor in thunderbird compose window today...
Assignee | ||
Comment 6•17 years ago
|
||
This should work for now. If bug 235441 gets fixed, one may need to
add a listener for bubbling phase to catch the case when
focus is dispatched to the editor's piTarget.
Attachment #270897 -
Flags: superreview?(peterv)
Attachment #270897 -
Flags: review?(peterv)
Assignee | ||
Comment 7•17 years ago
|
||
The patch fixes Thunderbird too.
Updated•17 years ago
|
Attachment #270897 -
Flags: superreview?(peterv)
Attachment #270897 -
Flags: superreview+
Attachment #270897 -
Flags: review?(peterv)
Attachment #270897 -
Flags: review+
Assignee | ||
Comment 8•17 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 10•17 years ago
|
||
Thanks for the quick response and fix Olli
Comment 11•17 years ago
|
||
(In reply to comment #8)
> Checked in.
In my SM-Trunk-build 2007070505 the cursor only appears, when I click into the text. Without the click, the bug always exists.
Comment 12•17 years ago
|
||
I'm seeing the same thing in Thunderbird message composition window. And arrowing around doesn't move the caret once clicking as gotten it to appear. This makes selecting text and editing challenging.
Assignee | ||
Comment 13•17 years ago
|
||
(In reply to comment #12)
> And
> arrowing around doesn't move the caret once clicking as gotten it to
> appear. This makes selecting text and editing challenging.
>
I see this too, but afaik that is an older regression, possibly from
contentEditable.
I'll look at the other cursor visibility problem.
Assignee | ||
Comment 14•17 years ago
|
||
(In reply to comment #13)
> I'll look at the other cursor visibility problem.
>
Actually, https://bugzilla.mozilla.org/show_bug.cgi?id=386872#c2
says that the visibility problem started 6-28, so that cursor visibility problem isn't the same as this one.
contentEditable was checked in 6-27
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-06-27&maxdate=2007-06-28&cvsroot=%2Fcvsroot
So, I'd say that cursor visibility problem is bug 386174.
The problem with arrow keys is bug 386189.
Comment 15•17 years ago
|
||
This problem still exists as of today. Is there some doubt?
Updated•17 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•