Gdevelop: Command to invert condition in event sheet condition context menu

[blurymind]

Right now to invert a condition in the event sheet, you have to click on it to edit it

Description

Adding a Command to invert condition in event sheet condition right click context menu will greatly speed up workflow Imo :)

Solution suggested

I can try to add it in another pull if you guys think it will be useful to have it in the menu like I do :)

[4ian]

Sounds good! Technically you might have to use getSelectedInstructionsContexts and check if the instruction are conditions (isCondition) when building the context menu :) Apart from that, don't see any warning or major things that would not work :)

[blurymind]

Ok Will give this a try in the next pull :) Might as well do it while working on the event sheet menus

[blurymind]

Ok this was surprisingly easy to implement :)
New pull request coming later after work. Here is a teaser:

It can invert on multiple selected. It can detect if selection contains conditions - if so apply the operation only on conditions. If none are selected it wont appear in the menu.

[4ian]

Great! The behavior looks good :)

[blurymind]

will do a pull after I sort out the other one actually :)
It's quite fun to use

[blurymind]

@4ian I noticed that you can disable entire events atm, but not individual conditions/actions. Is that something that is available, but not exposed to the newIde?

[4ian]

No, it's something that is not available in the way that instructions (conditions/actions) don't have a way to be marked as disabled. If it was the case, this would still requires changes in code generation to skip these instructions. Not impossible but more work that simply switching a boolean :)

[blurymind]

Ah I see. Thank you for clarifying :)

On Tue, 2 Oct 2018 13:35 Florian Rival, [e-mail] wrote:

No, it's something that is not available in the way that instructions
(conditions/actions) don't have a way to be marked as disabled. If it was
the case, this would still requires changes in code generation to skip
these instructions. Not impossible but more work that simply switching a
boolean :)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/4ian/GDevelop/issues/685#issuecomment-426256747, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGMbVSah450l8pRXzXNNk5AiJqALDAoJks5ug12dgaJpZM4XCoru
.

[blurymind]

I have this working great, but I need to move some of the code to selectionHandler and add flow checking to it before doing the pull :)

[blurymind]

This got merged now, so we can close the issue 👍