search

jsymonkj / pwi

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

showCaptionLength: Not working #69

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Type large caption on image in Picassa and upload.
2. set showCaptionLength: 10 (no comma as it is my last option)
3.

What is the expected output? What do you see instead?
Restricted caption is expected to 10 characters.
As much text as you place on image in Picassa shows in the web album thumbnails 
distorting the layout.

What version of the product are you using? On what browser and version and 
operating system?
IE8, Windows 7.

Please provide any additional information below.
I have tried changing it from default 9999 to loads of smaller numbers but it 
does not seem to work.
Here is my code:
        <script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script>

        <link   href="js/jquery.slimbox2/jquery.slimbox2.css" rel="stylesheet" type="text/css"/>
        <script src="js/jquery.slimbox2/jquery.slimbox2.js" type="text/javascript"></script>
        <script src="js/jquery.blockUI.js" type="text/javascript"></script>

        <link   href="css/pwi.css" rel="stylesheet" type="text/css"/>
        <script src="js/jquery.pwi.js" type="text/javascript"></script>

        <script type="text/javascript">
            $(document).ready(function () {

                $("#picassa").pwi({
                    username: '101316428959337362000',
                    mode: 'albums',
                    thumbSize: 144,
                    thumbCrop: 0,
                    showPhotoCaption: true,
                    showPhotoCaptionDate: false,
                    showCaptionLength: 10

                });
            });
        </script>

Original issue reported on code.google.com by adwo...@sensibleweb.co.uk on 4 Nov 2010 at 6:37

GoogleCodeExporter commented 9 years ago 
It's not a bug, the settings is used to control the caption length of photos 
and not the caption length of albums. This is actually not possible right now.
I'll put this issue on the list as a feature request!

Original comment by jdiderik on 5 Nov 2010 at 10:09

GoogleCodeExporter commented 9 years ago 
Sorry, it is actually a bug :-), it doesn't work correctly on thumbs of 
images... will try and fix it.

Original comment by jdiderik on 5 Nov 2010 at 10:12

GoogleCodeExporter commented 9 years ago 
Thanks for a great set of scripts to do this, it is a brilliant tool.

Yes I was meaning that it did not work in the thumbnails of an albulm.
i.e. If you have 4 albumns, click on any of the 4 album thumbnails and it will 
show all thumbnails of the photos in that album. 
It is the caption underneath all those photo thumbnails that I thought this 
setting was supposed to restrict.
Sorry if I did not explain it well enough.

Original comment by adwo...@sensibleweb.co.uk on 5 Nov 2010 at 10:19

GoogleCodeExporter commented 9 years ago 
Thx for the compliment! That's greatly appreciated!

You could fix it for yourself easily by editing line 91 where it says:
if($c > settings.showCaptionLength...

change it into:
if($c.length > settings.showCaptionLength...

Fix will be included in next version, which will be released soon (hopefully)

Original comment by jdiderik on 5 Nov 2010 at 10:25

GoogleCodeExporter commented 9 years ago 
Added fix to SVN, revision 140.

Original comment by borkh...@gmail.com on 11 Aug 2011 at 10:41