Your Time:
Our Time:
← Back to discussions

Portfolio – display errors

Need help with a different issue? Search the forums or open a new ticket.
Open a support ticket

This topic has 3 replies, 2 voices, and was last updated 12 years, 7 months ago by mlovelock.

Tagged: 

  • 2014-02-05 at 9:22 pm
    Support Expired
    #2788
    mlovelock
    Participant
    • Topics: 1
    • Replies: 2
    • Total posts: 3
    • Post count: 2

    Hi,

    Some errors / display fixes in the single-portfolio.php file

    Line 63 & 119, add space before post date:
    <li><strong><?php echo $projectdatetxt; ?></strong> <?php echo $post_date; ?></li>

    Line 67 & 123, remove unnecessary space before separator comma
    <?php echo strip_tags( get_the_term_list( $post->ID, 'portfolio_cat', '', ', ') ); ?>
    It would also be good to hyperlink these, though displaying by portfolio category seems to have errors at present.

    Line 72 & 128, change ‘portfolio_cat’ to ‘post_tag’ to display skills tags.
    <?php echo strip_tags( get_the_term_list( $post->ID, 'post_tag', '', ', ', '' ) ); ?>

    There also looks to be some errors when selecting to display a ‘sortable’ portfolio (same as portfolio category display), but I’ve not had a chance to delve into that yet.

    Thanks,
    M

    2014-02-06 at 12:38 am
    #2790
    Wajid Khan
    Keymaster
    • Topics: 5
    • Replies: 1543
    • Total posts: 1548
    • Post count: 1876

    If you can provide with WP Logins then we can take a closer look at it. Make sure you set the post as private reply so that only admin can see your wp logins.

    2014-02-06 at 12:44 am
    Support Expired
    #2791
    mlovelock
    Participant
    • Topics: 1
    • Replies: 2
    • Total posts: 3
    • Post count: 2

    Hi,

    Not really necessary as I’ve already fixed it, as per the code above, and am still on a local development site at the moment. Just thought I’d post it here so you were aware for any coming theme updates.

    Thanks,
    M

    2014-02-06 at 12:45 am
    Support Expired
    #2792
    mlovelock
    Participant
    • Topics: 1
    • Replies: 2
    • Total posts: 3
    • Post count: 2

    I’ve marked the post as resolved, just to avoid any confusion 🙂

You must be logged in to reply to this topic.