Skip to content

Conversation

jelbourn
Copy link
Member

@googlebot googlebot added cla: yes PR author has agreed to Google's Contributor License Agreement labels Aug 12, 2016
</md-toolbar>`
})

@Component({template: `<md-toolbar [color]="toolbarColor"><span>Test Toolbar</span></md-toolbar>`})
Copy link
Member

Choose a reason for hiding this comment

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

Not necessary, but since the template is now in a single line, we could probably remove the span element to make the template more clean.

@devversion
Copy link
Member

devversion commented Aug 12, 2016

LGTM

I've also added two minor comments, which could be changed, while your at it

FYI: It seems like there is an exception in the Travis Build (but seems to be flaky)
https://travis-ci.org/angular/material2/jobs/151702358#L431

@jelbourn jelbourn merged commit 708662d into angular:master Aug 12, 2016
@jelbourn jelbourn deleted the remove-tcb-toolbar branch September 13, 2017 04:36
@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 7, 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.

3 participants