BUG: double tags

  • Resolved Jim Reekes

    (@reekes)


    7 years, 11 months ago

    I can’t say how long this may be true, but if you include a Description in a Flyout menu the text will end with two </span> tags.

    I have included an example showing generated HTML.

    At the end of the mega-menu-description there are two /span tags.

    <li class='mega-menu-item mega-menu-item-type-custom mega-menu-item-object-custom mega-has-description mega-menu-item-25722' id='mega-menu-item-25722'><a class="mega-menu-link" href="https://argonsys.com/products/"><span class="mega-description-group"><span class="mega-menu-title">Product Overview</span><span class="mega-menu-description">General information</span></span></a></li>

    • This topic was modified 7 years, 11 months ago by Jim Reekes.
    • This topic was modified 7 years, 11 months ago by Jim Reekes.
Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author megamenu

    (@megamenu)

    Hi Jim,

    Thanks for getting in touch.

    There is an extra opening span tag that you may not have spotted. When formatted, the code looks like this:

    <li class='mega-menu-item mega-menu-item-type-custom mega-menu-item-object-custom mega-has-description mega-menu-item-25722' id='mega-menu-item-25722'>
    <a class="mega-menu-link" href="https://argonsys.com/products/">
        <span class="mega-description-group">
            <span class="mega-menu-title">Product Overview</span>
            <span class="mega-menu-description">General information</span>
        </span>
    </a>
</li>

    Regards,
    Tom

    Thread Starter Jim Reekes

    (@reekes)

    D’oh! You’re right, I was going blind looking at the lost tag.

    BTW – the error was flagged by https://validator.w3.org, and it seemed like a real error (at least at the time, and now on my fourth reading I see what I wasn’t seeing).

    I appreciate the quick response. I kept checking thinking how unlikely this was an error, but just couldn’t find that extra tag. Trying the validator just now it’s not getting flag, so I’m at a loss. Wonder if I happened to have tripped over a caching bug – whatever, it’s working correctly!

    Thread Starter Jim Reekes

    (@reekes)

    Ah ha! I found the actual bug. It’s in the Yoast plugin’s Breadcrumbs code. It’s adding an extra </span> tag.

    Plugin Author megamenu

    (@megamenu)

    Hi Jim,

    Good to hear you found the source. I agree the w3 validator can be very confusing.. as it doesn’t really understand where things are supposed to start and end (which is understandable!) ??

    Regards,
    Tom

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘BUG: double tags’ is closed to new replies.