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-23848] iOS: Ti.Media add focus methods and properties #8662
Conversation
} | ||
} | ||
} | ||
} |
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.
Some issues:
- Put the square brackets in the same line as the if-statement (
if (exp) {
), same infor (expr) {
- Put the configuration in the setter of the property, e.g.
- (void)setFlashMode:(id)value
and handle it there - Provide a getter, e.g.
- (id)flashMode
as well - I'm curious if setting the
AVCaptureDevice
will work here, because we don't use theAVFoundation
camera API's but theUIImagePickerController
API's
constants: Titanium.Media.FOCUS_MODE_* | ||
type: Number | ||
default: Depends on camera | ||
platforms: [android, iphone, ipad] |
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 don't need to specify the platforms if all three are supported. It will use all by default. Same for the other docs as well.
summary: Constant to indicate focus mode of camera. | ||
constants: Titanium.Media.FOCUS_MODE_* | ||
type: Number | ||
default: Depends on camera |
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.
Can we write something else here? It doesn't really help the developer. So either more infos on how it depends on the camera or leave it out and add a link to the Apple docs in the description
value.
Added some comments, but it looks pretty good already! |
Closing this PR as it will be part of the camera-module. |
https://jira.appcelerator.org/browse/TIMOB-23848