Graphite icon indicating copy to clipboard operation
Graphite copied to clipboard

Add missing StartTransaction for ToggleSelectedVisibility

Open dhiraj-143r opened this issue 1 month ago • 3 comments

Closes #3381

This PR fixes issue #3381, where hiding/unhiding a layer did not always update the document's saved state correctly.

The issue was that the ToggleSelectedVisibility action did not start a transaction.

dhiraj-143r avatar Nov 15 '25 11:11 dhiraj-143r

!build

TrueDoctor avatar Nov 15 '25 20:11 TrueDoctor

📦 Build Complete for 7f7858b064cd72dbbabe449aa8a10340a20ea4ea
https://a5bb6400.graphite.pages.dev

github-actions[bot] avatar Nov 15 '25 20:11 github-actions[bot]

This only seems to work if I hide the layer and then also select it, can you please double-check if the solution you implemented works on your end and addresses the original issues? We will look at the code once the functionality is complete

TrueDoctor avatar Nov 19 '25 21:11 TrueDoctor