PDA

View Full Version : PHP Photo Album script v2.0 - sortby:date/desc doesnt appear to work properly



Beverleyh
08-08-2009, 04:15 PM
1) Script Title: PHP Photo Album script v2.0

2) Script URL (on DD): http://www.dynamicdrive.com/dynamicindex4/php-photoalbum.htm

3) Describe problem: I set the gallery to sortby: ["date", "desc"], //["file" or "date", "asc" or "desc"] and the gallery doesnt show (only tested in IE7) although its fine when sorted ["date", "asc"]

ddadmin
08-08-2009, 07:24 PM
If "asc" for date works, then "desc" should also work, since all the script does is reverse the results of the former. Do you have a URL to the problem page? Be sure to enable "autodesc" to show the date ("%d"), so we can easily see the dates of each image.

Beverleyh
08-08-2009, 09:36 PM
Here's the page with the sortby set to ["date", "asc"]: http://temp.jemcon.org/images.php

And here's the page with the sortby set to ["date", "desc"]: http://temp.jemcon.org/images1.php

autodesc is set to "%d" for both.

Please note - there are other javascripts on the pages, but both are identical other than the above sortby setting, so why would one work in IE and not the other? (confused)
(I also tried the script on a basic version of the page, without any other javascripts, and experienced the same problem)

Additional: The "["date", "desc"]: http://temp.jemcon.org/images1.php" page works fine in FireFox and Safari - its just IE that doesnt like it.

Beverleyh
08-09-2009, 05:41 PM
Should this thread be moved to the main "Dynamic Drive scripts help" forum?

I've redownloaded/reinstalled the script but still cannot get it to work with sortby set to ["date", "desc"], in IE.

Is this just a problem with me as I see you have a note on the script page that says its compatiable with FF1+, IE5+ and Opr7+ ?

ddadmin
08-10-2009, 12:46 AM
In IE8 at least, I don't see the gallery on image1.php actually. I do see it on image.php though. Is there something in the former that may be hiding the gallery for me in IE?

Beverleyh
08-10-2009, 09:06 AM
hi again ddadmin,

No, the pages are exactly the same apart from the sortby setting of the gallery being ["date", "asc"] in the image.php page but ["date", "desc"] in the image1.php page.

I have used IETester to view the pages in all versions of IE but the ["date", "desc"] gallery in image1.php is not visible in any. (I have IE7 installed and set to my default browser)

I hope you can fix it as the upgrade really is a lovely gallery script.

ddadmin
08-10-2009, 08:42 PM
Ok I see what you mean now. Basically the "desc" didn't work in IE due to one of those IE specific "quirks". I believe I've fixed the issue- just revisit the script, and replace both "ddphpalbum.js" and "getalbumpics.php" with the revised versions.

Beverleyh
08-11-2009, 07:16 AM
Thats wonderful! - thank you for the fix, ddadmin

Beverleyh
08-12-2009, 08:47 PM
Hi ddadmin,

I've found another little bug, this time with the navigation links underneath the gallery.

As soon as the page button links exceed the gallery width and jump to the line below, they overlap the page button links on the line above - see page button link 14: http://www.jemcon.org/upload_gallery.php

I tried adding margin-bottom:5px; to the .albumnavlinks a class (under margin-right:5px;) in the CSS file, but that didnt work.

Can you look into this for us please too?

Beverleyh
08-13-2009, 11:26 AM
OK - I've been playing a little more with the CSS and come up with something that works in IE but not in FireFox or Safari.

I added the CSS below to the appropriate areas of the stylesheet:


.albumnavlinks{ /*CSS for DIV containing the navigational links*/
width: 100%;

.albumnavlinks a, .albumnavlinks a.current:hover{ /*CSS for each navigational link*/
height: 18px;
margin-bottom: 5px;
display: inline;

Which looks like this: http://www.jemcon.org/upload_gallery.php

I'm still having no joy with FireFox or Safari and the navigation button links in the 2nd row still overlap to row above.

If anyone could offer any further assistance, I would be very grateful.

Thanks

Beverley

Beverleyh
08-13-2009, 11:52 AM
HA - sussed it! All that is needed is the line-height attribute in the stylesheet.


.albumnavlinks a, .albumnavlinks a.current:hover{ /*CSS for each navigational link*/
line-height: 25px;

ddadmin, can you please add this to the CSS on the download page too.

Thanks for all your help so far.

Beverley

ddadmin
08-13-2009, 04:28 PM
As soon as the page button links exceed the gallery width and jump to the line below, they overlap the page button links on the line above - see page button link 14:

By button link 14, what do you mean? This sounds like a formatting issue actually.

Beverleyh
08-14-2009, 07:35 AM
I meant the pagination/navigation links (styled to look like buttons) that generate at the bottom of the gallery.

In my example page (which has now been converted to my actual live page) I dumped a hundred or so pics into the image folder to generate 14 pagination/navigation links below the gallery.
Once the pagination/navigation links exceeded the width of the gallery and wrapped to the line below, they overlapped the line of pagination/navigation links above.
After I set the line-height attribute in the CSS, this problem was resolved.

Hope that makes sense.

I just thought it was something that should be added to the CSS download file as a precaution so other users arnt affected.