New option and minor fixes for animated GIF export
This patch includes the following:
- Option to toggle reserving a color for GIF transparency
- Warn about transparency not working if user has an old FFMpeg version and trying to export a GIF
- Cleanup the palette files after rendering is done
- Fixed an error with exporting in single palette per frame option
- Displays the FFMpeg version user selects
Test Plan
- Create an animation with a transparent background and then Render Animation
- Select GIF and FFMpeg version that is below 4.2, the version should show up alongside a warning about transparency not working
- Now select a version of FFMpeg that is 4.2 or above, the warning should vanish
- Click the options and select Per Single Frame under palette, then render the animation
- Check the output location, the gif should have been rendered with no transparency and no palette pngs should be left behind
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.
Edited by Know Zero