Skip to content
This repository has been archived by the owner on Mar 13, 2018. It is now read-only.

Documentation for attributes #2

Closed
wants to merge 1 commit into from

Conversation

nevir
Copy link

@nevir nevir commented May 24, 2014

The interaction between tallMode and minHeaderHeight was a bit confusing - what do you think about renaming them to condenses and condensedHeaderHeight?

Otherwise, here's some quick documentation; I think this follows the latest style you folk are using for attribute docs?

@nevir nevir changed the title Documentation for attributes. Documentation for attributes May 24, 2014
* Attributes moved to a publish block
* tallMode -> condenses, minHeaderHeight -> condensedHeaderHeight
@nevir nevir added the cla: yes label May 26, 2014
@ebidel
Copy link
Contributor

ebidel commented May 27, 2014

Thanks for the docs! cc @frankiefu to take a look on the API changes.

@frankiefu
Copy link
Contributor

Yeah I like the suggestions. Thanks for the docs.

@ebidel
Copy link
Contributor

ebidel commented May 28, 2014

@frankiefu has been busy. It looks like this will need rebasing

frankiefu added a commit that referenced this pull request Jun 2, 2014
@frankiefu
Copy link
Contributor

merged in here: dcad5de

@frankiefu frankiefu closed this Jun 2, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants