genemu / GenemuFormBundle

Extra Form : Captcha GD, Tinymce, Recaptcha, JQueryDate, JQueryAutocomplete, JQuerySlider, JQueryFile, JQueryImage
587 stars 263 forks source link

Fix multiple colorpickers on the same page #331

Open vdesdoigts opened 10 years ago

vdesdoigts commented 10 years ago

If there are two colorpickers on the same page, the first colorpicker don't update his "background-color" properly when you changing its value. It's always the last colorpicker who gets the "background-color" information.

This commit resolve this problem by changing the global variable $picker to local variable.

gmourier commented 10 years ago

This pull request should be merged asap! I fixed it too but every composer install/update will break it.

ghost commented 9 years ago

+1 for this fix ! Thank you !