Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert remaining pattern library pages to newer Library components #175

Merged
merged 6 commits into from
Aug 31, 2021

Conversation

lyzadanger
Copy link
Contributor

This finishes the conversion of pattern-library pages over to Library components and removes unused components and styles.

There are a number of areas of improvements for follow-up, but I want to address them subsequently. A few obvious items:

  • Typography is still less than stellar
  • The ButtonComponents page in particular looks crowded and hard to scan
  • The LayoutFoundations page is not responsive because of the examples on it

But this does move us into a better position to address all that, as the pattern-library pages will all be on the same...page :) (using the same components).

Fixes #162

@esanzgar esanzgar self-requested a review August 30, 2021 10:11
Copy link
Contributor

@esanzgar esanzgar left a comment

Choose a reason for hiding this comment

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

I pointed to a couple typos and suggestions.

@lyzadanger
Copy link
Contributor Author

@esanzgar Thanks for your thorough review of this and catching some little typos and things!

@lyzadanger lyzadanger force-pushed the component-page-conversion branch from eea6ac2 to 4856b90 Compare August 31, 2021 13:22
@codecov
Copy link

codecov bot commented Aug 31, 2021

Codecov Report

Merging #175 (4856b90) into main (08a09bb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #175   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          158       158           
  Branches        55        55           
=========================================
  Hits           158       158           

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 08a09bb...4856b90. Read the comment docs.

@lyzadanger lyzadanger merged commit b8ca20d into main Aug 31, 2021
@lyzadanger lyzadanger deleted the component-page-conversion branch August 31, 2021 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Epic: Convert Pattern-Library pages to newer "Library" components
2 participants