Best practice forum (Archived)

This forum discussion has been removed

The content of this forum post has been removed and can no longer be accessed.

Hi, Gareth

Have you checked to ensure that "Custom Totara" is the current theme selected? Please go to Site administration > Appearance > Themes > Theme selector and double-check to ensure the current theme selected is "Custom Totara".

If you are using another theme e.g. "Custom Totara Responsive", you can follow the path to the theme settings to upload a new logo in the same way you did for "Custom Totara" - Site administration > Appearance > Themes > Custom Totara Responsive.

Cheers

Tom

Permalink | Show parent | Reply

I would also like to add that while you are there in the theme selector area, you will need to click on the "clear theme cache" button at the top to make sure your theme changes push out where you can see them right away. Usually after I do that and refresh my browser, I can see the new logo. Best,

Lesli

Permalink | Show parent | Reply

The content of this forum post has been removed and can no longer be accessed.

Permalink | Show parent | Reply

Hi Alan

Does the current logo functionality in the Custom Totara theme not meet accessability standards? If so if you could provide a little more detail on how it should work to meet standards then we'll get it fixed.

Permalink | Show parent | Reply

The content of this forum post has been removed and can no longer be accessed.

Permalink | Show parent | Reply

Hi Alan,

Thanks for the detailed info! I've filed a bug and Brian will be looking at it as part of the ongoing accessibility improvement project.

Permalink | Show parent | Reply

Hi Alan,

Thanks for pointing this out.

I am thinking that we output something along the lines of:
<div class="accesshide">ACME Company</div>
<div id="logo" class="custom"><a href="<?php echo $CFG->wwwroot; ?>"><img class="logo" src="<?php echo $logourl;?>" alt="ACME Company Logo" /></a></div>

The moodle class "accesshide" already does what you intend to do with "toptitle". I am thinking by adding the word "logo" to the alt tag, it will avoid a screen reader saying something along the lines of "... ACME Company, image ACME Company ..." (and say something along the lines of "... ACME Company, image ACME Company Logo ..."). What are your thoughts on that?

Kind Regards,

Brian Barnes

Permalink | Show parent | Reply

On further thoughts, is the company name needed there? I understand that currently there isn't a way to see the company name for a screen reader and I am improving that for 2.7. Having it as a "hidden" div as you have described will not present it in a useful manor for a screen reader (it would appear to be some random text).