bindViewModel has wrong cellSize

[maxifuchs]

General information

• IGListKit version: 3.4.0
• iOS version(s): 11.4
• Xcode version: 9.4.1
• Reproducible in the demo project?: Yes

Hello,
I think I found a bug: If you call contentView.frame.size inside your bindViewModel method, it returns the size of the xib (in storyboard), not the size of the cell, which was specified in the sectionController(_ sectionController: ListBindingSectionController<ListDiffable>, sizeForViewModel viewModel: Any, at index: Int).
This bug also occurs in the IGListKit-Binding-Guide example project.

The cell still has the right size (from sizeForViewModel), but I’m creating a MKSnapshotter inside bindViewModel, which unfortunately uses the wrong size of the contentView.

In my project, the bug only occurs when the cell is loaded for the first time. After that, contentView.frame.size returns the correct size.

[rnystrom]

This is probably an issue with how XIBs work. We don’t use them at all in Instagram so I’m not surprised this hasn’t come up.

Can you repro the issue in an example app or better-yet unit test?

I won’t be able to work on a fix for this, but I’d gladly accept a PR!

_{Sent with GitHawk}

[mstafakv]

https://github.com/thelinuxchoice/choicebot