Better positioning for the invalid form popup

RESOLVED FIXED in Firefox 4.0b10

Status

()

Firefox
General
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

Trunk
Firefox 4.0b10
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
This was originally part of bug 619223 but split to make things easier.

We should prevent this situation:
http://fredericiana.com/wp-content/uploads/2010/12/html5-feedback-forms-1.jpg
(Assignee)

Comment 1

8 years ago
Created attachment 502257 [details] [diff] [review]
Patch v1
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #502257 - Flags: review?(enndeakin)
(Assignee)

Updated

8 years ago
Whiteboard: [needs-review]

Comment 2

8 years ago
Comment on attachment 502257 [details] [diff] [review]
Patch v1

>+    let style = element.ownerDocument.defaultView.getComputedStyle(element, null);

This should be inside the else block.

>+    } else {
>+      offset = parseInt(style.paddingLeft) + parseInt(style.borderLeftWidth);
>+      position = "after_start";
>+    }

Do you need to use the right padding and border for rtl? Unfortunately, it doesn't look like paddingStart/borderStartWidth is implemented for getComputedStyle.
Attachment #502257 - Flags: review?(enndeakin) → review+
(Assignee)

Comment 3

8 years ago
(In reply to comment #2)
> Comment on attachment 502257 [details] [diff] [review]
> Patch v1
> 
> >+    let style = element.ownerDocument.defaultView.getComputedStyle(element, null);
> 
> This should be inside the else block.
> 
> >+    } else {
> >+      offset = parseInt(style.paddingLeft) + parseInt(style.borderLeftWidth);
> >+      position = "after_start";
> >+    }
> 
> Do you need to use the right padding and border for rtl? Unfortunately, it
> doesn't look like paddingStart/borderStartWidth is implemented for
> getComputedStyle.

Indeed, I should change that.
(Assignee)

Updated

8 years ago
Attachment #502257 - Flags: approval2.0?
(Assignee)

Updated

8 years ago
Whiteboard: [needs-review] → [needs-approval]
(Assignee)

Updated

8 years ago
Whiteboard: [needs-approval] → [needs approval][passed try]
Comment on attachment 502257 [details] [diff] [review]
Patch v1

a=beltzner
Attachment #502257 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 5

7 years ago
Pushed:
http://hg.mozilla.org/mozilla-central/rev/d910f58e6c9c
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [needs approval][passed try]
Target Milestone: --- → Firefox 4.0b10
(Assignee)

Updated

7 years ago
Depends on: 628695
You need to log in before you can comment on or make changes to this bug.