Forums

[resolved] Need to add LogIn link in header that works in Firefox, not just I.E. (5 posts)

  1. dankink
    Member
    Posted 2 years ago #

    Hey guys,

    Well I've searched and found alot of similar questions, but not this exact one.

    This is the site I'm working on. http://www.dankink.com/911/wordpress

    I was able to add a LogIn link using the Codex method. Only thing is that I can't get it to look and behave the same way as the other links by using Firefox. It's only working with Internet Explorer. If you view it through IE, you'll see that the LogIn link is next to the Register link, but in Firefox, the link is over to the left next to the About link and doesn't change when you roll over. It's like it's not really responding to the CSS hover command, but only works in IE.

    I'm working on a mac, so I'm testing with every browser other than IE, and it doesn't work on any of my browsers. When I test on a pc, it works only with IE.

    What could be going on? I've tried messing with the PHP and it's not responding at all.

    Help???

    -Juan

  2. dankink
    Member
    Posted 2 years ago #

    Anybody???

  3. jabecker
    Member
    Posted 2 years ago #

    It appears that the login link is missing <li>...</li> tags. It needs to be part of the list in order to display correctly. I think in FF it's floating under the other tabs and that's why the hover doesn't work. Take a look at the styles that are applied to the other links and do something similar with the login link.

  4. dankink
    Member
    Posted 2 years ago #

    Hmm... that's interesting that it removes it from the list when it publishes it, I hadn't noticed that.
    I'm not sure where I would fix that then, because in the header.php file, it looks like the tags are in the correct place.

    This is the code where I added the loginout command:

    <div align="center"><div id="header">
    
    <ul class="menu">
    <li class="<?php if (((is_home()) && !(is_paged())) or (is_archive()) or (is_single()) or (is_paged()) or (is_search())) { ?>current_page_item<?php } else { ?>page_item<?php } ?>"><a href="<?php echo get_settings('home'); ?>">Home</a></li>
    
    <?php wp_list_pages('sort_column=menu_order&depth=1&title_li='); ?>
    <?php wp_register('<li class="admintab">','</li>'); ?>
    <?php wp_loginout('<li class="login">','</li>'); ?> 
    
    </ul>
    
    </div></div>

    Thanks for pointing that out though. It's a step in the right direction.

  5. dankink
    Member
    Posted 2 years ago #

    Actually, it couldn't have been simpler. All I had to do was add the <li>...</li> tags around the wp_loginout tag and it fixed it. I thought that since it already had some <li> tags within the php tag, it was enough and didnt need more. Guess I was wrong.

    Thanks jabecker!

Topic Closed

This topic has been closed to new replies.

About this Topic