How to add active class in top links?Filter product collection for Criteo Tags in category page: module setup (config.xml)Top Menu cache problemHow to echo the configurable product price?Add active class to CMS page links in Top menu (static block links)Failing to add new attribute to orderMagento Top Links Login class disappears randomly!What is use of class names in base/default/template in magento?Add top links via extensionRedirect to external link from top links sectionMagento unable to access block class function from another phtml

How do you make your own symbol when Detexify fails?

Creepy dinosaur pc game identification

How do you respond to a colleague from another team when they're wrongly expecting that you'll help them?

Are Captain Marvel's powers affected by Thanos' actions in Infinity War

Why would a new[] expression ever invoke a destructor?

Hero deduces identity of a killer

What does chmod -u do?

Does an advisor owe his/her student anything? Will an advisor keep a PhD student only out of pity?

Is aluminum electrical wire used on aircraft?

Fear of getting stuck on one programming language / technology that is not used in my country

What are the advantages of simplicial model categories over non-simplicial ones?

Can I still be respawned if I die by falling off the map?

Extract more than nine arguments that occur periodically in a sentence to use in macros in order to typset

I'm the sea and the sun

What is going on with 'gets(stdin)' on the site coderbyte?

Why did the EU agree to delay the Brexit deadline?

Lowest total scrabble score

Store Credit Card Information in Password Manager?

Why is it that I can sometimes guess the next note?

Does Doodling or Improvising on the Piano Have Any Benefits?

Calculating total slots

A social experiment. What is the worst that can happen?

Why is so much work done on numerical verification of the Riemann Hypothesis?

What are some good ways to treat frozen vegetables such that they behave like fresh vegetables when stir frying them?



How to add active class in top links?


Filter product collection for Criteo Tags in category page: module setup (config.xml)Top Menu cache problemHow to echo the configurable product price?Add active class to CMS page links in Top menu (static block links)Failing to add new attribute to orderMagento Top Links Login class disappears randomly!What is use of class names in base/default/template in magento?Add top links via extensionRedirect to external link from top links sectionMagento unable to access block class function from another phtml













2















I want to add active class in top links?




appdesignfrontenddefaultdefaulttemplatepagetemplatelinks.phtml




<?php $_links = $this->getLinks(); ?>
<?php if (count($_links) > 0): ?>
<ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
<?php foreach ($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif; ?>
<?php endforeach; ?>
</ul>
<?php endif; ?>









share|improve this question
















bumped to the homepage by Community 12 mins ago


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.















  • on which condition you want to add active class

    – Bhupendra Jadeja
    Apr 17 '15 at 13:11















2















I want to add active class in top links?




appdesignfrontenddefaultdefaulttemplatepagetemplatelinks.phtml




<?php $_links = $this->getLinks(); ?>
<?php if (count($_links) > 0): ?>
<ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
<?php foreach ($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif; ?>
<?php endforeach; ?>
</ul>
<?php endif; ?>









share|improve this question
















bumped to the homepage by Community 12 mins ago


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.















  • on which condition you want to add active class

    – Bhupendra Jadeja
    Apr 17 '15 at 13:11













2












2








2








I want to add active class in top links?




appdesignfrontenddefaultdefaulttemplatepagetemplatelinks.phtml




<?php $_links = $this->getLinks(); ?>
<?php if (count($_links) > 0): ?>
<ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
<?php foreach ($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif; ?>
<?php endforeach; ?>
</ul>
<?php endif; ?>









share|improve this question
















I want to add active class in top links?




appdesignfrontenddefaultdefaulttemplatepagetemplatelinks.phtml




<?php $_links = $this->getLinks(); ?>
<?php if (count($_links) > 0): ?>
<ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
<?php foreach ($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif; ?>
<?php endforeach; ?>
</ul>
<?php endif; ?>






magento-1.9 module custom






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Feb 9 at 10:26









Satish Dubariya

315112




315112










asked Apr 17 '15 at 11:12









ravi patelravi patel

566418




566418





bumped to the homepage by Community 12 mins ago


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.







bumped to the homepage by Community 12 mins ago


This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.














  • on which condition you want to add active class

    – Bhupendra Jadeja
    Apr 17 '15 at 13:11

















  • on which condition you want to add active class

    – Bhupendra Jadeja
    Apr 17 '15 at 13:11
















on which condition you want to add active class

– Bhupendra Jadeja
Apr 17 '15 at 13:11





on which condition you want to add active class

– Bhupendra Jadeja
Apr 17 '15 at 13:11










2 Answers
2






active

oldest

votes


















0














what about:



<?php $_links = $this->getLinks(); ?>
<?php
// active states
// using $_link->getBeforeText() against the RouteName or Idenifier ti get active states
$routeName = Mage::app()->getRequest()->getRouteName();
$identifier = Mage::getSingleton('cms/page')->getIdentifier();
if ($identifier ==null):
$identifier = $routeName;
endif;
?>
<?php if(count($_links)>0): ?>
<ul class="links"<?php if($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif;?>>
<?php foreach($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract):?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li class="<?php echo $_link->getBeforeText(); if( $identifier == $_link->getBeforeText()): echo " active"; endif; ?> <?php if($_link->getIsFirst()||$_link->getIsLast()): ?><?php if($_link->getIsFirst()): ?>first<?php endif; ?><?php if($_link->getIsLast()): ?> last<?php endif; ?><?php endif; ?>" <?php echo $_link->getLiParams() ?>><?php // echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif;?>
<?php endforeach; ?>
</ul>

<?php endif; ?>





share|improve this answer























  • some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

    – ravi patel
    Apr 17 '15 at 12:52











  • Can you give me an example of same route-name

    – Ahmed Elawadi
    Apr 17 '15 at 12:53











  • "customer" all ways display active class only in my account just see default magento top links.

    – ravi patel
    Apr 17 '15 at 12:57












  • Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

    – Ahmed Elawadi
    Apr 17 '15 at 13:08



















0














A better and more cleaner and also simple approach would be to compare every link's url with the current url and if matches then add "active" class to it.



<?php $_links = $this->getLinks(); ?>
<?php $currentUrl = Mage::helper('core/url')->getCurrentUrl(); ?>

<?php if (count($_links) > 0): ?>
<ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
<?php foreach ($_links as $_link): ?>
<?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
<?php echo $_link->toHtml() ?>
<?php else: ?>
<li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?><?php if($_link->getUrl() == $currentUrl): ?> active<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
<?php endif; ?>
<?php endforeach; ?>
</ul>
<?php endif; ?>


But the urls if same shouldn't have even the difference of last slash or this condition won't work, so you might want to use rtrim($url, '/'); for removing last trailing slash.



Hope this helps.






share|improve this answer






















    Your Answer








    StackExchange.ready(function()
    var channelOptions =
    tags: "".split(" "),
    id: "479"
    ;
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function()
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled)
    StackExchange.using("snippets", function()
    createEditor();
    );

    else
    createEditor();

    );

    function createEditor()
    StackExchange.prepareEditor(
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: false,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: null,
    bindNavPrevention: true,
    postfix: "",
    imageUploader:
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    ,
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    );



    );













    draft saved

    draft discarded


















    StackExchange.ready(
    function ()
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f63959%2fhow-to-add-active-class-in-top-links%23new-answer', 'question_page');

    );

    Post as a guest















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    0














    what about:



    <?php $_links = $this->getLinks(); ?>
    <?php
    // active states
    // using $_link->getBeforeText() against the RouteName or Idenifier ti get active states
    $routeName = Mage::app()->getRequest()->getRouteName();
    $identifier = Mage::getSingleton('cms/page')->getIdentifier();
    if ($identifier ==null):
    $identifier = $routeName;
    endif;
    ?>
    <?php if(count($_links)>0): ?>
    <ul class="links"<?php if($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif;?>>
    <?php foreach($_links as $_link): ?>
    <?php if ($_link instanceof Mage_Core_Block_Abstract):?>
    <?php echo $_link->toHtml() ?>
    <?php else: ?>
    <li class="<?php echo $_link->getBeforeText(); if( $identifier == $_link->getBeforeText()): echo " active"; endif; ?> <?php if($_link->getIsFirst()||$_link->getIsLast()): ?><?php if($_link->getIsFirst()): ?>first<?php endif; ?><?php if($_link->getIsLast()): ?> last<?php endif; ?><?php endif; ?>" <?php echo $_link->getLiParams() ?>><?php // echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
    <?php endif;?>
    <?php endforeach; ?>
    </ul>

    <?php endif; ?>





    share|improve this answer























    • some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

      – ravi patel
      Apr 17 '15 at 12:52











    • Can you give me an example of same route-name

      – Ahmed Elawadi
      Apr 17 '15 at 12:53











    • "customer" all ways display active class only in my account just see default magento top links.

      – ravi patel
      Apr 17 '15 at 12:57












    • Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

      – Ahmed Elawadi
      Apr 17 '15 at 13:08
















    0














    what about:



    <?php $_links = $this->getLinks(); ?>
    <?php
    // active states
    // using $_link->getBeforeText() against the RouteName or Idenifier ti get active states
    $routeName = Mage::app()->getRequest()->getRouteName();
    $identifier = Mage::getSingleton('cms/page')->getIdentifier();
    if ($identifier ==null):
    $identifier = $routeName;
    endif;
    ?>
    <?php if(count($_links)>0): ?>
    <ul class="links"<?php if($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif;?>>
    <?php foreach($_links as $_link): ?>
    <?php if ($_link instanceof Mage_Core_Block_Abstract):?>
    <?php echo $_link->toHtml() ?>
    <?php else: ?>
    <li class="<?php echo $_link->getBeforeText(); if( $identifier == $_link->getBeforeText()): echo " active"; endif; ?> <?php if($_link->getIsFirst()||$_link->getIsLast()): ?><?php if($_link->getIsFirst()): ?>first<?php endif; ?><?php if($_link->getIsLast()): ?> last<?php endif; ?><?php endif; ?>" <?php echo $_link->getLiParams() ?>><?php // echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
    <?php endif;?>
    <?php endforeach; ?>
    </ul>

    <?php endif; ?>





    share|improve this answer























    • some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

      – ravi patel
      Apr 17 '15 at 12:52











    • Can you give me an example of same route-name

      – Ahmed Elawadi
      Apr 17 '15 at 12:53











    • "customer" all ways display active class only in my account just see default magento top links.

      – ravi patel
      Apr 17 '15 at 12:57












    • Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

      – Ahmed Elawadi
      Apr 17 '15 at 13:08














    0












    0








    0







    what about:



    <?php $_links = $this->getLinks(); ?>
    <?php
    // active states
    // using $_link->getBeforeText() against the RouteName or Idenifier ti get active states
    $routeName = Mage::app()->getRequest()->getRouteName();
    $identifier = Mage::getSingleton('cms/page')->getIdentifier();
    if ($identifier ==null):
    $identifier = $routeName;
    endif;
    ?>
    <?php if(count($_links)>0): ?>
    <ul class="links"<?php if($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif;?>>
    <?php foreach($_links as $_link): ?>
    <?php if ($_link instanceof Mage_Core_Block_Abstract):?>
    <?php echo $_link->toHtml() ?>
    <?php else: ?>
    <li class="<?php echo $_link->getBeforeText(); if( $identifier == $_link->getBeforeText()): echo " active"; endif; ?> <?php if($_link->getIsFirst()||$_link->getIsLast()): ?><?php if($_link->getIsFirst()): ?>first<?php endif; ?><?php if($_link->getIsLast()): ?> last<?php endif; ?><?php endif; ?>" <?php echo $_link->getLiParams() ?>><?php // echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
    <?php endif;?>
    <?php endforeach; ?>
    </ul>

    <?php endif; ?>





    share|improve this answer













    what about:



    <?php $_links = $this->getLinks(); ?>
    <?php
    // active states
    // using $_link->getBeforeText() against the RouteName or Idenifier ti get active states
    $routeName = Mage::app()->getRequest()->getRouteName();
    $identifier = Mage::getSingleton('cms/page')->getIdentifier();
    if ($identifier ==null):
    $identifier = $routeName;
    endif;
    ?>
    <?php if(count($_links)>0): ?>
    <ul class="links"<?php if($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif;?>>
    <?php foreach($_links as $_link): ?>
    <?php if ($_link instanceof Mage_Core_Block_Abstract):?>
    <?php echo $_link->toHtml() ?>
    <?php else: ?>
    <li class="<?php echo $_link->getBeforeText(); if( $identifier == $_link->getBeforeText()): echo " active"; endif; ?> <?php if($_link->getIsFirst()||$_link->getIsLast()): ?><?php if($_link->getIsFirst()): ?>first<?php endif; ?><?php if($_link->getIsLast()): ?> last<?php endif; ?><?php endif; ?>" <?php echo $_link->getLiParams() ?>><?php // echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
    <?php endif;?>
    <?php endforeach; ?>
    </ul>

    <?php endif; ?>






    share|improve this answer












    share|improve this answer



    share|improve this answer










    answered Apr 17 '15 at 11:24









    Ahmed ElawadiAhmed Elawadi

    1,438720




    1,438720












    • some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

      – ravi patel
      Apr 17 '15 at 12:52











    • Can you give me an example of same route-name

      – Ahmed Elawadi
      Apr 17 '15 at 12:53











    • "customer" all ways display active class only in my account just see default magento top links.

      – ravi patel
      Apr 17 '15 at 12:57












    • Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

      – Ahmed Elawadi
      Apr 17 '15 at 13:08


















    • some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

      – ravi patel
      Apr 17 '15 at 12:52











    • Can you give me an example of same route-name

      – Ahmed Elawadi
      Apr 17 '15 at 12:53











    • "customer" all ways display active class only in my account just see default magento top links.

      – ravi patel
      Apr 17 '15 at 12:57












    • Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

      – Ahmed Elawadi
      Apr 17 '15 at 13:08

















    some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

    – ravi patel
    Apr 17 '15 at 12:52





    some right but some times 2 or more links active because of route-name or identifier is same for 2 links.

    – ravi patel
    Apr 17 '15 at 12:52













    Can you give me an example of same route-name

    – Ahmed Elawadi
    Apr 17 '15 at 12:53





    Can you give me an example of same route-name

    – Ahmed Elawadi
    Apr 17 '15 at 12:53













    "customer" all ways display active class only in my account just see default magento top links.

    – ravi patel
    Apr 17 '15 at 12:57






    "customer" all ways display active class only in my account just see default magento top links.

    – ravi patel
    Apr 17 '15 at 12:57














    Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

    – Ahmed Elawadi
    Apr 17 '15 at 13:08






    Where I used this script I didn't have any links to customer pages, not sure if getActionName() will work.

    – Ahmed Elawadi
    Apr 17 '15 at 13:08














    0














    A better and more cleaner and also simple approach would be to compare every link's url with the current url and if matches then add "active" class to it.



    <?php $_links = $this->getLinks(); ?>
    <?php $currentUrl = Mage::helper('core/url')->getCurrentUrl(); ?>

    <?php if (count($_links) > 0): ?>
    <ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
    <?php foreach ($_links as $_link): ?>
    <?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
    <?php echo $_link->toHtml() ?>
    <?php else: ?>
    <li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?><?php if($_link->getUrl() == $currentUrl): ?> active<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
    <?php endif; ?>
    <?php endforeach; ?>
    </ul>
    <?php endif; ?>


    But the urls if same shouldn't have even the difference of last slash or this condition won't work, so you might want to use rtrim($url, '/'); for removing last trailing slash.



    Hope this helps.






    share|improve this answer



























      0














      A better and more cleaner and also simple approach would be to compare every link's url with the current url and if matches then add "active" class to it.



      <?php $_links = $this->getLinks(); ?>
      <?php $currentUrl = Mage::helper('core/url')->getCurrentUrl(); ?>

      <?php if (count($_links) > 0): ?>
      <ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
      <?php foreach ($_links as $_link): ?>
      <?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
      <?php echo $_link->toHtml() ?>
      <?php else: ?>
      <li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?><?php if($_link->getUrl() == $currentUrl): ?> active<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
      <?php endif; ?>
      <?php endforeach; ?>
      </ul>
      <?php endif; ?>


      But the urls if same shouldn't have even the difference of last slash or this condition won't work, so you might want to use rtrim($url, '/'); for removing last trailing slash.



      Hope this helps.






      share|improve this answer

























        0












        0








        0







        A better and more cleaner and also simple approach would be to compare every link's url with the current url and if matches then add "active" class to it.



        <?php $_links = $this->getLinks(); ?>
        <?php $currentUrl = Mage::helper('core/url')->getCurrentUrl(); ?>

        <?php if (count($_links) > 0): ?>
        <ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
        <?php foreach ($_links as $_link): ?>
        <?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
        <?php echo $_link->toHtml() ?>
        <?php else: ?>
        <li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?><?php if($_link->getUrl() == $currentUrl): ?> active<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
        <?php endif; ?>
        <?php endforeach; ?>
        </ul>
        <?php endif; ?>


        But the urls if same shouldn't have even the difference of last slash or this condition won't work, so you might want to use rtrim($url, '/'); for removing last trailing slash.



        Hope this helps.






        share|improve this answer













        A better and more cleaner and also simple approach would be to compare every link's url with the current url and if matches then add "active" class to it.



        <?php $_links = $this->getLinks(); ?>
        <?php $currentUrl = Mage::helper('core/url')->getCurrentUrl(); ?>

        <?php if (count($_links) > 0): ?>
        <ul class="links"<?php if ($this->getName()): ?> id="<?php echo $this->getName() ?>"<?php endif; ?>>
        <?php foreach ($_links as $_link): ?>
        <?php if ($_link instanceof Mage_Core_Block_Abstract): ?>
        <?php echo $_link->toHtml() ?>
        <?php else: ?>
        <li<?php if ($_link->getIsFirst() || $_link->getIsLast()): ?> class="<?php if ($_link->getIsFirst()): ?>first<?php endif; ?><?php if ($_link->getIsLast()): ?> last<?php endif; ?><?php if($_link->getUrl() == $currentUrl): ?> active<?php endif; ?>"<?php endif; ?> <?php echo $_link->getLiParams() ?>><?php echo $_link->getBeforeText() ?><a href="<?php echo $_link->getUrl() ?>" title="<?php echo $_link->getTitle() ?>" <?php echo $_link->getAParams() ?>><?php echo $_link->getLabel() ?></a><?php echo $_link->getAfterText() ?></li>
        <?php endif; ?>
        <?php endforeach; ?>
        </ul>
        <?php endif; ?>


        But the urls if same shouldn't have even the difference of last slash or this condition won't work, so you might want to use rtrim($url, '/'); for removing last trailing slash.



        Hope this helps.







        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Nov 6 '15 at 19:46









        Vicky DevVicky Dev

        1,27761539




        1,27761539



























            draft saved

            draft discarded
















































            Thanks for contributing an answer to Magento Stack Exchange!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid


            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.

            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function ()
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f63959%2fhow-to-add-active-class-in-top-links%23new-answer', 'question_page');

            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Can not update quote_id field of “quote_item” table magento 2Magento 2.1 - We can't remove the item. (Shopping Cart doesnt allow us to remove items before becomes empty)Add value for custom quote item attribute using REST apiREST API endpoint v1/carts/cartId/items always returns error messageCorrect way to save entries to databaseHow to remove all associated quote objects of a customer completelyMagento 2 - Save value from custom input field to quote_itemGet quote_item data using quote id and product id filter in Magento 2How to set additional data to quote_item table from controller in Magento 2?What is the purpose of additional_data column in quote_item table in magento2Set Custom Price to Quote item magento2 from controller

            How to solve knockout JS error in Magento 2 Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern) Announcing the arrival of Valued Associate #679: Cesar Manara Unicorn Meta Zoo #1: Why another podcast?(Magento2) knockout.js:3012 Uncaught ReferenceError: Unable to process bindingUnable to process binding Knockout.js magento 2Cannot read property `scopeLabel` of undefined on Product Detail PageCan't get Customer Data on frontend in Magento 2Magento2 Order Summary - unable to process bindingKO templates are not loading in Magento 2.1 applicationgetting knockout js error magento 2Product grid not load -— Unable to process binding Knockout.js magento 2Product form not loaded in magento2Uncaught ReferenceError: Unable to process binding “if: function()return (isShowLegend()) ” magento 2

            Nissan Patrol Зміст Перше покоління — 4W60 (1951-1960) | Друге покоління — 60 series (1960-1980) | Третє покоління (1980–2002) | Четверте покоління — Y60 (1987–1998) | П'яте покоління — Y61 (1997–2013) | Шосте покоління — Y62 (2010- ) | Посилання | Зноски | Навігаційне менюОфіційний український сайтТест-драйв Nissan Patrol 2010 7-го поколінняNissan PatrolКак мы тестировали Nissan Patrol 2016рвиправивши або дописавши її