-
Notifications
You must be signed in to change notification settings - Fork 77
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
build and preRender bug #55
Comments
The information above is used to solve two problems, one is that ColorPicker continues to response to clicking action after it is closed and the other is that system fails to initialize background color after providing doRender. |
Hi @huye , } else if (_colorPicker.$trigger) { Your second concern about 'initialize background color after providing doRender'... I don't quite know what the problem is and... Can you explain your problem more precisely? |
Reproduce code: <script src="http://libs.useso.com/js/jquery/1.9.1/jquery.min.js"></script> <label class="color"> |
have problem with continuous click ealier, the new version has fix the issue. |
Hi @huye , |
I hope to be able to trigger the render event when you click on the label. |
@huye <script src="http://libs.useso.com/js/jquery/1.9.1/jquery.min.js"></script>
<script src="http://www.dematte.at/tinyColorPicker/jqColorPicker.min.js"></script>
<label class="">
<span style="display: block">Text</span>
<input name="fore" class="no-alpha border-color color" value="#FF0000">
</label>
<script>
$('.color').colorPicker({
opacity: false
});
</script> You can click on the label and still be able to open the picker... which also triggers the |
add in line 18:
_toggle,
insert in line 200:
if(_toggle == toggled){ return } _toggle = toggled;
replace in line 312:
var value = extractValue(this), mode = value.split('('), $elm = findElement($(this));
to:
var $elm = findElement($(this)), value = extractValue($elm), mode = value.split('(');
The text was updated successfully, but these errors were encountered: