search

alexrothenberg / motion-addressbook

MIT License
89 stars 30 forks source link

Add :prefix property #39

Closed wejn closed 11 years ago

wejn commented 11 years ago

Hi, I've noticed the addressbook implementation on iOS is missing :prefix property, although the docs have it with the same compatibility as :suffix.

So I took the liberty of adding it.

I've also pushed it into OSX but I have no [easy] way of testing it works.

alexrothenberg commented 11 years ago

You're right it should have been there all along. Thanks for adding it.

wejn commented 11 years ago

Thanks.

Any chance for pushing out minor version of the gem (1.4.1?) incorporating this commit? I'd rather not "vendor" it in my app unless absolutely necessary.

jmay commented 11 years ago

OSX calls this KABTitleProperty instead of Prefix. I think we should stick with prefix as the standard attribute name. I'll add a test case.