1180558 - [RTL] About page doesn't support the RTL direction exactly.

Status: RESOLVED FIXED

(www.mozilla.org :: Pages & Content, defect)

Description

[Manel Rhaiem [:manel]]

Attachment: Screenshot from 2015-07-06 03:54:16.png

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150513191138

Steps to reproduce:

*get the last version of bedrock 
*open the bedrock project 
*activate the virtual venv
*run
*Go through about page with arabic language 


Actual results:

the article in the left (ltr direction) and the video with the red title in the right.


Expected results:

the video should be in the left and the article with the red title in the right.

Comment 1

[Kohei Yoshino [:kohei]]

Thanks for filing bugs! I'm working on a site-wide better RTL handing in Bug 1166055. There are many pages requiring an optimization but this specific page can be prioritized. I'll send a GitHub pull request hopefully this week.

Comment 2

[Manel Rhaiem [:manel]]

(In reply to Kohei Yoshino [:kohei] from comment #1)
> Thanks for filing bugs! I'm working on a site-wide better RTL handing in Bug
> 1166055. There are many pages requiring an optimization but this specific
> page can be prioritized. I'll send a GitHub pull request hopefully this week.

Great Kohei, I can help too in the RTl handing in CSS

Comment 3

[Manel Rhaiem [:manel]]

Attachment: [RTL] About page doesn't support the RTL direction exactly.

This is my PR needs review

Comment 4

[Kohei Yoshino [:kohei]]

Oh mermi, sorry I couldn't recognize your real name ;-) Please go ahead with your PR; my PR is an site-wide overhaul that may take longer to bake.

Comment 5

[[github robot]]

Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/109336eb60313bc0cf4d806782b85ac0669eff73
Merge pull request #3102 from mermi/bug-1180558-about-page-rtl

Fix Bug 1180558, aligning video correctly fixes RTL