Closed Bug 494588 Opened 15 years ago Closed 15 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: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Verified in https://preview.addons.mozilla.org/fa/firefox/.
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: