Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Several bug fixes. Closes #11, #12, #19, #23 and #46 #50

Merged
merged 9 commits into from
Aug 16, 2013

Conversation

efullea
Copy link
Member

@efullea efullea commented Aug 14, 2013

No description provided.

@@ -317,10 +319,12 @@
<section>
<h2><a>ContactField</a> Interface</h2>
<p>The <a>ContactField</a> interface represents a user's attribute and the types associated to it.
<dl title="[Constructor(DOMString[] types, DOMString value)] interface ContactField"
<dl title="[Constructor(DOMString[] types, boolean preferred, DOMString value)] interface ContactField"
Copy link

Choose a reason for hiding this comment

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

Personally, I would move preferred to 3rd place and make it optional (defaulting to false).

@cdumez
Copy link

cdumez commented Aug 16, 2013

I made a few comments but the change looks great overall!

@efullea
Copy link
Member Author

efullea commented Aug 16, 2013

Chris' comments addressed

efullea pushed a commit that referenced this pull request Aug 16, 2013
@efullea efullea merged commit ead0150 into sysapps:gh-pages Aug 16, 2013
@efullea efullea deleted the contactproperties branch August 16, 2013 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants