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

docs: add theming guide #152

Merged
merged 10 commits into from
Sep 20, 2018
Merged

docs: add theming guide #152

merged 10 commits into from
Sep 20, 2018

Conversation

levithomason
Copy link
Member

This PR is another breakout of a larger documentation branch. It adds only a theming guide.

@codecov
Copy link

codecov bot commented Aug 30, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   92.57%   92.57%           
=======================================
  Files          62       62           
  Lines        1131     1131           
  Branches      168      168           
=======================================
  Hits         1047     1047           
  Misses         81       81           
  Partials        3        3

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 5e5fac6...56d124c. Read the comment docs.

@levithomason levithomason force-pushed the docs/theming branch 2 times, most recently from 470c693 to f87138f Compare September 14, 2018 04:30
@alinais alinais changed the title [WIP] docs: add theming guide docs: add theming guide Sep 19, 2018
@@ -49,6 +49,7 @@ const buttonStyles: IComponentPartStylesInput = {
height,
minWidth,
maxWidth,
fontSize: variables.textSize,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed to remove this and hardcode style for now

@alinais alinais merged commit a7059e1 into master Sep 20, 2018
@alinais alinais deleted the docs/theming branch September 21, 2018 09:42
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.

5 participants