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

provide a way to get access to photo content and to call closeAndCall… #107

Closed
wants to merge 3 commits into from

Conversation

Rebolon
Copy link

@Rebolon Rebolon commented Feb 11, 2016

…back function to allow the template remove from DOM

It also add a way to specify options using Meteor.settings.public.Meteor.Camera :
{ camera: {type: "front"/"back", width: 640, height: 480, quality: 80, viewFinder: {width: 320, height: 240}}

fix #106

…back function to allow the template remove from DOM

It also add a way to specify options using Meteor.settings.public.Meteor.Camera :
{ camera: {type: "front"/"back", width: 640, height: 480, quality: 80, viewFinder: {width: 320, height: 240}}
@Rebolon
Copy link
Author

Rebolon commented Feb 11, 2016

Maybe that would be better to remove the listener on "change #videoFallback" because it's up to the developper to add what he wants. Maybe he wants to do some image manipulation (what i'm doing on my apps per exemple).

Here is the code to be removed if you wants :

"change #videofallback": function(ev, tpl) {
var files = ev.currentTarget.files; // FileList
if(!files.length){
error.set("Please, click on browse button to select a file.");
return;
}

var reader  = new FileReader();
reader.addEventListener("load", function () {
  MeteorCamera.getPhoto().set(reader.result);
}, false);

if (files[0]) {
  reader.readAsDataURL(files[0]);
}

}

@CLAassistant
Copy link

CLAassistant commented Oct 7, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@StorytellerCZ
Copy link
Collaborator

This PR is outdated and requires a significant update. As such I'm closing this for now, but feel free to update and re-open as we are now actively maintaining this repo.

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.

The template is not removed until the user click on .cancel
3 participants