feat(weather editor): add delete json button to objects window

[SkrubbySkrubInAShrub]

• Shows which objects have a json file active
• Allows deletion of the json file without openign the object widget
• Sorting functionality

Summary by CodeRabbit

• New Features

  • JSON attachment management with per-widget caching and visual indicators in the widget list
  • New JSON column in the widget details table for visibility
  • JSON-aware sorting to prioritize widgets with attachments

• Bug Fixes / Improvements

  • Inline JSON delete action with confirmation modal and improved delete workflow
  • Cache invalidation on resource updates and after widget changes to keep state consistent
  • Improved multi-column row interactions and delete-button styling for more reliable UX

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds a per-widget JSON-attachment cache, a new JSON column and delete button in the details table, JSON-aware sorting, a global confirmation modal for JSON deletion, cache invalidation hooks, and widget notifications when their JSON attachment changes.

Changes

Cohort / File(s)	Summary
EditorWindow core & cache src/WeatherEditor/EditorWindow.h, src/WeatherEditor/EditorWindow.cpp	Added jsonAttachmentCache (std::unordered_map<Widget*, bool>), methods RefreshJsonAttachmentCache, HasCachedJsonAttachment, InvalidateJsonAttachmentCache, OnWidgetJsonAttachmentChanged, pendingDeleteWidget/pendingDeletePopupRequested. Invalidate cache in SetupResources() and call refresh before sorting.
Details table & sorting src/WeatherEditor/EditorWindow.cpp	Expanded details table from 5→6 columns (new JSON column), introduced enum-based ColumnID and SortColumn::JsonAttachment, wired sort mapping from ImGui specs to new column, updated sorting to prioritize cached JSON attachments, added JSON delete button rendering and ImGuiSelectableFlags_AllowOverlap adjustments.
Widget deletion & modal API src/WeatherEditor/Widget.h, src/WeatherEditor/Widget.cpp	Changed DrawDeleteConfirmationModal() → DrawDeleteConfirmationModal(const char* popupId = "DeleteConfirmation"); widgets notify editor via EditorWindow::GetSingleton()->OnWidgetJsonAttachmentChanged(this) after save/delete; refactored delete button styling to Util::ErrorButtonStyle().
UI flow & modal src/WeatherEditor/EditorWindow.cpp	Added global JSON delete confirmation modal (ListDeleteConfirmation), pending delete state flow, and reset logic after modal closes.

Sequence Diagram

sequenceDiagram
    participant User
    participant EditorWindow as EditorWindow UI
    participant Widget
    participant JsonCache as JsonCache

    User->>EditorWindow: Click JSON delete button
    EditorWindow->>EditorWindow: Set pendingDeleteWidget + open modal
    User->>EditorWindow: Confirm deletion
    EditorWindow->>Widget: Invoke delete JSON attachment
    Widget->>Widget: Remove saved JSON state
    Widget->>EditorWindow: OnWidgetJsonAttachmentChanged(widget)
    EditorWindow->>JsonCache: InvalidateJsonAttachmentCache(widget)
    EditorWindow->>EditorWindow: RefreshJsonAttachmentCache(sortedWidgets) / Refresh UI

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

• fix(weather editor): clear feature overrides on revert button #1884 — Modifies widget deletion/notification flow and delete/modal signatures in Widget.cpp/.h, closely related to the deletion/notification changes here.

Suggested reviewers

• davo0411
• doodlum

Poem

🐰 I nibbled bytes and stored a tiny trace,
A JSON crumb behind each widget's face.
A button blinks — confirm, then hop away,
The cache forgets, the lists are neat today.
Hooray for tidy fields and clearer play!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 6.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main feature: adding a delete JSON button to the objects window. This aligns with the primary changes of introducing a new JSON delete button column, cache mechanism for JSON attachments, and sorting functionality.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Using provided base ref: e825ccc
Using base ref: e825ccc
Base commit date: 2026-02-20T19:54:03+10:00 (Friday, February 20, 2026 07:54 PM)
No actionable suggestions for changed features.

[coderabbitai]

Actionable comments posted: 3

🧹 Nitpick comments (3)

src/WeatherEditor/EditorWindow.cpp (2)

134-706: Conventional commit: use lowercase description and consider adding a scope.

Per project guidelines, commit titles should use a lowercase description. The PR title also benefits from a scope to make it easier to search:

feat(weather-editor): add delete json button to objects window

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.cpp` around lines 134 - 706, The commit
message uses an uppercase description and lacks a scope; update the commit/title
to follow conventional commits with a lowercase description and a scope—for
example change the PR title to "feat(weather-editor): add delete json button to
objects window" (apply when creating the commit for changes in
EditorWindow::ShowObjectsWindow or related modifications) so it matches project
guidelines and improves searchability.

---

528-543: Duplicate json/delete column rendering — extract to a lambda.

These two blocks (lines 528–543 for the "current cell's lighting template" path and lines 666–681 for the regular list) are identical except for the ImageButton label prefix. Any future change (icon size, tooltip text, style) must be applied twice.

♻️ Suggested refactor — inline lambda before the table iteration

auto drawJsonDeleteColumn = [&](Widget* w, const char* idPrefix) {
    ImGui::TableNextColumn();
    if (HasCachedJsonAttachment(w)) {
        auto* menu = globals::menu;
        if (menu && menu->uiIcons.deleteSettings.texture) {
            const float iconSize = ImGui::GetFrameHeight() * 0.85f;
            {
                auto _style = Util::ErrorButtonStyle();
                ImGui::SetNextItemAllowOverlap();
                if (ImGui::ImageButton(std::format("{}_{}", idPrefix, w->GetFormID()).c_str(),
                        menu->uiIcons.deleteSettings.texture, { iconSize, iconSize }))
                    pendingDeleteWidget = w;
            }
            if (ImGui::IsItemHovered())
                ImGui::SetTooltip("Delete json file");
        }
    }
};

Then replace both blocks:

- // json / delete column
- ImGui::TableNextColumn();
- if (HasCachedJsonAttachment(sortedWidgets[i])) {
-     ...
- }
+ drawJsonDeleteColumn(sortedWidgets[i], "##jsondel_cur");   // current-cell path
// and
+ drawJsonDeleteColumn(sortedWidgets[i], "##jsondel");       // regular list path

Also applies to: 666-681

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.cpp` around lines 528 - 543, Extract the
duplicated JSON/delete column rendering into a local lambda (e.g.,
drawJsonDeleteColumn) that takes the Widget* (or Widget& w) and a const char*
idPrefix, and inside it call ImGui::TableNextColumn(), check
HasCachedJsonAttachment(w), read globals::menu and
menu->uiIcons.deleteSettings.texture, compute iconSize via
ImGui::GetFrameHeight() * 0.85f, scope Util::ErrorButtonStyle(), call
ImGui::SetNextItemAllowOverlap(), and call ImGui::ImageButton with
std::format("{}_{}", idPrefix, w->GetFormID()).c_str() to set
pendingDeleteWidget on click and show the same tooltip when hovered; then
replace both duplicated blocks (the one using the "current" label and the
regular list) with calls to drawJsonDeleteColumn(sortedWidgets[i],
"##jsondel_cur") and drawJsonDeleteColumn(sortedWidgets[i], "##jsondel") (or the
exact prefixes used) respectively.

src/WeatherEditor/EditorWindow.h (1)

208-216: Fragile SortColumn-to-column-index coupling via static_cast.

EditorWindow.cpp line 292 converts spec.ColumnIndex directly: currentSortColumn = static_cast<SortColumn>(spec.ColumnIndex). This works today because the enum values exactly match the column indices (0–5), but it silently breaks if any column is inserted, removed, or reordered.

Note: this also fixes a pre-existing bug where SortColumn::Status was absent — sorting by that column previously fell through to default: break and did nothing.

♻️ Suggested approach: map column index via a lookup table

- currentSortColumn = static_cast<SortColumn>(spec.ColumnIndex);
+ static constexpr SortColumn kColumnToSort[] = {
+     SortColumn::None,          // 0 – Fav (NoSort)
+     SortColumn::EditorID,      // 1
+     SortColumn::FormID,        // 2
+     SortColumn::File,          // 3
+     SortColumn::Status,        // 4
+     SortColumn::JsonAttachment, // 5
+ };
+ if (spec.ColumnIndex < IM_ARRAYSIZE(kColumnToSort))
+     currentSortColumn = kColumnToSort[spec.ColumnIndex];
+ else
+     currentSortColumn = SortColumn::None;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.h` around lines 208 - 216, The code currently
relies on a fragile static_cast from spec.ColumnIndex to the SortColumn enum
(SortColumn) when assigning currentSortColumn, which breaks if columns are
reordered or changed; replace the static_cast with an explicit mapping (e.g.,
constexpr array or switch) that maps each column index (spec.ColumnIndex) to the
corresponding SortColumn value, validate bounds and fall back to
SortColumn::None for unknown indices, and ensure SortColumn::Status is included
in the mapping so sorting for that column is handled correctly (update the
assignment site that currently does currentSortColumn =
static_cast<SortColumn>(spec.ColumnIndex) to use this lookup).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/WeatherEditor/EditorWindow.cpp`:
- Around line 693-702: The code calls ImGui::OpenPopup every frame while
pendingDeleteWidget is set and always calls InvalidateJsonAttachmentCache when
the popup closes; change this to call ImGui::OpenPopup exactly once when the
popup is first requested and remove the redundant InvalidateJsonAttachmentCache
call on popup close. Concretely: introduce a boolean or check (e.g., a new flag
like pendingDeletePopupOpened or use ImGui::IsPopupOpen/BeginPopup to gate a
single OpenPopup call) so OpenPopup("ListDeleteConfirmation") is invoked only
once when pendingDeleteWidget is first assigned, keep
pendingDeleteWidget->DrawDeleteConfirmationModal("ListDeleteConfirmation")
as-is, and delete the InvalidateJsonAttachmentCache(pendingWidget) invocation
from the popup-close branch (relying on Delete() ->
OnWidgetJsonAttachmentChanged to handle invalidation on confirm).
- Around line 340-346: The JsonAttachment sort currently computes comparison as
static_cast<int>(bHasJson) - static_cast<int>(aHasJson) which places true before
false for ascending; change the arithmetic so ascending yields false before true
(e.g., set comparison = static_cast<int>(aHasJson) - static_cast<int>(bHasJson)
or otherwise invert the sign) in the SortColumn::JsonAttachment branch that uses
HasCachedJsonAttachment to ensure standard boolean sort direction; update only
the comparison assignment in EditorWindow.cpp.

In `@src/WeatherEditor/Widget.cpp`:
- Around line 17-20: The call to
EditorWindow::GetSingleton()->OnWidgetJsonAttachmentChanged(this) is invoked
before I/O in Widget::Save(), Widget::Load(), and Widget::Delete(); move that
call to the end of each function's success path so it runs only after the
operation completes successfully. Specifically, in Widget::Save() call
OnWidgetJsonAttachmentChanged(this) after SaveSettings() and any file-write
success checks; in Widget::Load() invoke it after the load completes and any
parsing checks; and in Widget::Delete() invoke it after the file is deleted
successfully. Ensure any early-return or error paths do not call
OnWidgetJsonAttachmentChanged so the cache/indicator is only invalidated on
successful completion.

---

Nitpick comments:
In `@src/WeatherEditor/EditorWindow.cpp`:
- Around line 134-706: The commit message uses an uppercase description and
lacks a scope; update the commit/title to follow conventional commits with a
lowercase description and a scope—for example change the PR title to
"feat(weather-editor): add delete json button to objects window" (apply when
creating the commit for changes in EditorWindow::ShowObjectsWindow or related
modifications) so it matches project guidelines and improves searchability.
- Around line 528-543: Extract the duplicated JSON/delete column rendering into
a local lambda (e.g., drawJsonDeleteColumn) that takes the Widget* (or Widget&
w) and a const char* idPrefix, and inside it call ImGui::TableNextColumn(),
check HasCachedJsonAttachment(w), read globals::menu and
menu->uiIcons.deleteSettings.texture, compute iconSize via
ImGui::GetFrameHeight() * 0.85f, scope Util::ErrorButtonStyle(), call
ImGui::SetNextItemAllowOverlap(), and call ImGui::ImageButton with
std::format("{}_{}", idPrefix, w->GetFormID()).c_str() to set
pendingDeleteWidget on click and show the same tooltip when hovered; then
replace both duplicated blocks (the one using the "current" label and the
regular list) with calls to drawJsonDeleteColumn(sortedWidgets[i],
"##jsondel_cur") and drawJsonDeleteColumn(sortedWidgets[i], "##jsondel") (or the
exact prefixes used) respectively.

In `@src/WeatherEditor/EditorWindow.h`:
- Around line 208-216: The code currently relies on a fragile static_cast from
spec.ColumnIndex to the SortColumn enum (SortColumn) when assigning
currentSortColumn, which breaks if columns are reordered or changed; replace the
static_cast with an explicit mapping (e.g., constexpr array or switch) that maps
each column index (spec.ColumnIndex) to the corresponding SortColumn value,
validate bounds and fall back to SortColumn::None for unknown indices, and
ensure SortColumn::Status is included in the mapping so sorting for that column
is handled correctly (update the assignment site that currently does
currentSortColumn = static_cast<SortColumn>(spec.ColumnIndex) to use this
lookup).

[github-actions]

✅ A pre-release build is available for this PR:
Download

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (2)

src/WeatherEditor/EditorWindow.cpp (2)

292-311: Hardcoded column indices in the sort switch are fragile.

Mapping spec.ColumnIndex values 1–5 directly to SortColumn cases will silently mis-sort if a column is inserted, removed, or reordered. ImGuiTableColumnSortSpecs::ColumnUserID is designed exactly for this: assign a unique uint32_t ID to each sortable column via the optional user_id argument to TableSetupColumn, then switch on ColumnUserID instead.

♻️ Proposed refactor (column IDs)

+enum ColumnID : uint32_t { ColFav = 0, ColEditorID, ColFormID, ColFile, ColStatus, ColJson };
 ImGui::TableSetupColumn("Fav",       ImGuiTableColumnFlags_WidthFixed | ImGuiTableColumnFlags_NoSort, 38.0f);
-ImGui::TableSetupColumn("Editor ID", ImGuiTableColumnFlags_WidthStretch, 3.5f);
-ImGui::TableSetupColumn("Form ID",   ImGuiTableColumnFlags_WidthFixed,   90.0f);
-ImGui::TableSetupColumn("File",      ImGuiTableColumnFlags_WidthStretch, 2.0f);
-ImGui::TableSetupColumn("Status",    ImGuiTableColumnFlags_WidthStretch, 1.5f);
-ImGui::TableSetupColumn("JSON",      ImGuiTableColumnFlags_WidthFixed,   55.0f);
+ImGui::TableSetupColumn("Editor ID", ImGuiTableColumnFlags_WidthStretch, 3.5f,  0, ColEditorID);
+ImGui::TableSetupColumn("Form ID",   ImGuiTableColumnFlags_WidthFixed,   90.0f, 0, ColFormID);
+ImGui::TableSetupColumn("File",      ImGuiTableColumnFlags_WidthStretch, 2.0f,  0, ColFile);
+ImGui::TableSetupColumn("Status",    ImGuiTableColumnFlags_WidthStretch, 1.5f,  0, ColStatus);
+ImGui::TableSetupColumn("JSON",      ImGuiTableColumnFlags_WidthFixed,   55.0f, 0, ColJson);
 // ...
-switch (spec.ColumnIndex) {
-case 1: currentSortColumn = SortColumn::EditorID; break;
-case 2: currentSortColumn = SortColumn::FormID; break;
-case 3: currentSortColumn = SortColumn::File; break;
-case 4: currentSortColumn = SortColumn::Status; break;
-case 5: currentSortColumn = SortColumn::JsonAttachment; break;
-default: currentSortColumn = SortColumn::None; break;
+switch (spec.ColumnUserID) {
+case ColEditorID: currentSortColumn = SortColumn::EditorID; break;
+case ColFormID:   currentSortColumn = SortColumn::FormID; break;
+case ColFile:     currentSortColumn = SortColumn::File; break;
+case ColStatus:   currentSortColumn = SortColumn::Status; break;
+case ColJson:     currentSortColumn = SortColumn::JsonAttachment; break;
+default:          currentSortColumn = SortColumn::None; break;
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.cpp` around lines 292 - 311, The switch is
using fragile hardcoded spec.ColumnIndex values; change the table setup to
assign stable uint32_t user IDs for each sortable column via
TableSetupColumn(..., user_id) and then switch on spec.ColumnUserID (not
spec.ColumnIndex) to map to SortColumn (e.g., EditorID, FormID, File, Status,
JsonAttachment, None). Update the switch in EditorWindow.cpp to use
spec.ColumnUserID and ensure each column registration that calls
TableSetupColumn uses the corresponding unique user_id constant so
reordering/insertion won't break sorting.

---

374-390: std::format heap allocation inside the per-widget render loop.

Line 382 constructs a temporary std::string via std::format on every frame for every widget that is visible. Prefer a stack buffer to avoid repeated heap allocation in the hot render path.

♻️ Proposed fix

-if (ImGui::ImageButton(std::format("##jsondel_{}", widget->GetFormID()).c_str(), ...))
+char idBuf[32];
+snprintf(idBuf, sizeof(idBuf), "##jsondel_%s", widget->GetFormID().c_str());
+if (ImGui::ImageButton(idBuf, ...))

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.cpp` around lines 374 - 390, The per-widget
render lambda drawJsonDeleteButton currently calls std::format("##jsondel_{}",
widget->GetFormID()) each frame which allocates on the heap; replace that with a
stack-allocated char buffer (e.g., char buf[32]) and write the ID into it using
snprintf or std::to_chars (format "##jsondel_<id>") before calling
ImGui::ImageButton, so the unique widget ID remains the same but avoids
per-frame heap allocations; update the ImageButton call in drawJsonDeleteButton
to use this buffer and keep pendingDeleteWidget/pendingDeletePopupRequested
logic unchanged.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/WeatherEditor/EditorWindow.cpp`:
- Around line 336-337: RefreshJsonAttachmentCache currently iterates
sortedWidgets and calls widget->HasSavedFile() (std::filesystem::exists) for
every widget on first view, causing an O(N) filesystem hit; change the approach
to populate the JSON-attachment cache lazily by moving the exists() check into
drawJsonDeleteButton (or whatever function renders per-widget UI) so only
visible/filtered widgets trigger HasSavedFile(), or implement an async/batched
prefetch that marks cache entries without blocking the UI; update
RefreshJsonAttachmentCache to only initialize metadata for already-visible
widgets (use the same visibility/filtering logic used when building the display
list) and remove the global eager loop over sortedWidgets to avoid the one-frame
stall.
- Line 283: Update the ImGui table column header string from "json" to "JSON" to
match the Title Case convention used by other headers; locate the
ImGui::TableSetupColumn call that currently passes "json" (in EditorWindow.cpp)
and replace the literal with "JSON" so the UI label is consistent with "Fav",
"Editor ID", "Form ID", "File", and "Status".

---

Nitpick comments:
In `@src/WeatherEditor/EditorWindow.cpp`:
- Around line 292-311: The switch is using fragile hardcoded spec.ColumnIndex
values; change the table setup to assign stable uint32_t user IDs for each
sortable column via TableSetupColumn(..., user_id) and then switch on
spec.ColumnUserID (not spec.ColumnIndex) to map to SortColumn (e.g., EditorID,
FormID, File, Status, JsonAttachment, None). Update the switch in
EditorWindow.cpp to use spec.ColumnUserID and ensure each column registration
that calls TableSetupColumn uses the corresponding unique user_id constant so
reordering/insertion won't break sorting.
- Around line 374-390: The per-widget render lambda drawJsonDeleteButton
currently calls std::format("##jsondel_{}", widget->GetFormID()) each frame
which allocates on the heap; replace that with a stack-allocated char buffer
(e.g., char buf[32]) and write the ID into it using snprintf or std::to_chars
(format "##jsondel_<id>") before calling ImGui::ImageButton, so the unique
widget ID remains the same but avoids per-frame heap allocations; update the
ImageButton call in drawJsonDeleteButton to use this buffer and keep
pendingDeleteWidget/pendingDeletePopupRequested logic unchanged.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

RefreshJsonAttachmentCache triggers O(N) filesystem I/O for all category widgets on first visit.

RefreshJsonAttachmentCache calls widget->HasSavedFile() (i.e., std::filesystem::exists()) for each widget not yet in the cache. Because this is called on sortedWidgets — the entire category list before filtering — all widgets in the category incur a filesystem hit on the first frame that category is viewed. For categories with many records (Skyrim has hundreds of weathers/lighting templates), this can cause a one-frame stall.

Consider populating the cache lazily in drawJsonDeleteButton (only for visible/filtered widgets) or batching it asynchronously. Alternatively, document the expected stall so it is a conscious trade-off.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/WeatherEditor/EditorWindow.cpp` around lines 336 - 337,
RefreshJsonAttachmentCache currently iterates sortedWidgets and calls
widget->HasSavedFile() (std::filesystem::exists) for every widget on first view,
causing an O(N) filesystem hit; change the approach to populate the
JSON-attachment cache lazily by moving the exists() check into
drawJsonDeleteButton (or whatever function renders per-widget UI) so only
visible/filtered widgets trigger HasSavedFile(), or implement an async/batched
prefetch that marks cache entries without blocking the UI; update
RefreshJsonAttachmentCache to only initialize metadata for already-visible
widgets (use the same visibility/filtering logic used when building the display
list) and remove the global eager loop over sortedWidgets to avoid the one-frame
stall.

[coderabbitai]

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@src/WeatherEditor/EditorWindow.cpp`:
- Around line 1861-1871: RefreshJsonAttachmentCache currently calls
widget->HasSavedFile() (which does filesystem::exists) for every widget not in
jsonAttachmentCache, causing O(N) FS hits on category open; remove that eager
population and make the cache lazy: change RefreshJsonAttachmentCache(…) to only
ensure entries are not removed (or to no-op) and move the
jsonAttachmentCache.emplace(widget, widget->HasSavedFile()) into
drawJsonDeleteButton so that drawJsonDeleteButton checks jsonAttachmentCache for
the widget and, on miss, calls widget->HasSavedFile() once and inserts the
result; reference jsonAttachmentCache, RefreshJsonAttachmentCache,
drawJsonDeleteButton, and HasSavedFile to locate and update the code.
- Line 294: The TableSetupColumn call uses a lowercase header string "json";
update the header to "JSON" in the ImGui::TableSetupColumn invocation that sets
up ColJson (ImGui::TableSetupColumn("json", ImGuiTableColumnFlags_WidthFixed,
55.0f, ColJson)) so it matches the Title Case style of the other columns.

[SkrubbySkrubInAShrub]

the rest of the AI comments are not relevant or I don't agree .