-
-
Notifications
You must be signed in to change notification settings - Fork 50
Allow only one behavior selection #2541
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
Comments
This could probably use a little whiteboarding, as there are a few complexities. Just off of the top of my head, as examples:
I think not all of this complexity is introduced by adding custom behaviors, but it certainly highlights the need to figure out a design solution and effectively communicate to users what they can expect. |
WIP for #2541 ## Changes - Moves custom behaviors table to behind "Use Custom Behaviors" checkbox. - Updates autoclick selector to match checkbox reveal layout. - Adds minimum viable user guide documentation of custom behaviors.
Yeah, the issue is that the crawler only will run autoclick, autoscroll, OR a custom behavior per page, but different pages could have different rules. Autoclick and Autoscroll can both run on the same page, but its sort of not recommended, can result in unexpected results - though perhaps can be improved. The custom behavior only runs if it matches the specific regex. |
The UI and API currently allows selecting multiple built-in behaviors or custom behaviors. The crawler actually only will run autoclick, autoscroll, OR a custom behavior. The UI should reflect the exclusion options.
If there are existing workflows with both autoscroll and autoclick enabled, autoscroll should be unchecked in favor of autoclick, which is opt-in.
The text was updated successfully, but these errors were encountered: