I would like the put the service links in the links area of each node, not the node itself. However with this option all the images are treated as

  • inside the entire links
    <div class="links">&raquo; 
      <ul class="links inline">
        <li class="comment_comments first"></li> 
         <li class="comment_new_comments"></li> 
         <li class="node_read_more"></li> 
         <li class="statistics_counter"><span>301 reads</span></li> 
         <li class="service_links_delicious"></li> 
         <li class="service_links_digg">get="_blank"></li> 
         <li class="service_links_facebook"></li> 
         <li class="service_links_google"></li> 
         <li class="service_links_reddit"></li> 
         <li class="service_links_twitter last"></li> 
      </ul>
    </div> 
    

    I removed the href and img tags to make it smaller. But I what I would like to do is have push the service links to a new line either by adding a div around them and positioning or overriding a template or function. I know that if I add them to the node, there will be a div, but I really would like the links inside drupals links area.

    Attached files will show what it happening now, and what I would like.

  • CommentFileSizeAuthor
    future.jpg6.38 KBgenDisarray
    current.jpg6.55 KBgenDisarray

    Comments

    nancydru’s picture

    I wanted a way to style all links, rather than an individual one, so I made one minor change at line 492 in service_links.module:
    $links['link'][$service_id] = theme('service_links_build_link',
    to $links['link']['service-links ' . $service_id] = theme('service_links_build_link',

    If you do that then at least you can do something like .service-links { margin: 0;} which will bring the SL links closer together than the other links, making them look like a group.

    nancydru’s picture

    Status: Needs review » Active

    I don't think that adding a div around them is possible because they are elements in a <ul>. I looked at modifying the code to add a "first" and "last" class to the <li>, but this is non-trivial. I could add them to the <a>, but that looks awkward.

    It might be possible to build the SL list as a normal link list and then add it to the node links as HTML => TRUE, but I don't have time to try that right now.

    nancydru’s picture

    Status: Active » Needs review
    TheCrow’s picture

    Status: Active » Closed (works as designed)

    Probably this have to be fixed in the tpl.php of your theme