BAH-2994 - fix the dropdown select issue on IOS devices #104
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The issue is that in IOS devices the onChange handler is not getting invoked after selecting a value from the dropdown menu. Further analysis showed that it is happening due to another library called "fastclick" in openmrs-bahmniapps repo. The "fastclick" library causing this issue with react-select in IOS devices can be found in the below link.
JedWatson/react-select#2891
Fix:
The "fastclick" library provides a way to ignore its effect on a certain element by adding classname "needsclick" to that element. This can be found on their official npm page.
https://www.npmjs.com/package/fastclick
JedWatson/react-select#1036