Closed Bug 522043 Opened 10 years ago Closed 10 years ago

Text in nsPromptService authentication dialog is pannable

Categories

(Firefox for Android Graveyard :: General, defect)

Fennec 1.1
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
fennec1.0b5

People

(Reporter: aakashd, Assigned: mfinkle)

Details

Attachments

(1 file)

Build Id:

Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2b1pre) Gecko/20091013 Fennec/1.0a4pre

and

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091013
Fennec/1.0b5pre

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091013
Fennec/1.0b5pre

Note: this is a follow-up bug for bug 514208

Steps to Reproduce:
1. Go to a set that requires nsPromptService dialog
2. Pan up and down on the text area

Actual Results:
The text area above user name and password is pannable

Expected Results:
The text area should not be pannable
Flags: wanted-fennec1.0?
Attached patch patchSplinter Review
This patch adds the <description> margin into the calculations. No panning after that is added into the height.
Assignee: nobody → mark.finkle
Attachment #406071 - Flags: review?(fabrice.desre)
Comment on attachment 406071 [details] [diff] [review]
patch

Looks fine to me.
Attachment #406071 - Flags: review?(fabrice.desre) → review+
pushed:
https://hg.mozilla.org/mobile-browser/rev/b72c487b8c81
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091015
Fennec/1.0b5pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.