vasiljevski / admirorgallery

Joomla! - Admiror Gallery
http://www.admiror-design-studio.com/admiror-joomla-extensions/admiror-gallery
GNU General Public License v2.0
4 stars 3 forks source link

Article + Galleryview -> EL Undefined #12

Open vasiljevski opened 8 years ago

vasiljevski commented 8 years ago

Did try this in an article view

{AG rootFolder="/images/sampledata/" thumbWidth="200" thumbHeight="120" newImageTag="1" newImageTag_days="7" paginUse="1" paginImagesPerGallery="10" albumUse="1" backgroundColor="#ffffff" foregroundColor="#808080" highliteColor="#fea804" frame_width="500" frame_height="300" showSignature="1" plainTextCaptions="1" ignoreError="1" ignoreAllError="0" thumbAutoSize="width" template="galleryView" popupEngine="slimbox" arrange="priority"}parks{/AG}

did notice in Firefox that there is an undefined "el" and it points to line 325 of jquery.galleryview.2.1.1.js It's repeating that function a lot of times due to that error so definately something that should change!

vasiljevski commented 8 years ago

it's pointing to this code, Code: var left = 0, top = 0; var el_id = el.id;

but shouldn't it be something like Code: function getPos(el) { var left = 0; var top = 0; var el_id = el.id;

think the "top" should be defined seperatly but can be wrong.