Graphite icon indicating copy to clipboard operation
Graphite copied to clipboard

Path tool no longer overrides other hints

Open AustinHen opened this issue 3 months ago • 1 comments

Closes https://discord.com/channels/731730685944922173/881073965047636018/1419968518086197269

Previously the path tool hints would override the hints from other operations (Eg GRS's hints would not show. This was the case for everything but the drag) It was just an order of operations issue. When "update_dynamic_hints" was created (replacing "update_hints"), the path tool updates would always happen after the other tools' hints. This means that, when you are in GRS, the GRS hints will be written and then immediately overwrite. Sending ToolMessage::UpdateHints() after updating the path tool's hints fixes this.

AustinHen avatar Sep 23 '25 23:09 AustinHen

an alternative would be to call responses.add_front(FrontendMessage::UpdateInputHints { hint_data }); as opposed to responses.add(FrontendMessage::UpdateInputHints { hint_data });

AustinHen avatar Sep 23 '25 23:09 AustinHen

Tested. Works as expected.

Video:

https://github.com/user-attachments/assets/72c2c016-6a2a-44d2-865d-ffcb4d374e95

CC @Keavon

Ezbaze avatar Dec 12 '25 02:12 Ezbaze