Skip to content

Let the palette docker update it's palette name upon move.

Wolthera van Hövell requested to merge work/bug-446147 into krita/5.0

This also shifts the isDirty-boolean to be stored inside the palette-resource instead of a boolean elsewhere. Not completely necessary, but it does simplify the code a teensy bit.

BUG:#446147

Test Plan

Just move a swatch in the palette docker. See if the palette name is considered 'dirty'.

Formalities Checklist

  • I confirmed this builds.
  • I confirmed Krita ran and the relevant functions work.
  • [N/A] 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