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

Adding generation of custom 'class'-tags in div and img #44

Closed
wants to merge 5 commits into from
Closed

Adding generation of custom 'class'-tags in div and img #44

wants to merge 5 commits into from

Conversation

poikilotherm
Copy link

I wanted to style the image picker with some classes within css.

There was no option to add custom classes to the div or img element generated by jQuery, so I added the tag 'data-img-class' to be used for that.

I hope this is of use for others, too. Would be glad if you accept this PR. Thanks!

@rvera
Copy link
Owner

rvera commented Sep 25, 2016

This is a good idea, due to some conflicts I can't merge this but I added this on 0.3.0 and gave you credit for the good idea!

@rvera rvera closed this Sep 25, 2016
@kevinwheeler
Copy link

Is anyone else having issues with this not working? I am using version 0.3.0 via bower. data-img-src works fine, but data-img-class does nothing.

@rvera
Copy link
Owner

rvera commented Oct 18, 2016

Hi Kevin, maybe the bower version is wrong?

Check the demo page http://rvera.github.io/image-picker and look at the options

screen shot 2016-10-18 at 5 33 04 pm

It produces the expected result

screen shot 2016-10-18 at 5 33 17 pm

@hariag
Copy link

hariag commented Dec 6, 2016

Hi, kevinwheeler, tried use image-picker.js instead of image-picker.min.js and it is working. I guess image-picker.min.js not updated yet.

@rvera
Copy link
Owner

rvera commented Dec 6, 2016

uh, that's weird. Thanks @hariA will take a look at that.

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.

4 participants