Reason behind the existence of `contentLayer` about shimmer HOT 5 CLOSED

Objective-C usually favors composition over subclassing. With the current approach, you can add shimmering to any existing view or layer, without having to make a subclass of it.

Using composition can be especially useful when you don't control the creation of the view hierarchy. For example, you could add a shimmering effect to an MPMoviePlayerController's view, which is created and managed by iOS itself.

You can still use it with any view or layer you create yourself as long as you put the contents inside a single view or layer. Then, internally set that as the contentLayer.

from shimmer.

Sorry if I’ll look dumb, but I really don’t get it.

Using composition can be especially useful when you don't control the creation of the view hierarchy. For example, you could add a shimmering effect to an MPMoviePlayerController's view, which is created and managed by iOS itself.

You’re still going to create a view (whose .layer will be a FBShimmeringLayer) to contain the MPMoviePlayerController whose view’s layer will be used as a contentLayer. Even using -setLayer: you could set an ‘auto-shimmering’ layer to it (I mean, the container).

You can still use it with any view or layer you create yourself as long as you put the contents inside a single view or layer. Then, internally set that as the contentLayer.

That’s exactly what I tried to do, but my special environment has spit on me. (Just for clarity, I’m porting Shimmer to Titanium SDK, where views lay in a very special way, and adding middle-views without breaking the layout is not that trivial.)

from shimmer.

I see what you mean now, the shimmering layer could shimmer all of its children, rather than just the contentLayer. That's an interesting idea — I would suggest forking the code and modifying it as you need for that, since it's likely you'll need other changes as well to work outside the standard Cocoa Touch environment.

from shimmer.

Actually I just made that changes, it’s just a fact of removing everything contentLayer related. To port that on Titanium I just had to clone what you did on FBShimmeringView, setting the +layerClass.

Would you mind if I make a PR for you to explore the idea?

from shimmer.

(Oh, by the way, your code is really self explanatory. Awesome component perfectly executed)

from shimmer.