Skip to content

[pull] master from mapeditor:master - #126

Merged
pull[bot] merged 2 commits into
dogboydog:masterfrom
mapeditor:master
Sep 17, 2026
Merged

pull[bot] merged 2 commits into
dogboydog:masterfrom
mapeditor:master

Conversation

@pull

@pull pull Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

#4614)

setProperty and setProperties did not check whether the asset is
read-only when there was no document, unlike removeProperty and
setClassName. This allowed modifying the map or tileset passed to the
write function of a scripted format, which bypasses the undo stack of
the document being saved.

Now they raise the "Asset is read-only" error as well.
Forwards the selection change signal of the tileset document, so that
extensions no longer need to poll Tileset.selectedTiles.

The tileset editor now also avoids setting the selection on the document
again after updating its view to match the document, which would have
caused the signal to be emitted twice for a change made from script.
@pull pull Bot locked and limited conversation to collaborators Sep 17, 2026
@pull pull Bot added the ⤵️ pull label Sep 17, 2026
@pull
pull Bot merged commit d853f58 into dogboydog:master Sep 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant