search

mkatch / apricot

Cross platform pixel art editor
2 stars 0 forks source link

Add onion skin display to AnimationFrameView #19

Closed mkatch closed 10 years ago

mkatch commented 10 years ago

AnimationFrameView should be able to display a configurable amount of frames back and ahead of the currently set frame. Onion skinning on Wikipedia.

mkatch commented 10 years ago

The assembling of translucent frames should be entirely done in AnimationFrameView, not in AnimationView. The only reason for involving AnimationView in onion skinning would be to embed the controls for changing the onion skinning parameters.

But for now, remove any onion skinning related code from AnimationView. The AnimationFrameView should provide mutable properties for setting the number of forward and backward frames displayed. It should assemble it's list by itself, not relying on AnimationVeiw to provide it.

Also I left some comments at a70c6a97129c31618e9a4d1a26bf137577498822.

mkatch commented 10 years ago

AnimationFrameView should have onionSkinBackward and onionSkinForward properties of type int and public setter slots associated with them.