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

feat(docs): add FAQ entry for Flex, Grid, Layout, Box, Segment #1646

Merged
merged 12 commits into from
Jul 19, 2019

Conversation

lucivpav
Copy link
Contributor

This PR aims to clarify the differences between usages of the following components: Flex, Grid, Layout, Box, Segment.

This PR resolves Issue #1531.

@lucivpav
Copy link
Contributor Author

Example(s) should be given on Segment component.

@vercel vercel bot requested a deployment to staging July 16, 2019 10:35 Abandoned
@codecov
Copy link

codecov bot commented Jul 16, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1646   +/-   ##
=======================================
  Coverage   71.16%   71.16%           
=======================================
  Files         857      857           
  Lines        7102     7102           
  Branches     2032     2052   +20     
=======================================
  Hits         5054     5054           
  Misses       2042     2042           
  Partials        6        6

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 c965ce6...870c712. Read the comment docs.

@vercel vercel bot requested a deployment to staging July 16, 2019 11:03 Abandoned
@lucivpav
Copy link
Contributor Author

Example(s) should be given on Segment component.

Relevant issue #1530

docs/src/views/FAQ.tsx Outdated Show resolved Hide resolved
docs/src/views/FAQ.tsx Outdated Show resolved Hide resolved
@kuzhelov kuzhelov added 📚 docs needs author feedback Author's opinion is asked labels Jul 19, 2019
@vercel vercel bot temporarily deployed to staging July 19, 2019 10:18 Inactive
docs/src/views/FAQ.tsx Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

looks good to me 👍
@mnajdova, @layershifter, please, share your thoughts on this. Thank you!

@kuzhelov kuzhelov removed the needs author feedback Author's opinion is asked label Jul 19, 2019
@layershifter
Copy link
Member

Please add the question in the title

image

docs/src/views/FAQ.tsx Outdated Show resolved Hide resolved
@lucivpav
Copy link
Contributor Author

Please add the question in the title

image

Fixed.

@lucivpav lucivpav merged commit a46551e into master Jul 19, 2019
@delete-merged-branch delete-merged-branch bot deleted the docs/box-segment-flex-grid-layout-faq branch July 19, 2019 15:13
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