[rtl] Add RTL support to about:firstrun in Fennec

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: linostar, Assigned: linostar)

Tracking

({rtl})

Details

Attachments

(4 attachments, 2 obsolete attachments)

(Assignee)

Description

7 years ago
The firstrun page needs some tweaks in order to be displayed properly on rtl fennec.
(Assignee)

Updated

7 years ago
Assignee: nobody → linux.anas
Status: NEW → ASSIGNED
(Assignee)

Updated

7 years ago
Keywords: rtl
(Assignee)

Comment 1

7 years ago
Created attachment 523246 [details] [diff] [review]
patch for RTL-izing about:firstrun

I wasn't sure if I should include the flipped arrow image (nav-arrow-rtl.png) in the patch or if I had to attach the image separately. I've done the first option.
(Assignee)

Comment 2

7 years ago
Created attachment 523247 [details] [diff] [review]
patch for RTL-izing about:firstrun (image included)
Attachment #523246 - Attachment is obsolete: true
Attachment #523247 - Flags: review?(21)
(In reply to comment #1)
> Created attachment 523246 [details] [diff] [review]
> patch for RTL-izing about:firstrun
> 
> I wasn't sure if I should include the flipped arrow image (nav-arrow-rtl.png)
> in the patch or if I had to attach the image separately. I've done the first
> option.

Adding image in the patch is fine, you can also provide an image of the final result as an attachment (sometimes people provide an image before and after the patch) so it is easy to see what the images looks like if you have many.


Looking at your patch I'm wondering if we can't avoid the cost of adding an extra image by adding an <img src="chrome://browser/content/firstrun/nav-arrow.png"/> directly in the html, align it properly and use the -moz-transform css rules to rotate it?

Can you try that?
(Assignee)

Comment 4

7 years ago
Created attachment 523271 [details]
screenshot for firstrun page before the patch
(Assignee)

Comment 5

7 years ago
Created attachment 523272 [details]
screenshot for firstrun page after the patch
(Assignee)

Comment 6

7 years ago
Created attachment 523278 [details] [diff] [review]
patch v2
Attachment #523247 - Attachment is obsolete: true
Attachment #523247 - Flags: review?(21)
Attachment #523278 - Flags: review?(21)
(Assignee)

Comment 7

7 years ago
BTW,tested the patch on an en-US Fennec, and it didn't break the firstrun page.
Comment on attachment 523278 [details] [diff] [review]
patch v2

Looks good. Thanks for the patch Anas.
Attachment #523278 - Flags: review?(21) → review+
http://hg.mozilla.org/mobile-browser/rev/bfd90c18b939
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 10

7 years ago
Created attachment 523541 [details]
firstrun

Mozilla /5.0 (Android;Linux armv7l;rv:2.2a1pre) Gecko/20110331 Firefox/4.2a1pre Fennec /4.1a1pre 
Device: HTC Desire

This is how it looks for me. Is it ok? Can I mark this bug as verified?
(Assignee)

Comment 11

7 years ago
Looks fine. Go ahead and mark it as verified.
verified:
Mozilla/5.0 (Android; Linux armv71; rv2.2a1pre) Gecko/20110405 Firefox/4.2a1pre Fennec/4.1a1pre
Device: Droid 2 
OS: Android 2.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.