search

JRG-Developer / AutoLayoutCells

AutoLayoutCells makes working with dynamic table view cells easy.
MIT License
26 stars 4 forks source link

Revised ALTableViewCellFactory #7

Closed JRG-Developer closed 9 years ago

JRG-Developer commented 9 years ago

@AnthonyMDev

Please see the latest development branch commit and message.

I've revised ALTableViewCellFactory to reduce/completely eliminate boiler plate code. I think this is a great improvement, but it will require code modifications in your consuming projects (you'll most likely be deleting more code than having to modify. :)

Please see the demo project too.

Please provide feedback/questions that you have as I plan to create a new version with this soon.

:]

Thanks! Joshua

AnthonyMDev commented 9 years ago

@JRG-Developer Sorry I didn't get to this yet. I was out of town last week. I'll try to take a look later today and provide feedback!

AnthonyMDev commented 9 years ago

This looks great. I definitely see the value in the changes you've made. I have no problem with pushing this up to the trunk as a new version.

AnthonyMDev commented 9 years ago

This actually also fixes another problem I ran into today. If I have multiple table views in the same view controller (such as when using a UISearchDisplayController in a table view), I need to use multiple ALTableViewCellFactory instances, for the different table views, however the configureCell: atIndexPath: method does not allow me to determine which table view I'm configuring a cell for.

This new way of doing this fixes that problem for me. So I actually need this ASAP.

Thanks @JRG-Developer