Home Forums Pro Themes Corpo Eye Pro Empty HREF

This topic contains 4 replies, has 3 voices, and was last updated by  wensolutions 2 months, 3 weeks ago.

Viewing 5 posts - 1 through 5 (of 5 total)
  • Author
    Posts
  • #87202

    HammondOne
    Participant

    Hi,

    The following has an empty HREF.
    Is there an update? There shouldn’t be any empty HREFs.

    Thanks!

    wp-content\themes\corpo-eye-pro\inc\hook\custom.php

    line 95

    <i class=”fa fa-bars”></i>

    #87293

    wensolutions
    Keymaster

    Hello @hammondone,

    There is the # link, the href is not empty. For your information, the #link should be the same as the menu id or else it won’t work.

    For example in wp-content\themes\corpo-eye-pro\inc\hook\custom.php line no: 95 and 96. The Id and the #link should be same which is “mob-menu” and “#mob-menu” respectively.

    Also, the mobile menu is controlled through the JS code.

    Hope this clears the confusion.

    Thank you.

    Regards!!

    #87324

    HammondOne
    Participant

    What I mean is that there is no clickable element in the tag.

    This is how it outputs to the browser:

    <a id="mobile-trigger" href="#mob-menu"><i class="fa fa-bars"></i></a>

    That is an “empty” href. There is no clickable item within the anchor the tag, so therefore it is empty.

    You cannot click on this: <i class="fa fa-bars"></i>

    HREFs should never be empty.

    #87742

    Molly
    Participant

    This exact issue is giving me fits! It is now the lone error I am getting on my site using multiple accessibility tools. My school is facing a lawsuit over ADA compliance and we have fixed everything but this one empty href error showing up in Google Accessibility Audit, WAVE, SortSite and AChecker. Some of the scan tools also don’t like the tag <i></i> (https://support.siteimprove.com/hc/en-gb/articles/115002726312-Accessibility-Bold-vs-Strong-and-Italic-vs-Emphasis), either. I can’t get a clean scan anywhere to send my lawyer. This is a good theme, please make it come into the modern day.

    #87762

    wensolutions
    Keymaster

    Hello @molly and @hammondone,

    We will further discuss your query and let you know.

    Thank you.

    Happy New Year 🙂

Viewing 5 posts - 1 through 5 (of 5 total)

You must be logged in to reply to this topic.