Skip to content

Conversation

kara
Copy link
Contributor

@kara kara commented Jan 27, 2016

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jan 27, 2016
Copy link
Member

Choose a reason for hiding this comment

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

+1 space between property and value.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, updating

Copy link
Member

Choose a reason for hiding this comment

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

The second param for md-color doesn't actually need quotes, I believe.

@kara kara force-pushed the card branch 2 times, most recently from 9594f47 to 9466869 Compare January 27, 2016 18:42
@kara kara mentioned this pull request Jan 27, 2016
4 tasks
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we put the templates in separate files?

@kara kara force-pushed the card branch 2 times, most recently from 0849035 to 41e7cc2 Compare January 27, 2016 21:20
@kara
Copy link
Contributor Author

kara commented Jan 27, 2016

@jelbourn @hansl Ready for a second pass!

Copy link
Member

Choose a reason for hiding this comment

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

+1 space between ) and {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Member

Choose a reason for hiding this comment

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

Add

TODO(kara): update link to demo site when it exists

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added!

@kara kara force-pushed the card branch 2 times, most recently from 7429d00 to 6612669 Compare January 28, 2016 19:39
@jelbourn
Copy link
Member

LGTM

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes PR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants