Closed Bug 350801 Opened 18 years ago Closed 18 years ago

Search Go button doesn't get turned around in rtl mode

Categories

(Firefox :: Search, defect)

2.0 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 2

People

(Reporter: myk, Assigned: myk)

References

Details

(Keywords: fixed1.8.1, rtl, Whiteboard: [Fx2 theme change])

Attachments

(2 files)

The Search Go button doesn't get turned around in right-to-left mode. It stays the same as in left-to-right mode, which makes it look strangely disconnected from the search box. The problem is that the rule responsible for turning it around relies on the presence of a "chromedir" attribute on the toolbarbutton element, but that attribute doesn't exist on that element.
Attachment #236172 - Attachment is patch: false
Attachment #236172 - Attachment mime type: text/plain → image/png
Attached patch patch v1: fixes problem (deleted) — Splinter Review
Here's a fix for the problem. Note that this is branch only. Trunk doesn't even have an rtl version of this icon, although we'll need this fix there when we land the new theme on the trunk.
Attachment #236178 - Flags: review?(gavin.sharp)
Attachment #236178 - Flags: review?(gavin.sharp) → review+
Comment on attachment 236178 [details] [diff] [review] patch v1: fixes problem Notes for drivers considering this approval request: This trivial, low-risk fix makes Firefox display the right-to-left search button in right-to-left mode. Currently Firefox is displaying the left-to-right button in both modes, which looks funky in right-to-left mode (see the attached screenshot). This patch is branch-only, since the theme on the trunk doesn't have a separate rtl search button. But we'll need to land this fix on the trunk as well when we land the new theme there later.
Attachment #236178 - Flags: approval1.8.1?
*** Bug 350862 has been marked as a duplicate of this bug. ***
Flags: blocking-firefox2?
Per mconnor, I'll land this on trunk as well, resolving the trivial conflict, so this bug doesn't regress when mconnor lands the new theme on the trunk.
Flags: blocking-firefox2? → blocking-firefox2+
Attachment #236178 - Flags: approval1.8.1? → approval1.8.1+
Blocks: NewThemeRTL
Fix checked in to trunk and 1.8 branch.
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Target Milestone: --- → Firefox 2
Whiteboard: [Fx2 theme change]
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: