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

Expose parts for styling #62

Merged
merged 5 commits into from Jan 8, 2019
Merged

Expose parts for styling #62

merged 5 commits into from Jan 8, 2019

Conversation

oluwasayo
Copy link
Contributor

@oluwasayo oluwasayo commented Dec 28, 2018

Fixes #5

Expose new parts form-title, description, error-message-title and error-message-description


This change is Reviewable

@coveralls
Copy link

coveralls commented Dec 28, 2018

Coverage Status

Coverage remained the same at 98.347% when pulling 0a7b387 on themeability2 into 303c021 on master.

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

How about showing an example of using those exposed parts in a demo?

Reviewed 4 of 4 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oluwasayo)

a discussion (no related file):
How about showing an example of using those exposed parts in a demo?


Copy link

@alexberazouski alexberazouski left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 4 files at r1.
Reviewable status: 4 of 8 files reviewed, 1 unresolved discussion (waiting on @oluwasayo)

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 4 files at r2.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @oluwasayo)

a discussion (no related file):

Previously, tulioag (Tulio Garcia) wrote…

How about showing an example of using those exposed parts in a demo?

Thanks for the demo. Some thoughts:

  • Shouldn't we keep the basic demo as is and add a demo to the advanced section?
  • I noticed that the "title with custom html" demo also has the image, which is confusing because it doesn't show the code to add it.


demo/custom-brand.jpg, line 0 at r2 (raw file):
This looks nice, thanks. Is this image free for use?

Copy link

@alexberazouski alexberazouski left a comment

Choose a reason for hiding this comment

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

Reviewable status: 6 of 10 files reviewed, 2 unresolved discussions (waiting on @tulioag and @oluwasayo)


demo/custom-brand.jpg, line at r2 (raw file):

Previously, tulioag (Tulio Garcia) wrote…

This looks nice, thanks. Is this image free for use?

The image is taken from bakery

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

Reviewable status: 6 of 10 files reviewed, 1 unresolved discussion (waiting on @tulioag and @oluwasayo)


demo/custom-brand.jpg, line at r2 (raw file):

Previously, alexberazouski wrote…

The image is taken from bakery

Ok.

How did I not notice that? 😩

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 5 files at r3.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oluwasayo)

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oluwasayo)

Copy link
Contributor

@tulioag tulioag left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link

@alexberazouski alexberazouski left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 4 files at r2, 4 of 5 files at r3.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@alexberazouski alexberazouski merged commit 8c19dd3 into master Jan 8, 2019
@alexberazouski alexberazouski deleted the themeability2 branch January 8, 2019 15:47
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.

None yet

4 participants