Navigation
This is archived content. Visit our new forum.

Tagged: 

  • Author
    Posts
  • #1314934
    dpe
    Participant

    carried out a few checks for SEO and noticed this in the code <h1 class=”visually-hidden”></h1> – my understanding is that there should only be one set of H1 tags on a page, surely better to not be there than use the visually hidden class

    #1315237
    Prasant Rai
    Moderator

    Hello There,

    Thanks for writing in!

    Could you please provide us with your website URL so we can take a closer look?

    Thanks.

    #1315445
    dpe
    Participant
    #1315461
    Thai
    Moderator

    Hi There,

    Because this requires a template change, I’d advise that you setup a child theme. This allows you to make code changes that won’t be overwritten when an X update is released. After your child theme is setup, please review how we recommend making template changes in Customization Best Practices.

    Create _brand.php file inside YOUR CHILD THEME -> framework -> views -> global and add this code in there:

    <?php
    
    // =============================================================================
    // VIEWS/GLOBAL/_BRAND.PHP
    // -----------------------------------------------------------------------------
    // Outputs the brand.
    // =============================================================================
    
    $site_name        = get_bloginfo( 'name' );
    $site_description = get_bloginfo( 'description' );
    $logo             = x_make_protocol_relative( x_get_option( 'x_logo' ) );
    $site_logo        = '<img src="' . $logo . '" alt="' . $site_description . '">';
    
    ?>
    
    <a href="<?php echo home_url( '/' ); ?>" class="<?php x_brand_class(); ?>" title="<?php echo $site_description; ?>">
      <?php echo ( $logo == '' ) ? $site_name : $site_logo; ?>
    </a>

    Thank you.

    #1315937
    dpe
    Participant

    that seems to have done the trick

    thanks

    #1315961
    Prasant Rai
    Moderator

    You are most welcome. 🙂

    #1316196
    dpe
    Participant

    Actually there is still a bug;

    I replaced this line (because there was no Alt tag for the logo image;

    $site_description = get_bloginfo( ‘description’ );

    with

    $site_description = “Dorset Event Photographer Mike Weeks”;

    would be great to get it working properly

    Mike

    #1316351
    Darshana
    Moderator

    Hi there,

    Keep the original code and you can fill that information under Settings -> General -> Tagline section within your WordPress back-end.

    Hope that helps.

    #1316376
    dpe
    Participant

    Yep that sorted it, many thanks

    #1316531
    Jade
    Moderator

    You’re most welcome.

  • <script> jQuery(function($){ $("#no-reply-1314934 .bbp-template-notice, .bbp-no-topic .bbp-template-notice").removeClass('bbp-template-notice'); }); </script>