Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ability to re-order conditions #565

Open
1 task done
sbdixson opened this issue Dec 4, 2024 · 4 comments
Open
1 task done

ability to re-order conditions #565

sbdixson opened this issue Dec 4, 2024 · 4 comments
Labels
accepted Improvements have been accepted by the developer enhancement New feature or request
Milestone

Comments

@sbdixson
Copy link

sbdixson commented Dec 4, 2024

Describe the feature

I would love a way to re-order screen conditions inside of an event, it seems like the only way to do this now is to wait for the events to show up in the right order and add them in that way which can be very challenging / time consuming

Describe the usage

I'm using the order of conditions to simulate a sort of decision tree, but if I want to change that logic I have to drop conditions and recreate them in the right order

Is it a duplicate?

  • There is no duplicates
@sbdixson sbdixson added the enhancement New feature or request label Dec 4, 2024
@Nain57
Copy link
Owner

Nain57 commented Jan 16, 2025

Hi and thank you for your feddback.

The order of the image conditions cannot be changed because there is no impact on the result (besides computation optimization, but that's overkill). If we only take the result of a boolean operation into consideration, A & B & C == C & A & B and A | B | C == C | B | A

Closed as won't do

@Nain57 Nain57 closed this as completed Jan 16, 2025
@Nain57 Nain57 added the wontdo This feature has been rejected by the developer label Jan 16, 2025
@sbdixson
Copy link
Author

sbdixson commented Jan 16, 2025 via email

@Nain57
Copy link
Owner

Nain57 commented Jan 16, 2025

Ah yes, thanks for the precision.
Here what matters is the click on detected condition, and with an OR, this will have an impact, as the clicked condition is the first one detected.

Now I see the usage, i will re open the ticket. Can you update it to clarifie it ? You want to reorder the condition list in order to control which condition to click on when multiple condition are matched.

@Nain57 Nain57 reopened this Jan 16, 2025
@Nain57 Nain57 removed the wontdo This feature has been rejected by the developer label Jan 16, 2025
@sbdixson
Copy link
Author

Yes thank you! thats exactly what I'm looking for,

I want to reorder the condition list in order to control which condition to click when multiple condition are matched.

thank you for all your awesome work!!

Nain57 added a commit that referenced this issue Jan 17, 2025
Implementation of Feature Request
#565
@Nain57 Nain57 added the accepted Improvements have been accepted by the developer label Jan 17, 2025
@Nain57 Nain57 added this to the Klick'r 3.3.0 milestone Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Improvements have been accepted by the developer enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants