RBLView and requiresConstraintBasedLayout default setting about rebel HOT 7 OPEN

I would be fine with changing the default, since it was implemented before we ran into some of our own troubles with Auto Layout as well.

It'd be nice to have a consistent subview layout hook on 10.7 and 10.8, though. I've done this with -resizeSubviewsWithOldSize:, but that seems unlikely to work with views that do use Auto Layout.

from rebel.

Lets lay out what I think is happening here:

On OS X 10.8:

• resizeSubviewsWithOldSize: is called without auto layout
• resizeSubviewsWithOldSize: is not called with auto layout
• layout is called without auto layout
• layout is called with auto layout

On OS X 10.7:

• resizeSubviewsWithOldSize: is called without auto layout
• resizeSubviewsWithOldSize: is not called with auto layout
• layout is not called without auto layout
• layout is called with auto layout

Not sure where to go from here. We could do an #ifdef and enable the resizeSubviewsWithOldSize: code if we're on anything less than 10.7. Any ideas?

from rebel.

At this point, the only thing that occurs to me is an overridable method named layoutSubviews, which is invoked from whichever layout hooks we might get (without OS conditionalizing).

That's sucky and confusing, though, and makes me think this responsibility should just lie on the user of the class.

from rebel.

I'm just desperately thinking of ways we could get rid of the enabling of constraints as that causes no end of problems.

from rebel.

Agree here, constraints cause a whole host of problems. Literally no existing code works with constraints layout enabled without some serious tweaking. I think either @jwilling's conditional or just diverting the layout responsibilities from RBLView to the subclass would be the best solutions.

from rebel.

@jwilling I wasn't disagreeing with changing the default return value of that method. I just think there's no way we can compensate for it elsewhere given the shitty layout APIs.

So, let's just remove any default layout code from RBLView.

from rebel.

👍

from rebel.