605930 - [RTL]Pressing on the search icon on the awesome page shows the dialog in LTR style

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: N900 Screenshot

Mozilla/5.0 (Maemo;Linux armv71; rv:2.0b8pre) Gecko/20101020 Firefox/4.0b8pre Fennec/4.0b2pre
Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101020 Firefox/4.0b8pre Fennec/4.0b2pre

1. set browser to RTL (ie download/install arabic plugin : http://people.mozilla.com/~nhirata/html_tp/installplugin.html

Note : plugin installed notification will not show due to another bug)
2. click on awesome bar to go to awesome page
3. click on the search icon

Expected:
a list of search sites and a RTL pointer pointing to the magnifying glass icon

Actual:
1) no search sites
2) no pointer to the magnifying glass.

Note:
1. madhava noticed that there was an issue with RTL and the new theme

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: Droid 2 Screenshot

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: favicon click

Note: arrow is also pointing in the wrong location for clicking on the favicon when the browser is set to RTL.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: Android Droid 2 Screenshot

Note: Issue 3. Clicking on the bookmark icon in the control panel also shows the arrow issue.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Separating the search engine issue as a different issue : Bug 606056
Retitled this bug.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

I have fixed the issues by doing 2 things:
 * adding a dir="ltr" to the arrowbox container  to have the inner arrow always on the left part of the box
 * adding container.dir="ltr" otherwise we're wrong when we think about normal/reverse mode in rtl

These changes does not affect the layout of the content, just the way the box is ordered

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

http://hg.mozilla.org/mobile-browser/rev/295bccfd274c

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Cannot verify for search engines in Awesomepage (search icon is disabled until plugins are installed), however favorites and site page are fixed.

Mozilla/5.0 (Maemo;Linux armv71; rv:2.0b8pre) Gecko/20101022 Firefox/4.0b8pre Fennec/4.0b2pre
Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101022 Firefox/4.0b8pre Fennec/4.0b2pre

Comment 8

[Dão Gottwald [:dao]]

bugspam