Skip to content

Animation: Autopin

Emmet O'Neill requested to merge (removed):ani/pin into master

(These two commits are in one MR because one builds on top of the other.)

  • "Show in timeline" language has been changed to "Pin to timeline" across both the code and the user interface. Adding a layer to the timeline should now be referred to as "pinning" and removing it should be referred to as "unpinning". This should hopefully make things more clear for users. BUG:407416

  • New option: "automatically pin new layers to timeline". Users can now decided whether they want new layers to be pinned by default or not. Added a user setting for Auto-Pinning paint layers to the animation docker to general preferences > miscellaneous. BUG:414690 BUG:413805

image

Test

Test creation of layers with the automatic pinning option on and off.

Formalities Checklist

  • I confirmed this builds.
  • I confirmed Krita ran and the relevant functions work.
  • I tested the relevant unit tests and can confirm they are not broken. (If not possible, don't hesitate to ask for help!)
  • I made sure my commits build individually and have good descriptions as per KDE guidelines.
  • I made sure my code conforms to the standards set in the HACKING file.
  • I can confirm the code is licensed and attributed appropriately, and that unattributed code is mine, as per KDE Licensing Policy.

Merge request reports