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
Feature/2.1 upgrade #143
Feature/2.1 upgrade #143
Conversation
@@ -19,84 +44,55 @@ <h4 class="md-subhead">with custom headings, columns, and inline editing</h4> | |||
<th td-data-table-column | |||
*ngFor="let column of columns" | |||
[numeric]="column.numeric"> | |||
{{column.label}} | |||
{ {column.label}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formatting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its an angular2 issue where if it finds {{
anywhere in an html file, it passes it to its compiler.
So we need to add a space between them to be able to render it in the highlight component.
</th> | ||
<tr td-data-table-row *ngFor="let row of basicData"> | ||
<td td-data-table-cell (click)="openPrompt(row, 'comments')"> | ||
<button md-button [class.md-accent]="!row['comments']">{{row['comments'] || 'Add Comment'}}</button> | ||
<button md-button [class.md-accent]="!row['comments']">{ {row['comments'] || 'Add Comment'}}</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here also
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup good catch but like Ed said we have to do this to render code snippets in highlight.js for docs
</td> | ||
<td td-data-table-cell *ngFor="let column of columns" [numeric]="column.numeric"> | ||
{{row[column.name]}} | ||
{ {row[column.name]}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here
Description
Upgrade @angular dependencies
Fixes #141
What's included?
angular
dependencies to@2.2.1
zone.js
to@0.6.25
angular-cli
tobeta.19-3
material
toalpha.10
and fixed breaking changes https://github.com/angular/material2/blob/master/CHANGELOG.md#200-alpha10-mithril-hoverboard-2016-11-10[attr.disabled]
with no value errors in 2.1.1 angular/angular#12583Test Steps
npm uninstall -g angular-cli
npm cache clean
npm install -g angular-cli@latest
rm -rf node_modules dist tmp
npm i
ng serve