Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Some tweaks to better support rtl #3551

Merged
merged 1 commit into from
May 22, 2018

Conversation

meandavejustice
Copy link
Contributor

  • fixes Suport RTL for testpilot FrontEnd #1906
  • footer fixes were not addressed because footer is in process of being updated,
    will need a rtl review once Photonize footer #3550 is merged.
  • experiment page has changed recently, from my reviews it appears that all of
    the referened issues have been addressed already.

- fixes mozilla#1906
- footer fixes were not addressed because footer is in process of being updated,
will need a rtl review once mozilla#3550 is merged.
- experiment page has changed recently, from my reviews it appears that all of
the referened issues have been addressed already.
@codecov-io
Copy link

Codecov Report

Merging #3551 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3551      +/-   ##
==========================================
- Coverage   82.07%   82.02%   -0.06%     
==========================================
  Files         110      110              
  Lines        2935     2937       +2     
==========================================
  Hits         2409     2409              
- Misses        526      528       +2
Impacted Files Coverage Δ
frontend/src/app/containers/App/index.js 13.33% <0%> (-0.21%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d853d7...73900a1. Read the comment docs.

@meandavejustice meandavejustice merged commit af98e0d into mozilla:master May 22, 2018
@meandavejustice meandavejustice deleted the 1906-support-rtl branch May 22, 2018 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants