Closed Bug 595507 Opened 15 years ago Closed 15 years ago

When an invalid form in an IFRAME is submitted, error messages are not displayed

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 4.0b7
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: jk1700, Assigned: mounir)

References

()

Details

(Keywords: html5)

Attachments

(1 file, 1 obsolete file)

User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100911 Firefox/4.0b6pre Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100911 Firefox/4.0b6pre When the page contains an IFRAME with the form containing required fields the error message "This field is required" is not displayed Reproducible: Always Steps to Reproduce: 1. Open the page http://files.karczmarczyk.pl/x/ 2. Don't fill in the text input, just click "Submit" Actual Results: Nothing happens Expected Results: Error message "This field is required" should be displayed When you open the iframe page http://files.karczmarczyk.pl/x/iframe.html the error message is displayed
Blocks: 344614
Keywords: html5
Version: unspecified → Trunk
Nice catch ;) I will investigate that. I think this should probably be a blocker considering this will give a very bad experience to our users.
Assignee: nobody → mounir.lamouri
Blocks: 561636
No longer blocks: 344614
Status: UNCONFIRMED → NEW
blocking2.0: --- → ?
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86_64 → All
Assignee: mounir.lamouri → nobody
Component: General → DOM: Core & HTML
Product: Firefox → Core
QA Contact: general → general
Oops - didn't mean to reset assignee.
Assignee: nobody → mounir.lamouri
I do not think that's a dom bug actually. The message is shown by browser.js.
Blocking based on comment 1.
blocking2.0: ? → betaN+
Status: NEW → ASSIGNED
Component: DOM: Core & HTML → General
Product: Core → Firefox
QA Contact: general → general
Attached patch Patch v1 (obsolete) — Splinter Review
Attachment #475987 - Flags: review?(dolske)
Attached patch Patch v1.1Splinter Review
Oups, qnew put some other changes inside the patch. Should be better now.
Attachment #475987 - Attachment is obsolete: true
Attachment #475988 - Flags: review?(dolske)
Attachment #475987 - Flags: review?(dolske)
Comment on attachment 475988 [details] [diff] [review] Patch v1.1 r=me if you also change "gBrowser.selectedTab.linkedBrowser.contentDocument" to "gBrowser.contentDocument" :)
Attachment #475988 - Flags: review?(dolske) → review+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: