WordPress.org

1. piratepictures
   Member
   Posted 2 years ago #

   I made a custom attachment view using image.php and put in the next/prev links below the photo. i also added effects to make the links fade on hover and show "previous image" and "next image" text underneath.

   problem is that i used divs to place the text underneath and now on the very first page (and the very last page) i have the "previous image" (and "next image") text showing but obviously the php prev/next image links are gone. this messes up the page completely...

   what i think i need is a php code for not showing the "previous image" div on the first page but i dont know coding at all. it should be really easy to do i presume...

   it's a little confusing to explain so please check out the following:

   normal: http://www.missprestin.com/wordpress/?attachment_id=127
   first (or last) attachment pages: http://www.missprestin.com/wordpress/?attachment_id=126

   thanks in advance.

2. alchymyth
   The Sweeper
   Posted 2 years ago #

   you could probably work with a conditional statement to check if the thumbnail exists, and if not use a default thumbnail in its place.

   could you paste the code of your image.php into a pastebin and post the link to it here?

3. piratepictures
   Member
   Posted 2 years ago #

   thanks for getting back.

   here's the pastebin link.

   i think what you're saying would work. and actually, it might make it much easier than what I was thinking which was to remove the 'previous pic' text. first, because i'm really not sure how i'd even do that and second because it would mess up the float alignment of my comment box, which it already has...

4. alchymyth
   The Sweeper
   Posted 2 years ago #

   the reason seems to be that '.previmglink' does not have any width values set in stytle.css, so if it is empty, it just collapses.

   try to add these settings to style.css:
   

   .previmglink {width:110px;margin-right:50px;
     height:110px;display:block;}
   .nextimglink {width:110px;margin-right:30px;
     height:110px;display:block;}

   and then edit line 36 in your file:
   <div class="alignright toppadding5"><div class="nextimgtxt">NEXT PIC</div><div class="previmglink"><?php next_image_link() ?></div></div>

   change it to:
   <div class="alignright toppadding5"><div class="nextimgtxt">NEXT PIC</div><div class="nextimglink"><?php next_image_link() ?></div></div>

   hope this works, (if anything, it may need adjustment of the margin values in the extra styles).

5. piratepictures
   Member
   Posted 2 years ago #

   perfect, that stops it from collapsing.

   but i still have the problem of the 'previous image' text showing up on the first photo page without there being any thumbnail.

   I dont know how to write a conditional statement so that when there is no 'previous image' thumbnail to not show the 'previous image' text either.

   any ideas?

   the css for that section is here

6. alchymyth
   The Sweeper
   Posted 2 years ago #

   i can't see any conditional statement using wordpress templates and php to do this, using my limited abilities.
   unfortunately, the core function echos the link ( http://phpxref.com/xref/wordpress/wp-includes/media.php.source.html#l709 ) instead of returning it, so it can't be used easily in a conditional statement.

   in extreme, you could use php ob_start() function:

   change from this code:
   <div class="alignleft toppadding5"><div class="previmgtxt">PREV IOUS PIC</div><div class="previmglink"><?php previous_image_link() ?></div></div>

   to this code:
   

   <div class="alignleft toppadding5"><div class="previmgtxt">
   <?php ob_start(); previous_image_link(); $link = ob_get_contents(); ob_end_clean(); if($link != '') { echo 'PREV IOUS PIC'; } ; ?></div><div class="previmglink"><?php previous_image_link() ?></div></div>

   totally untested -

   make backup copies before trying

7. piratepictures
   Member
   Posted 2 years ago #

   actually, that got it half way there. the text doesnt show up but the black box is still there.

   anyway to put the black box inside that code as well?

8. alchymyth
   The Sweeper
   Posted 2 years ago #

   <div class="alignleft toppadding5">
   <?php ob_start(); previous_image_link(); $link = ob_get_contents(); ob_end_clean(); if($link != '') { echo '<div class="previmgtxt">PREV IOUS PIC'</div>; } ; ?><div class="previmglink"><?php previous_image_link() ?></div></div>

   the only concern is that you may get alignment issues if the div with the black background is not there - but give it a try.

9. piratepictures
   Member
   Posted 2 years ago #

   yah, i figured i might have trouble with that. i'll try. maybe i can realign the comments to the right or margin them or something...

10. piratepictures
    Member
    Posted 2 years ago #

    Parse error: syntax error, unexpected '/' in /home/content/m/i/s/missprestin/html/wordpress/wp-content/themes/pixel/image.php on line 26

    hmmmm.

11. alchymyth
    The Sweeper
    Posted 2 years ago #

    my bad - got the insertion of the closing div wrong - here the corrected version:

    <div class="alignleft toppadding5">
    <?php ob_start(); previous_image_link(); $link = ob_get_contents(); ob_end_clean(); if($link != '') { echo '<div class="previmgtxt">PREV IOUS PIC</div>'; } ; ?><div class="previmglink"><?php previous_image_link() ?></div></div>

12. piratepictures
    Member
    Posted 2 years ago #

    that did it. no alignment issues or anything else that i can see. works in chrome, firefox and IE.

    so glad that worked out, thanks a lot.

13. alchymyth
    The Sweeper
    Posted 2 years ago #

    well done,
    just leaves the Next Pic link (?)

    you should be able to tackle that yourself ;-)

14. piratepictures
    Member
    Posted 2 years ago #

    yep, i can do that. just totally forgot....

    thanks again!!

Topic Closed

This topic has been closed to new replies.