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

Don't render previews if previewsContainer == false #487

Merged
merged 1 commit into from May 11, 2014

Conversation

kevinkrouse
Copy link

Fix for #175

If the previewsContainer option is false, previews won't be rendered. Alternatively, we could introduce a new 'renderPreviews' boolean option instead of piggy-backing on previewsContainer.

@Oluwafemikorede
Copy link

Hello Kevin,

I need a little help on dropzone preview area, I will like to make my preview image fill the whole dropzone area.. Please how can I do this.

Thanks

@kevinkrouse
Copy link
Author

@Oluwafemikorede Sorry, I don't know. You'll probably need to change the previewTemplate and the css.

@Oluwafemikorede
Copy link

@kevinkrouse Thanks, I have actually solved the problem, I tweaked the css and it worked

@Sheffer
Copy link

Sheffer commented Apr 23, 2014

Would love if this got merged, it is exactly what I need. I was just in the process of changing the source myself, but it would definitely be better if it was supported officially.

@jnwelzel
Copy link

jnwelzel commented May 8, 2014

+1 would love to see this feature being offered out-of-the-box instead of having to resort to CSS class to hide the preview element.

@luizkowalski
Copy link

👍

enyo added a commit that referenced this pull request May 11, 2014
Don't render previews if previewsContainer == false
@enyo enyo merged commit f7ab637 into dropzone:master May 11, 2014
@enyo
Copy link
Collaborator

enyo commented May 11, 2014

Thanks. Great PR. Will release in a few days.

@thatisuday
Copy link

@enyo This feature doesn't reflect in docs.

@lkgarrison
Copy link

@enyo It would be excellent if it was mentioned in the docs at http://www.dropzonejs.com/#config-previewsContainer that "previewContainers" can be set to false

Currently this is not explained in the docs and I think there may be more users such as myself who will have trouble tracking down this pull request to realize that this feature already exists but isn't documented.

Thanks for adding this feature though!

@Jack97 Jack97 mentioned this pull request Oct 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants