Closed Bug 494588 Opened 16 years ago Closed 16 years ago

Go button in the search form points to right in RTL mode

Categories

(addons.mozilla.org Graveyard :: Public Pages, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

()

Details

(Keywords: rtl)

Attachments

(1 file)

Attached patch Patch (v1)Splinter Review
Go button in the search form points to right in RTL mode, while it should point to left.
Attachment #379348 - Flags: review?(stephen.donner)
No longer blocks: amo2009-rtl
Depends on: amo2009-rtl
Blocks: amo2009-rtl
No longer depends on: amo2009-rtl
Hey Ehsan -- mind switching all these reviews to Wil, or someone else on the dev team? I'm QA, and not really qualified to review these patches (though I'd love it if I were!) Thanks!
Comment on attachment 379348 [details] [diff] [review] Patch (v1) (In reply to comment #1) > Hey Ehsan -- mind switching all these reviews to Wil, or someone else on the > dev team? I'm QA, and not really qualified to review these patches (though I'd > love it if I were!) Thanks! Sure, sorry for the confusion!
Attachment #379348 - Flags: review?(stephen.donner) → review?(clouserw)
Comment on attachment 379348 [details] [diff] [review] Patch (v1) Clearleft decided to make IE6 not load styles (!) so this will be broken for them. I think that is a larger overall problem though so I don't have a problem with this.
Attachment #379348 - Flags: review?(clouserw) → review+
Keywords: checkin-needed
Committed in r26356.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: