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

Bug fixes and optimizations #65

Closed
wants to merge 29 commits into from
Closed

Conversation

uniphonic
Copy link
Contributor

This is a pull request from my own fork where I pulled in other pull requests from #36 and #39 and also added in some bug fixes including:

  • Adding files for Android 6 Marshmallow compatibility and 64 bit devices, such as the Nexus 5x.
  • fixing orientation of camera viewport for Android Nexus 5x devices.
  • Wrapping some code in try catches (flash and focus mode), so they don't crash when they fail on older devices. Samsung Tab 4 was crashing because it didn't have a flash or autofocus.
  • Increased the autofocus interval from 500ms to 2000ms, because older devices weren't allowed enough time to focus before they were being forced to reset their focus, which had the effect of constantly being out of focus.

This has been tested on Samsung Tab 4, Samsung Galaxy S3, Samsung Galaxy S4, Samsung Galaxy S5, Nexus 5x, iPhone 4, iPhone 5, and iPhone 6.

micaTewe and others added 28 commits September 15, 2015 00:50
Adding button to activate/deactivate flash during the capturing the phase.
Added a method to turn on the camera's flash during the scanning process.
Added some block in to handle the device orientation
Added a method to turn on the flash button
Fix the sigh light size on device orientation change
A change to activity configuration has been made to allow the orientation change.
update so files to be compatible with android M (23)
Orientation change and flash button added
wait until viewController is dismissed before callback, fixes tjwoon#47
…ch as the Galaxy S3, to finish focusing before it tries to refocus. Before this change it was appearing as if the autofocus wasn't working at all, because it kept resetting the focus before it got a chance to focus.
…s. Samsung Tab 4 was crashing because it didn't have a flash or autofocus.
adding support lib to plugin.xml
@dcousens dcousens self-assigned this Apr 15, 2016
@dcousens
Copy link
Contributor

dcousens commented Apr 18, 2016

I'll be on this this week.

ACK 39af9e1

Pulling in latest changes from tjwoon master.
@@ -194,7 +196,7 @@ public void onResume ()
Log.d("csZBar", "Unsupported camera parameter reported for flash mode: "+flashMode);
}

tryStartPreview();
tryStartPreview();*/
Copy link
Contributor

@dcousens dcousens Apr 21, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you just remove the unused code, not comment it out?

@dcousens dcousens mentioned this pull request Apr 22, 2016
@dcousens
Copy link
Contributor

Closed in favour of #66

@dcousens dcousens closed this Apr 22, 2016
@uniphonic uniphonic mentioned this pull request May 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants