Skip to content

Commit

Permalink
Added a new "allow_expand" setting
Browse files Browse the repository at this point in the history
The new setting is a boolean that defaults to true.
It is only relevant when a large image is resized.
If true, the expand control is displayed just as before.
If false, the expand control is not displayed; the user
sees the scaled image but cannot zoom in to the original size.

I added this setting as an option to simplify the user interface.
Some users confuse the expand control with a close button, then
can't easily figure out how to un-expand the image and close.
  • Loading branch information
sbrauer committed Oct 26, 2011
1 parent fa2258a commit 1f5df07
Showing 1 changed file with 27 additions and 19 deletions.
46 changes: 27 additions & 19 deletions js/jquery.prettyPhoto.js
Expand Up @@ -15,6 +15,7 @@
opacity: 0.80, /* Value between 0 and 1 */
show_title: true, /* true/false */
allow_resize: true, /* Resize the photos bigger than viewport. true/false */
allow_expand: true, /* Allow the user to expand a resized image. true/false */
default_width: 500,
default_height: 344,
counter_separator_label: '/', /* The separator for the gallery counter 1 "of" 2 */
Expand Down Expand Up @@ -393,7 +394,9 @@
rel_index = set_position;

if(!doresize) doresize = true; // Allow the resizing of the images
$('.pp_contract').removeClass('pp_contract').addClass('pp_expand');
if(settings.allow_expand) {
$('.pp_contract').removeClass('pp_contract').addClass('pp_expand');
}

_hideContent(function(){ $.prettyPhoto.open(); });
};
Expand Down Expand Up @@ -518,10 +521,12 @@
// Show the nav
if(isSet && _getFileType(pp_images[set_position])=="image") { $pp_pic_holder.find('.pp_hoverContainer').show(); }else{ $pp_pic_holder.find('.pp_hoverContainer').hide(); }

if(pp_dimensions['resized']){ // Fade the resizing link if the image is resized
$('a.pp_expand,a.pp_contract').show();
}else{
$('a.pp_expand').hide();
if(settings.allow_expand) {
if(pp_dimensions['resized']){ // Fade the resizing link if the image is resized
$('a.pp_expand,a.pp_contract').show();
}else{
$('a.pp_expand').hide();
}
}

if(settings.autoplay_slideshow && !pp_slideshow && !pp_open) $.prettyPhoto.startSlideshow();
Expand Down Expand Up @@ -827,20 +832,23 @@

$('a.pp_close').bind('click',function(){ $.prettyPhoto.close(); return false; });

$('a.pp_expand').bind('click',function(e){
// Expand the image
if($(this).hasClass('pp_expand')){
$(this).removeClass('pp_expand').addClass('pp_contract');
doresize = false;
}else{
$(this).removeClass('pp_contract').addClass('pp_expand');
doresize = true;
};

if(settings.allow_expand) {
$('a.pp_expand').bind('click',function(e){
// Expand the image
if($(this).hasClass('pp_expand')){
$(this).removeClass('pp_expand').addClass('pp_contract');
doresize = false;
}else{
$(this).removeClass('pp_contract').addClass('pp_expand');
doresize = true;
};

_hideContent(function(){ $.prettyPhoto.open(); });

_hideContent(function(){ $.prettyPhoto.open(); });

return false;
});
return false;
});
}

$pp_pic_holder.find('.pp_previous, .pp_nav .pp_arrow_previous').bind('click',function(){
$.prettyPhoto.changePage('previous');
Expand Down Expand Up @@ -902,4 +910,4 @@

})(jQuery);

var pp_alreadyInitialized = false; // Used for the deep linking to make sure not to call the same function several times.
var pp_alreadyInitialized = false; // Used for the deep linking to make sure not to call the same function several times.

0 comments on commit 1f5df07

Please sign in to comment.