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

Fixed issue entering text into Logic Edtior for C.4.18.0700 #5

Closed
wants to merge 1 commit into from

Conversation

mmcev106
Copy link
Contributor

@mmcev106 mmcev106 commented Oct 2, 2024

@aldefouw, I believe this fixes the issue with characters getting skipped when text is entered into the Logic Editor.

@aldefouw
Copy link
Owner

aldefouw commented Oct 7, 2024

@mmcev106 - can you remind me where we left off on this one?

Did this pattern work to alleviate this particular issue without resorting to delays:

And I click on "" in the textarea field labeled "Enter logic for new rule"
And I clear field and enter "[integer]='1'" in the textarea field labeled "Logic Editor" in the dialog box

If so, I'd recommend we leave the code as is without the delay.

@mmcev106
Copy link
Contributor Author

mmcev106 commented Oct 7, 2024

@aldefouw, yes, that plan sounds good to me. It's on the TODO list. I had some REDCap production issues that took precedence today...

@mmcev106
Copy link
Contributor Author

mmcev106 commented Oct 9, 2024

The changes in the comment above do not work consistently. There is still an intermittent timing issue, likely due to the editor not being fully initialized yet. Adding a 1 min wait time in this PR resolves it, but of course we should find a solution that doesn't involve waiting. Adam D recommended trying this. It's on my TODO list. I'll go ahead and close this PR.

@mmcev106 mmcev106 closed this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants