Let pack developers choose the item capture layout for attachable notes (#106)

Allow pack developers to point attachable markdown notes' item capture box at
a grid other than the hardcoded tracker_capture_item.

- Add pack-wide default: DefaultItemCaptureLayout on TrackerState, sourced
  from settings.json's capture_item_layout_default (fallback
  tracker_capture_item), reset on each pack load.
- Add Location-level ItemCaptureLayout with child-Location inheritance
  (nearest explicit override wins, else pack default, else
  tracker_capture_item), parsed from the locations capture_item_layout key.
- Resolve MarkdownTextWithItemsNote.ItemCaptureLayout through its owning
  Location (or the state pack default for non-Location sites) instead of the
  hardcoded string.

No user-facing picker; value is pack-config only. Existing packs/notes
without the keys fall back to tracker_capture_item.

Author: EmoSaru

EmoTracker.Data/LocationDatabase.cs

@@ -643,6 +643,8 @@ Location LoadLocation(IGamePackage package, Location parent, JObject data, Sessi
             if (!string.IsNullOrWhiteSpace(color))
                 instance.Color = color;
 
+            instance.ItemCaptureLayout = data.GetValue<string>("capture_item_layout");
+
             JArray rules = data.GetValue<JArray>("access_rules");
             if (rules != null)
             {

EmoTracker.Data/Locations/Location.cs

@@ -115,6 +115,31 @@ public string Color
             set { ColorRaw = value; NotifyPropertyChanged(); }
         }
 
+        // Item capture layout: per-location override, inherited by child
+        // Locations until overridden deeper in the hierarchy. Used by
+        // attachable markdown notes to resolve the item grid they present.
+        [KVMutable]
+        private partial string ItemCaptureLayoutRaw { get; set; }
+
+        public string ItemCaptureLayout
+        {
+            get
+            {
+                var raw = ItemCaptureLayoutRaw;
+                if (!string.IsNullOrWhiteSpace(raw)) return raw;
+
+                var parent = Parent;
+                if (parent != null) return parent.ItemCaptureLayout;
+
+                var state = this.OwnerState as Sessions.TrackerState;
+                var packDefault = state?.DefaultItemCaptureLayout;
+                if (!string.IsNullOrWhiteSpace(packDefault)) return packDefault;
+
+                return "tracker_capture_item";
+            }
+            set { ItemCaptureLayoutRaw = value; NotifyPropertyChanged(); }
+        }
+
         [KVMutable]
         public partial ImageReference Thumbnail { get; set; }
 

EmoTracker.Data/Notes/MarkdownTextWithItemsNote.cs

@@ -1,15 +1,29 @@
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
+using EmoTracker.Data.Locations;
+using EmoTracker.Data.Sessions;
 
 namespace EmoTracker.Data.Notes
 {
     public class MarkdownTextWithItemsNote : MarkdownTextNote, IItemCollection
     {
         ObservableCollection<ITrackableItem> mItems = new ObservableCollection<ITrackableItem>();
 
+        internal NoteTakingSite Site { get; set; }
+
         public string ItemCaptureLayout
         {
-            get { return "tracker_capture_item"; }
+            get
+            {
+                if (Site?.Owner is Location loc)
+                    return loc.ItemCaptureLayout;
+
+                var state = Site?.OwnerState as TrackerState;
+                var packDefault = state?.DefaultItemCaptureLayout;
+                if (!string.IsNullOrWhiteSpace(packDefault)) return packDefault;
+
+                return "tracker_capture_item";
+            }
         }
 
         public IEnumerable<ITrackableItem> Items

EmoTracker.Data/Notes/NoteTakingSite.cs

@@ -26,7 +26,7 @@ public void SetOwnerState(Sessions.TrackerState state)
             mOwnerStateOverride = state;
         }
 
-        Sessions.TrackerState OwnerState
+        internal Sessions.TrackerState OwnerState
             => mOwnerStateOverride ?? (Owner?.OwnerState as Sessions.TrackerState);
         ObservableCollection<Note> mNotes = new ObservableCollection<Note>();
 
@@ -51,6 +51,8 @@ public bool AddNote(Note note)
             {
                 note.PropertyChanged += Note_PropertyChanged;
                 mNotes.Add(note);
+                if (note is MarkdownTextWithItemsNote itemNote)
+                    itemNote.Site = this;
                 return true;
             }
 

EmoTracker.Data/Sessions/PackageLoader.cs

@@ -235,6 +235,7 @@ static void ResetPackageSettings(TrackerState target)
         {
             target.AllowResize = true;
             target.DisabledImageFilterSpec = TrackerDefaults.DisabledImageFilterSpec;
+            target.DefaultItemCaptureLayout = TrackerDefaults.ItemCaptureLayout;
         }
 
         // Mirrors Tracker.LoadPackageSettings, but operates on the target's
@@ -262,6 +263,10 @@ static void LoadPackageSettings(TrackerState target, IGamePackage package)
                                 if (spec != null)
                                     target.DisabledImageFilterSpec = spec;
 
+                                string defaultLayout = root.GetValue<string>("capture_item_layout_default", null);
+                                if (!string.IsNullOrWhiteSpace(defaultLayout))
+                                    target.DefaultItemCaptureLayout = defaultLayout;
+
                                 target.Locations.ParseLocationVisualProperties(root, target.Locations.Root, package);
 
                                 AccessibilityRule.EnableCache = root.GetValue<bool>("enable_accessibility_rule_caching", true);
@@ -313,5 +318,6 @@ static void LoadPackageSettings(TrackerState target, IGamePackage package)
     static class TrackerDefaults
     {
         public const string DisabledImageFilterSpec = "grayscale, dim";
+        public const string ItemCaptureLayout = "tracker_capture_item";
     }
 }

EmoTracker.Data/Sessions/TrackerState.PackageOps.cs

@@ -71,6 +71,20 @@ public bool AllowResize
             set { SetProperty(ref mAllowResize, value); }
         }
 
+        string mDefaultItemCaptureLayout = "tracker_capture_item";
+        /// <summary>
+        /// Pack-wide default item-capture layout grid used by attachable
+        /// markdown notes that aren't covered by a Location-level
+        /// <c>capture_item_layout</c>. Set by <c>settings.json</c>'s
+        /// <c>capture_item_layout_default</c> key during pack load; falls
+        /// back to <c>"tracker_capture_item"</c>.
+        /// </summary>
+        public string DefaultItemCaptureLayout
+        {
+            get => mDefaultItemCaptureLayout;
+            set { SetProperty(ref mDefaultItemCaptureLayout, value); }
+        }
+
         // ---- Reload / Activate ------------------------------------------
 
         bool mbReloadInProgress;