Tagged: x
-
AuthorPosts
-
January 3, 2017 at 5:08 pm #1314934
dpeParticipantcarried 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
January 3, 2017 at 10:10 pm #1315237
Prasant RaiModeratorHello There,
Thanks for writing in!
Could you please provide us with your website URL so we can take a closer look?
Thanks.
January 4, 2017 at 3:26 am #1315445
dpeParticipantJanuary 4, 2017 at 3:39 am #1315461
ThaiModeratorHi 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.
January 4, 2017 at 9:52 am #1315937
dpeParticipantthat seems to have done the trick
thanks
January 4, 2017 at 10:07 am #1315961
Prasant RaiModeratorYou are most welcome. 🙂
January 4, 2017 at 12:58 pm #1316196
dpeParticipantActually 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
January 4, 2017 at 2:49 pm #1316351
DarshanaModeratorHi there,
Keep the original code and you can fill that information under Settings -> General -> Tagline section within your WordPress back-end.
Hope that helps.
January 4, 2017 at 3:05 pm #1316376
dpeParticipantYep that sorted it, many thanks
January 4, 2017 at 5:42 pm #1316531
JadeModeratorYou’re most welcome.
-
AuthorPosts
- <script> jQuery(function($){ $("#no-reply-1314934 .bbp-template-notice, .bbp-no-topic .bbp-template-notice").removeClass('bbp-template-notice'); }); </script>
