Conversation
@@ -6,9 +6,10 @@ | |||
<!DOCTYPE html> |
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.
I'd change this and remove the DOCTYPE, html, head, and body tags so it simply looks like:
<link rel="import" href="packages/polymer/polymer.html">
<polymer-element name="tute-slambook-form" extends="form">
...
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.
(but yes, what you currently have is correct. Just a bit more verbose :))
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.
Done
lgtm - just a minor suggestion |
A few suggestions about comments, nothing of them deal breaking or earth shaking. |
It was hard to tell, but did you delete the |
|
||
<html> | ||
<body> | ||
<link rel="import" href="../packages/polymer/polymer.html"> |
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.
minor nit - now that <html><body>
are gone in all these files, you might want to unindent everything.
Updating the polymer dart examples to 0.15.1 and incorporating Siggi's comments. Also updating the comment style to use "///" instead of "/*" for doc comments.
@kwalrath @sigmundch