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-23992] iOS: Disable context menu and contextual menu in Ti.UI.WebView #8488
Conversation
test this please |
// Disable user selection and the attached callout | ||
BOOL disableSelection = [TiUtils boolValue:[[self proxy] valueForKey:@"disableSelection"] def:NO]; | ||
// Disable the context menu when selecting text | ||
BOOL disableSelection = [TiUtils boolValue:[[self proxy] valueForKey:@"disableContextMenu"] def:NO]; |
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.
might want to rename the variable to match the property: disableContextMenu - rather than disableSelection.
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.
Agree. The Travis job is running for 2 days now, hopefully it's re-triggered correctly after the commit.
Hey Chris, so I finally updated the docs to have the Android part documented as well (not done in #8494). We should be cool now! Please squash-merge or I'll rebase the commits, it became a bit messy at the end, sorry for that! |
JIRA: https://jira.appcelerator.org/browse/TIMOB-23992