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

REPORT-495: Added an onkeyup event to the Widgets and Text input elem… #205

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

IamMujuziMoses
Copy link

…ents which disables and enables the smallButtons when the input elements have or don't have input respectively

https://issues.openmrs.org/browse/REPORT-495

…ents which disables and enables the smallButtons when the input elements have or don't have input respectively
@HerbertYiga
Copy link

@IamMujuziMoses Thanks for working on this,can you make one pr for this instead of having two!

@IamMujuziMoses
Copy link
Author

@HerbertYiga I made changes to two different modules; the reporting module and the htmlwidgets module and I don't know yet how to make one pull request for two different modules

@IamMujuziMoses
Copy link
Author

IamMujuziMoses commented Sep 14, 2020

@HerbertYiga is there anything more to do to this PR?

@k-joseph @davidpalacios @dkayiwa requesting review on this

@dkayiwa
Copy link
Member

dkayiwa commented Jan 8, 2022

@IamMujuziMoses are you still able to reproduce this problem on this server? https://qa-refapp.openmrs.org/

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.

3 participants