Merge pull request #1437 from golamrabbiazad/master

fix: pinned task position captured
alainm23 · Sep 16, 2024 · eac1516 · eac1516
2 parents 1febec3 + 1205a75
commit eac1516
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/src/Widgets/PinnedItemsFlowBox.vala b/src/Widgets/PinnedItemsFlowBox.vala
@@ -66,15 +66,11 @@ public class Widgets.PinnedItemsFlowBox : Adw.Bin {
  });
 
  project.item_deleted.connect ((item) => {
- if (!items_map.has_key (item.id)) {
- return;
- }
+ update_pinboard (item);
+ });
 
- items_map[item.id].hide_widget ();
- box_layout.remove (items_map[item.id]);
- items_map.unset (item.id);
-
- check_reveal_child ();
+ Services.EventBus.get_default ().item_moved.connect ((item) => {
+ update_pinboard (item);
  });
 
  Services.Store.instance ().item_pin_change.connect ((item) => {
@@ -137,7 +133,19 @@ public class Widgets.PinnedItemsFlowBox : Adw.Bin {
  }
  }
 
+ private void update_pinboard (Objects.Item item) {
+ if (!items_map.has_key (item.id)) {
+ return;
+ }
+
+ items_map[item.id].hide_widget ();
+ box_layout.remove (items_map[item.id]);
+ items_map.unset (item.id);
+
+ check_reveal_child ();
+ }
+
  private void check_reveal_child () {
  main_revealer.reveal_child = items_map.size > 0;
  }
-}
+}