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
TIMOB-9991-added a new Android property to disable lightTouchEnabled #4520
Conversation
@@ -138,7 +138,6 @@ public TiUIWebView(TiViewProxy proxy) | |||
settings.setSupportMultipleWindows(true); | |||
settings.setJavaScriptCanOpenWindowsAutomatically(true); | |||
settings.setLoadsImagesAutomatically(true); | |||
settings.setLightTouchEnabled(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You shouldn't remove this since this is default setting. The developers can set it to false if they want.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The behaviour didn't change, I just moved this from here to set the value in the 'defaultValues' in the constructor above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I didn't see that. Apologies.
Code reviewed. Please address comments. |
Code reviewed and functionally tested. Also modified test case in JIRA. Request accepted |
@@ -502,6 +502,20 @@ properties: | |||
platforms: [iphone, ipad] | |||
since: "1.8.2" | |||
default: true | |||
|
|||
- name: lightTouchEnabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add two spaces before the dash.
Reviewed. Ran validate and deploy. Left a few comments. |
Reviewed. Ran validate and docgen. DR approved. |
TIMOB-9991-added a new Android property to disable lightTouchEnabled
Added a new Android only property 'lightTouchEnabled' to turn off the value.