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

Add root components directory #736

Merged
merged 3 commits into from
May 9, 2017
Merged

Add root components directory #736

merged 3 commits into from
May 9, 2017

Conversation

aduth
Copy link
Member

@aduth aduth commented May 9, 2017

Previously: #716

This pull request expands upon the restructuring outlined in #716, creating a new common root components directory containing components which have shown to be generic between blocks- and editor-specific components.

Testing instructions:

Ensure the build completes, tests pass, and editor initializes and appears without regressions. Specifically observe that there are no regressions in displayed buttons, icon buttons, icons, and toolbar components.

@aduth aduth added the Framework Issues related to broader framework topics, especially as it relates to javascript label May 9, 2017
@aduth aduth requested a review from mtias May 9, 2017 16:55
Components
==========

This directory includes a library of generic React components to be used for creating common UI elements shared between screens and features of the WordPress dashboard.
Copy link
Contributor

Choose a reason for hiding this comment

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

I love this. Shared UI components inside WordPress. This makes me think we should showcase those.

Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

I think there's a style regression in the inserter buttons. They look bigger (more padding maybe).

Otherwise, this is great.

@aduth
Copy link
Member Author

aduth commented May 9, 2017

Good catch @youknowriad . Looks like this was just a typo, fixed in 6f0f79d.

@aduth aduth force-pushed the add/components-directory branch from 6f0f79d to 2c168f5 Compare May 9, 2017 17:52
@aduth aduth merged commit 3c5ddd3 into master May 9, 2017
@aduth aduth deleted the add/components-directory branch May 9, 2017 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues related to broader framework topics, especially as it relates to javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants