Move zoom function inside closure so it isn't potentially knocked out by other functions (with same name) in global scope.

This commit is contained in:
jgen 2015-06-01 15:38:04 -07:00
parent 8a8a26e23a
commit 6a6480181d

View File

@ -1,4 +1,29 @@
$(function() {
function zoom(zoom_type) {
var img = $('.shm-main-image');
if(zoom_type == "full") {
img.css('max-width', img.data('width') + 'px');
img.css('max-height', img.data('height') + 'px');
}
if(zoom_type == "width") {
img.css('max-width', '95%');
img.css('max-height', img.data('height') + 'px');
}
if(zoom_type == "height") {
img.css('max-width', img.data('width') + 'px');
img.css('max-height', (window.innerHeight * 0.95) + 'px');
}
if(zoom_type == "both") {
img.css('max-width', '95%');
img.css('max-height', (window.innerHeight * 0.95) + 'px');
}
$(".shm-zoomer").val(zoom_type);
$.cookie("ui-image-zoom", zoom_type, {path: '/', expires: 365});
}
$(".shm-zoomer").change(function(e) {
zoom(this.options[this.selectedIndex].value);
});
@ -14,27 +39,3 @@ $(function() {
zoom($.cookie("ui-image-zoom"));
}
});
function zoom(zoom_type) {
var img = $('.shm-main-image');
if(zoom_type == "full") {
img.css('max-width', img.data('width') + 'px');
img.css('max-height', img.data('height') + 'px');
}
if(zoom_type == "width") {
img.css('max-width', '95%');
img.css('max-height', img.data('height') + 'px');
}
if(zoom_type == "height") {
img.css('max-width', img.data('width') + 'px');
img.css('max-height', (window.innerHeight * 0.95) + 'px');
}
if(zoom_type == "both") {
img.css('max-width', '95%');
img.css('max-height', (window.innerHeight * 0.95) + 'px');
}
$(".shm-zoomer").val(zoom_type);
$.cookie("ui-image-zoom", zoom_type, {path: '/', expires: 365});
}