Closed Bug 320938 Opened 19 years ago Closed 18 years ago

autocompletion of a text-field pops up above firefox

Categories

(Toolkit :: Autocomplete, defect)

Product:
Toolkit
Component:
Autocomplete
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9alpha1

People

(Reporter: michael.arnauts+mozilla, Assigned: mwu)

References

(
URL
)

Details

(Keywords: fixed1.8.1, testcase)

Attachments

(2 files)

Make input field visible before opening popup
(deleted), patch
jst
: first-review+
dbaron
: approval1.8.1+
Details | Diff | Splinter Review
Make input field visible before opening popup (branch)
(deleted), patch
Details | Diff | Splinter Review
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; nl; rv:1.8) Gecko/20051111 Firefox/1.5 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; nl; rv:1.8) Gecko/20051111 Firefox/1.5 Open the testcase: http://users.skynet.be/bk333867/testcase.html Click inside the inputbox Scroll down, without clicking inside the browserarea (so the cursor stays in the inputfield) Press key down You should see the combobox of the inputfield (with the previous entered words) appear on top of firefox. Reproducible: Always Steps to Reproduce:
Flags: testcase+
Flags: testcase+
Keywords: testcase
Component: General → Autocomplete
Product: Firefox → Toolkit
QA Contact: general → autocomplete
Reproducible with Firefox 1.0.6 and Firefox 1.5 (Linux), related to bug 187772?
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20051220 Firefox/1.6a1 Masayuki's patch for bug 187772 does not fix this for me.
OS: Windows XP → All
Hardware: PC → All
Version: unspecified → Trunk
Summary: autocompletition of a text-field pops up above firefox → autocompletion of a text-field pops up above firefox
Thanks, I'll create the patch.
Assignee: nobody → masayuki
Status: UNCONFIRMED → NEW
Ever confirmed: true
Target Milestone: --- → mozilla1.9alpha1
Umm... the patch conflicts with bug 187772's patch. We need to fix bug 187772 before this...
Depends on: 187772
Is the patch still around somewhere?
No, but I don't have much time for this now, if you can fix, please take this.
(In reply to comment #6) > No, but I don't have much time for this now, if you can fix, please take this. > Sure, I'll take a look.
Assignee: masayuki → michael.wu
Attachment #229183 - Flags: first-review?(jst)
Comment on attachment 229183 [details] [diff] [review] Make input field visible before opening popup Looks like the right thing to do. r=jst
Attachment #229183 - Flags: first-review?(jst) → first-review+
Checked in to trunk
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Attachment #229183 - Flags: approval1.8.1?
Comment on attachment 229183 [details] [diff] [review] Make input field visible before opening popup a=dbaron on behalf of drivers. Please check in to MOZILLA_1_8_BRANCH and mark fixed1.8.1 after doing so.
Attachment #229183 - Flags: approval1.8.1? → approval1.8.1+
Keywords: fixed1.8.1
The branch version of GetPrimaryFrameFor sucks.
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
Checking in nsFormFillController.cpp; /cvsroot/mozilla/toolkit/components/satchel/src/nsFormFillController.cpp,v <-- nsFormFillController.cpp new revision: 1.51.4.13; previous revision: 1.51.4.12 done Checked into 1.8.1 branch.
Whiteboard: [checkin needed (1.8 branch)]
Keywords: fixed1.8.1
*** Bug 336689 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: