sberrevoets / SDCAlertView

The little alert that could
MIT License
1.86k stars 297 forks source link

ActionSheet customization, improved dark mode #303

Closed futuretap closed 4 years ago

futuretap commented 4 years ago

This PR squashes all commits of #285 into one clean commit.

sberrevoets commented 4 years ago

Thanks for this! I merged and then tweaked some stuff so I can keep track of what's going on in the future. I changed 2 things:

  1. I changed the API and developer expectations around the image/accessory views and brought them more in line with how UITableViewCell works. The image view is now provided by the framework and the developer can only set an image on it and provide constraints. I also made the layout for these two a bit more explicit, messing with existing constraints on a view that wasn't created by the framework felt a bit odd.

  2. Removed the cellLabelAlignment bit because the interaction felt a bit awkward because it had to be set for all actions at once and didn't apply to alert views. If this break some use case, let me know what it is and I can try to see if there's a nicer way to support this.

Let me know if the changes still support your use case and I will cut a new release.

gereons commented 4 years ago

I can't speak for @futuretap obviously, but I'd love to see this released :)

futuretap commented 4 years ago

Thanks, @sberrevoets, this looks good, I appreciate your changes to make the API cleaner. I personally don't need cellLabelAlignment, so I'm fine with removing it.