Don't set track id to 0 when importing frames

Because that's not a valid track id and it messes things up.
drawpile · Aug 9, 2024 · aead7ac · aead7ac
1 parent e3e04b6
commit aead7ac
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -3,6 +3,7 @@ Unreleased Version 2.2.2-pre
  * Fix: Don't try to update the roles list before becoming an operator, which could lead to a crash. Thanks Bluestrings for reporting.
  * Fix: Don't update sessions that were just added to the session browser, since that can lead to a crash because of what is probably a bug in Qt's filtering and sorting. Thanks Bluestrings for reporting.
  * Feature: Turn too large transforms into cut and paste operations instead of cancelling them outright.
+ * Fix: Make frame import not create a track with an invalid ID that messes up the UI. Thanks BulletPepper and greendyno for reporting.
 
 2024-08-04 Version 2.2.2-beta.2
  * Feature: Allow choosing a different cursor for erase and alpha locked brushes. Thanks Hipofiz and Rylan for suggesting.

diff --git a/src/drawdance/libimpex/dpimpex/load_animation.c b/src/drawdance/libimpex/dpimpex/load_animation.c
@@ -74,6 +74,7 @@ DP_CanvasState *DP_load_animation_frames(
             child_tll = DP_transient_layer_list_new_init(path_count);
             child_tlpl = DP_transient_layer_props_list_new_init(path_count);
             tt = DP_transient_track_new_init(path_count);
+            DP_transient_track_id_set(tt, BASE_LAYER_ID);
             set_track_title(tt, 0);
         }