Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added Subclass Documentation/Headers #37

Merged
merged 8 commits into from
Aug 16, 2013
Merged

Added Subclass Documentation/Headers #37

merged 8 commits into from
Aug 16, 2013

Conversation

larsacus
Copy link
Contributor

I've added some basic support for making it easier for devs to subclass MMDrawerController (#36) and handle some of the more specific requests that I've been seeing.

I'd like to wait until at least #20 is merged before merging this.

This would make #26 & #31 pretty easy for a developer.

This is a start to exposing some of the more "private" methods of MMDrawerController to a subclass. If anyone feels any of the exposed methods should be removed or some should be added feel free to chime in.

@kcharwood
Copy link
Contributor

I'd rather start small here, and move forward with people's specific needs, so I removed a few things that seemed overkill right now.

If we see people with specific needs that could use access to specific methods, we can expand it in the future.

kcharwood added a commit that referenced this pull request Aug 16, 2013
Added Subclass Documentation/Headers
@kcharwood kcharwood merged commit 6748a8b into master Aug 16, 2013
@kcharwood kcharwood deleted the subclass branch August 16, 2013 16:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants