Fix five review issues: shortcut hints, ItemsDictionary duplicates, null ItemsSource, duplicate chip handlers, Freeform key hint

[Copilot]

Five correctness issues flagged in code review of the grab-templates PR, covering misleading UI hints, data duplication, and defensive null/event-handler handling.

Changes

• FullscreenGrab.xaml.cs – AddPostGrabActionMenuItem: Cap InputGestureText to Ctrl+1–Ctrl+9; omit for indices > 9 since no keyboard handler exists beyond Key.D9.

  InputGestureText = shortcutIndex <= 9 ? $"Ctrl+{shortcutIndex}" : string.Empty

• QuickSimpleLookup.xaml.cs – LoadDataGridContent: Add ItemsDictionary.Clear() before AddRange; without it, re-invoking the method (e.g. via "Import from CSV") duplicates all rows, history, and templates.

• InlinePickerRichTextBox.cs – ItemsSource getter: Guard against the DP returning null before the property is ever set — previously threw NullReferenceException.

  get => (IEnumerable<InlinePickerItem>?)GetValue(ItemsSourceProperty) ?? [];

• InlineChipElement.cs – OnApplyTemplate: Store PART_RemoveButton in a field and detach the previous Click handler before re-attaching via a named method. Lambda-per-call caused multiple RemoveRequested fires on a single click after theme/template switches.

• FullscreenGrab.xaml – Freeform menu item: Correct InputGestureText from "P" to "D" to match GetSelectionStyleKey returning Key.D for FsgSelectionStyle.Freeform.

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.