Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Additional parameters for drawer paddings #26

Closed

Conversation

dulacp
Copy link

@dulacp dulacp commented May 21, 2013

Let us make the drawer controllers a little bit bigger than the maximumLeftDrawerWidth to cover round corners of a central controller for instance with the background of the drawer.

let us make the drawer a little bit bigger than the
`maximumLeftDrawerWidth` to cover round corners of a central controller
for instance
@kcharwood
Copy link
Contributor

Thanks for pull request the @dulaccc!

I've been chewing on this feature since we before we open sourced it, and haven't quite landed on how I would like to handle it. I would really like to avoid adding more properties if possible...

I've been brainstorming if there is way to handle this automatically for the user (i.e. inspect the center view controller to find out if there are rounded corners, and automatically adjust based on that, but I haven't had time to research it).

Also, if you look at the Facebook app, they don't handle this correctly (you'll see a tiny black spec under the rounded corner).

Let me think about this some more.

🍻

@dulacp
Copy link
Author

dulacp commented May 22, 2013

So glad to hear to that you've been thinking about it too :)

Just to saying, thanks for your amazing work, that's clearly the best implementation of a "Facebook/Path 2.0" panel-like interface I've found, and I tested a lot of them!

I will check if I can make the PR without any extra parameters, I agree it'd be a lot better.

@larsacus
Copy link
Contributor

Dare I say this might be best handled as a subclass of MMDrawerController to avoid more properties being added to the core.

@kcharwood
Copy link
Contributor

I'm going to close this for now. It's probably best handled in a subclass, which we need to provide better documented support for.

If more people clamor for this, we'll consider it for a first class citizen feature in the library, but so far 3 months in there has not been much demand for it.

@kcharwood kcharwood closed this Aug 13, 2013
@dulacp
Copy link
Author

dulacp commented Aug 14, 2013

Fair enough. I'll use now the subclass solution too.
Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants