Skip to content
This repository has been archived by the owner on Dec 26, 2024. It is now read-only.

Fab/docs #140

Closed
wants to merge 4 commits into from
Closed

Fab/docs #140

wants to merge 4 commits into from

Conversation

Juyeong-Byeon
Copy link
Contributor

@Juyeong-Byeon Juyeong-Byeon commented Oct 6, 2021

Description

I wrote a document for FAB.
I explained the definition of FAB and interface.

스크린샷 2021-10-06 오후 11 45 47

Test Plan

none

Related Issues

Related Issues

Tests

none

Checklist

Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • Run yarn test:all and make sure nothing fails. You can run yarn test -u to update snapshots if needed.
  • I am willing to follow-up on review comments in a timely manner.

before:
the gap between the buttons are not the same across the environment

after
1. change the unit for the gap. % => px
2. gap will be decided with the height of FAB view
1. const GAP => props gap
2. omit generic
1. add story for FAB button in doc project
2. add basic introduction  doc for FAB
@codecov
Copy link

codecov bot commented Oct 6, 2021

Codecov Report

Merging #140 (5153505) into master (8c40090) will not change coverage.
The diff coverage is 85.71%.

❗ Current head 5153505 differs from pull request most recent head d1fe7db. Consider uploading reports for the commit d1fe7db to get more accurate results

@@           Coverage Diff           @@
##           master     #140   +/-   ##
=======================================
  Coverage   94.59%   94.59%           
=======================================
  Files          27       27           
  Lines         648      648           
  Branches      283      283           
=======================================
  Hits          613      613           
  Misses         34       34           
  Partials        1        1           

@Juyeong-Byeon
Copy link
Contributor Author

I will open again in PR

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant