Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

fix(RTL): apply static styles using the top-most Fela renderer #960

Merged
merged 6 commits into from
Feb 28, 2019

Conversation

miroslavstastny
Copy link
Member

@miroslavstastny miroslavstastny commented Feb 25, 2019

Static styles were always applied on LTR Fela renderer, fixes #948.
Now the first (top level) Provider decides which renderer to use, caches it, and all other Providers reuse that renderer to apply static styles.

  • update changelog

@codecov
Copy link

codecov bot commented Feb 25, 2019

Codecov Report

Merging #960 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
+ Coverage   80.72%   80.73%   +0.01%     
==========================================
  Files         659      659              
  Lines        8457     8462       +5     
  Branches     1429     1495      +66     
==========================================
+ Hits         6827     6832       +5     
  Misses       1616     1616              
  Partials       14       14
Impacted Files Coverage Δ
packages/react/src/lib/felaRenderer.tsx 93.75% <100%> (ø) ⬆️
...ackages/react/src/components/Provider/Provider.tsx 96.72% <100%> (+0.29%) ⬆️

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 600115a...8e84a21. Read the comment docs.

@codecov
Copy link

codecov bot commented Feb 25, 2019

Codecov Report

Merging #960 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
+ Coverage    81.1%   81.11%   +0.01%     
==========================================
  Files         671      671              
  Lines        8612     8617       +5     
  Branches     1514     1452      -62     
==========================================
+ Hits         6985     6990       +5     
  Misses       1612     1612              
  Partials       15       15
Impacted Files Coverage Δ
packages/react/src/lib/felaRenderer.tsx 93.75% <100%> (ø) ⬆️
...ackages/react/src/components/Provider/Provider.tsx 96.72% <100%> (+0.29%) ⬆️

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 310f4fd...0d766ba. Read the comment docs.

@miroslavstastny miroslavstastny merged commit 35ab460 into master Feb 28, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix/fela-static-styles branch February 28, 2019 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RTL Provider breaks styles
3 participants