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

Add ability to configure shadowRadius, shadowOpacity, and shadowColor #294

Closed
wants to merge 2 commits into from
Closed

Conversation

lbasile
Copy link

@lbasile lbasile commented Oct 1, 2014

This also includes an example app update to play with the configurations.

This is not the only open pull request for drawer shadow configuration. However, I didn't see one that also updates the example. We need these for our own app and I suspect other folks out there would love to see this feature addition.

@kcharwood kcharwood added this to the 0.6.0 milestone Mar 10, 2015
kcharwood added a commit that referenced this pull request Mar 11, 2015
Added shadow color property from #294
@kcharwood
Copy link
Contributor

I added in the shadow color method in dd06fde, but I prefer dealing with UIColor so I changed that.

I also didnt take the example changes. I'd prefer to see a slider or a stepper to configure those values, so we can circle back on that.

Thanks!

🍻

@kcharwood kcharwood closed this Mar 11, 2015
@lbasile
Copy link
Author

lbasile commented Mar 11, 2015

Wonderful, glad I could help.

Cheers!

boherna added a commit to boherna/MMDrawerController that referenced this pull request Sep 4, 2015
* upstream/master: (21 commits)
  Fixes mutualmobile#367
  Added notes for reference
  Add missing appearance transition forwarding
  Fixed changelog
  Updated changelog
  Updated changelog
  updated changelog
  Fix infinite, recursive init call for subclass
  Replaced iOS 6 deprecated `-shouldAutorotateToInterfaceOrientation:` method with new one
  Cleaned up PR mutualmobile#288
  Prevent several rotation methods from getting called twice in iOS8, where they are automatically called from [UIViewController viewWillTransitionToSizeWithTransitionCoordinator:]
  Update MMDrawerController.m
  Updated documentation
  Removed pre iOS 7 support
  Switched id to instancetype
  Fix for mutualmobile#287
  Added iPhone 6/6+ support
  Cleaned up setter methods Added shadow color property from mutualmobile#294
  Updated UI properties from mutualmobile#338
  Call removeFromParentViewController after removeFromSuperview in setCenterViewController to avoid EXC_BAD_ACCESS due to nil view value
  ...

Conflicts:
	MMDrawerController/MMDrawerController.h
	MMDrawerController/MMDrawerController.m
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.

2 participants