AltynbekKZ / galleriffic

Automatically exported from code.google.com/p/galleriffic
0 stars 1 forks source link

Class of horizontal or vertical within span.image-wrapper #74

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
This is not a problem or bug. But an enhancement to the library.

I found it necessary to know whether the image was oriented horizontally or
vertically. Therefore I hacked the code to make it happen. I would find it
useful to have this in future iterations of the code.

The following is the additional code (which is inserted within the
buildImage method):

                // MODIFIED BY DAVIS HAMMON ON 2009-12-12
                /* 
                 * orientation will be either vertical or horizontal (as determined by
natural dimensions of the image)
                 * the value of orientation is then placed into the class string (as
can be found through declaration of the newSlide variable, or by searching
'MODIFIED BY DAVIS HAMMON')
                */
                var orientation = (imageData.image.width > imageData.image.height) ?
"horizontal" : "vertical";

                // Construct new hidden span for the image
                var newSlide = this.$imageContainer
                    .append('<span class="image-wrapper ' + /* MODIFIED BY DAVIS HAMMON */
orientation +' current"><a class="advance-link" rel="history"
href="#'+this.data[nextIndex].hash+'"
title="'+imageData.title+'">&nbsp;</a></span>')
                    .find('span.current').css('opacity', '0');

Original issue reported on code.google.com by davis.ha...@gmail.com on 24 Dec 2009 at 8:51