Added "Tileset > Edit Tileset Image Parameters..." action

[bjorn]

This makes the functionality easier to discover. Previously editing the tileset image parameters was only possible by clicking the value of the Image property in the Properties view, and then the appearing "Edit" button.

[bjorn]

@CodiumAI-Agent /review

(just curious what feedback it might have)

[CodiumAI-Agent]

PR Reviewer Guide 🔍

(Review updated until commit c8a7cda)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Update Consistency The new action for editing tileset parameters is integrated using updateActions instead of the former updateAddRemoveActions. Please verify that all UI state changes (such as enabling/disabling actions) are consistently applied in all contexts where the state update is needed. updateActions(); } Default Initialization The TilesetParameters struct now includes a default constructor and default initializers for tileSpacing and margin. Confirm that these default values are appropriate across all usages and do not unintentionally affect behavior when comparing parameter changes. TilesetParameters() = default; explicit TilesetParameters(const Tileset &tileset); bool operator != (const TilesetParameters &other) const; QUrl imageSource; QColor transparentColor; QSize tileSize; int tileSpacing = 0; int margin = 0; };

[bjorn]

Oh well, the agent failed to understand a line was added to a function rather than a "duplicate" being added. For a moment I thought it might have caught the duplication between TilesetEditor::editTilesetParameters and TilesetParametersEdit::buttonClicked, but no, I think the latter was likely not in the context.

[Copilot]

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/tiled/tileseteditor.h:95

• [nitpick] Consider renaming 'editTilesetParametersAction' (and related functions) to 'editTilesetImageParametersAction' for consistency with the UI label 'Edit Tileset Image Parameters...'.

QAction *editTilesetParametersAction() const;

[CodiumAI-Agent]

Persistent review updated to latest commit c8a7cda