GraphiteEditor / Graphite

2D vector & raster editor that melds traditional layers & tools with a modern node-based, non-destructive, procedural workflow.
https://graphite.rs
Apache License 2.0
7.29k stars 385 forks source link

Crash when selecting node with Widget::NumberInput in properties panel #1761

Open adamgerhant opened 1 month ago

adamgerhant commented 1 month ago

When the cursor is in a text box, such as in the Text node, and the transform node is clicked, the editor crashes. This is due to the Value::String not being reset when switching nodes, so the string value from the text box gets passed to the number input, which causes a panic.

0HyperCube commented 1 month ago

Note that this doesn't happen in firefox and only in chromium. Also the text node needs to be selected in the graph with the cursor in the text box before clicking on the transform node.

Keavon commented 1 month ago

Yeah, I've run into other bugs where Firefox and Chrome had different event ordering. I can try looking into this. Thanks for that extra data @0HyperCube.