short-d / short

URL shortening service written in Go and React
https://short-d.com
MIT License
870 stars 148 forks source link

Add story for Table component #1001

Open Coteh opened 4 years ago

Coteh commented 4 years ago

Addresses the Table story in #954.

Leaving in Draft until these items are done:

This PR also adds Storybook Backgrounds, as I wanted to add the color #fafafa, which is being used underneath the Favourite short links table in the homepage, as a background colour for the canvas. This will make the Table component stand out better. I also set #fafafa as the default background colour for story canvases.

image
codecov[bot] commented 4 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1001   +/-   ##
=======================================
  Coverage   52.58%   52.58%           
=======================================
  Files         142      142           
  Lines        3564     3564           
  Branches      168      168           
=======================================
  Hits         1874     1874           
  Misses       1623     1623           
  Partials       67       67           
Flag Coverage Δ
#golang 72.03% <ø> (ø)
#typescript 22.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
frontend/src/component/ui/Table.tsx 100.00% <ø> (ø)

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 ebc6a04...c243c26. Read the comment docs.

Coteh commented 4 years ago

Readied this PR for review, will convert it to Storybook Controls if we start using it.

magicoder10 commented 4 years ago

@Coteh Control seems good. Let's do it!

Coteh commented 4 years ago

@byliuyang I think what I'll do is merge this in as-is, then make a separate PR for Controls and convert all existing stories to use Controls in that PR. What do you think?

Coteh commented 4 years ago

@byliuyang Screenshot added