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

fix(docs): Fela and fontAwesome classes conflict#952

Closed
bmdalex wants to merge 1 commit intomasterfrom
fix/docs-font-awesom-fela-conflict
Closed

fix(docs): Fela and fontAwesome classes conflict#952
bmdalex wants to merge 1 commit intomasterfrom
fix/docs-font-awesom-fela-conflict

Conversation

@bmdalex
Copy link
Collaborator

@bmdalex bmdalex commented Feb 22, 2019

fix(docs): Fela and fontAwesome classes conflict

Fixes #950

Fix is proposed by @rofrischmann (creator of Fela) here

Downside:

The prefix is added to all class names:
screenshot 2019-02-22 at 15 47 28

EDIT:

Alternative to #951

@miroslavstastny
Copy link
Member

What about #951?

@bmdalex
Copy link
Collaborator Author

bmdalex commented Feb 22, 2019

@miroslavstastny haha, let's vote? 😆

@DustyTheBot
Copy link
Collaborator

Warnings
⚠️ There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

Generated by 🚫 dangerJS

@codecov
Copy link

codecov bot commented Feb 22, 2019

Codecov Report

Merging #952 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #952   +/-   ##
=======================================
  Coverage   80.51%   80.51%           
=======================================
  Files         659      659           
  Lines        8448     8448           
  Branches     1492     1428   -64     
=======================================
  Hits         6802     6802           
  Misses       1631     1631           
  Partials       15       15
Impacted Files Coverage Δ
packages/react/src/lib/felaRenderer.tsx 100% <ø> (ø) ⬆️

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 ccc755b...1d13e4a. Read the comment docs.

@bmdalex
Copy link
Collaborator Author

bmdalex commented Feb 22, 2019

closing in favor of #951

@bmdalex bmdalex closed this Feb 22, 2019
@bmdalex bmdalex deleted the fix/docs-font-awesom-fela-conflict branch February 22, 2019 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🧰 fix Introduces fix for broken behavior. 🚀 ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] font-family gets randomized in docSite examples

3 participants