search

gjbarnard / moodle-theme_foundation

Foundation theme for Moodle.
GNU General Public License v3.0
8 stars 0 forks source link

Site Logo needs Alt Text for Accessibility #30

Closed emmarichardson closed 5 months ago

emmarichardson commented 6 months ago

Describe the bug When using Foundation, Accessiblity checker are throwing an error because the logo does not include Alt Text. This issue does not happen when I switch to Boost.

To Reproduce Steps to reproduce the behavior:

  1. Go to Home page.
  2. Run any Accessibility Checker.
  3. Check errors.
  4. See that Site Logo is flagged for no Alt Text.

Expected behavior This should include Alt Text.

Versions (please complete the following information):

gjb2048 commented 6 months ago

Dear Emma,

Thanks for the report, I'll look as soon as I can. M4.1 should be running a 4.1.x version of the theme please.

Kind regards,

Gareth

gjb2048 commented 6 months ago

Can replicate. Will fix as soon as I can.

gjb2048 commented 6 months ago

Dear @emmarichardson,

All fixed and patched. Will properly release on Moodle dot org at some point.

G

emmarichardson commented 6 months ago

Thanks, Gareth!

Emma Richardson Distance Learning Coordinator 719-771-4022 Need to visit? Click below to schedule an appointment. https://calendar.app.google/vy9C7FEp2GCNQrbE7

On Sun, Feb 11, 2024, 9:02 AM Gareth J Barnard @.***> wrote:

Dear @emmarichardson https://github.com/emmarichardson,

All fixed and patched. Will properly release on Moodle dot org at some point.

G

— Reply to this email directly, view it on GitHub https://github.com/gjbarnard/moodle-theme_foundation/issues/30#issuecomment-1937763168, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAP3TGIF4HLVMH3MENZVBDTYTDFNVAVCNFSM6AAAAABC4PWNWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZXG43DGMJWHA . You are receiving this because you were mentioned.Message ID: @.***>

gjb2048 commented 5 months ago

Dear Emma,

Released: https://moodle.org/mod/forum/discuss.php?d=456449.

G