Home Forums WoodMart support forum Accessibility issue with “WPML Languages” widget in the header builder

Accessibility issue with “WPML Languages” widget in the header builder

Viewing 19 posts - 1 through 19 (of 19 total)
  • Author
    Posts
  • #642013

    eliv3n
    Participant

    When scanning trough https://pagespeed.web.dev/ you will find the accessibility issue both in Mobile and Desktop in the “WPML Languages” widget which is in the Woodmart header builder.

    Accessibility issue:
    Image elements have [alt] attributes that are redundant text.
    This issue is happening when you show the FLAG icon in the language switcher, because the ALT for the flag will be exactly same text as the Title “English” which is showing duplication and that’s why the error is showing.

    Suggestion:
    The ideal is to change the ALT from English to English Language, or to EN or anything rather than the text to prevent the duplication.

    I appreciate if you can create a patch for this issue, or alternatively, provide me with an Snippet to solve this matter.

    Please note that I currently removed the FLAG icon in my website so the accessibility issue is disappeared. it will appear again if I add the Flag icon into the language switcher.

    • This topic was modified 1 week, 6 days ago by Artem Temos.
    #642360

    Luke Nielsen
    Keymaster

    Hello,

    The thing is that the flag icons come from WPML, so please check how it works with some default WordPress themes like WooCommerce Storefront to understand if it is our theme issue or not.

    Kind Regards

    #642759

    eliv3n
    Participant

    Hello Luke, Thank you for your responds.

    I have taken a deep look into this issue and also I have contacted WPML support and we try to remediate this issue with WPML, after week of trying we findout this issue is within the theme not WPML.
    Here is the link to WPML support ticket (https://wpml.org/forums/topic/error-in-alt-attributes-when-scanning-the-website-via-pagespeed-web-dev/#post-16736784)

    To make it clear, WPML also have a language switcher widget which works smoothly on Woodmart, however, the “WPML Languages” widget inside the Header Builder is being handled by Woodmart theme and it’s not being controlled by WPML itself.

    #643044

    eliv3n
    Participant

    Hello Luke,

    Any updates on the issue raised?

    #643074

    Luke Nielsen
    Keymaster

    Hello,

    You can change it on your website by copying the function woodmart_get_wpml_languages_in_mobile_menu from the parent theme, which is located in inc/integrations/wpml.php on line 255 and adding the child theme to function.php and modernizing it there as you need (you can just remove the alt text as now in the WPML menu).

    Kind Regards

    • This reply was modified 1 week, 3 days ago by Artem Temos.
    #643152

    eliv3n
    Participant

    Hello Luke,

    I came across your explanation on this forum thread regarding the accessibility issue with the WPML language switcher.
    https://xtemos.com/forums/topic/accessibility-issue-with-wpml-language-switcher-in-mobile-menu/

    However, I do not have a child theme activated. Instead, I am using the Code Snippets plugin. Would you be able to provide a snippet that I can apply to fix this issue?

    Additionally, this error occurs on both mobile and desktop.

    Thank you in advance for your help!

    #643309

    Luke Nielsen
    Keymaster

    Hello,

    The thing is the same, just copy woodmart_get_wpml_languages_in_mobile_menu function to the code snippets and remove the alt text, you can send me access to the admin panel so I can show on ho to do it.

    Thank you for your time.

    Kind Regards

    #645354

    eliv3n
    Participant

    Hello Luke,

    I sincerely apologize for repeatedly asking you the same question. Would you be able to help me by writing a snippet? I recently purchased a new license and renewed my support, so I would greatly appreciate your assistance in crafting this snippet for me. Thank you!

    #645446

    Artem Temos
    Keymaster

    Hello,

    We will change this alt text in our next theme update.

    Kind Regards

    #646044

    eliv3n
    Participant

    Thank you Artem

    Are you referring to the upcoming Woodmart theme update, or will this be addressed in a patch?

    • This reply was modified 2 weeks, 6 days ago by eliv3n.
    • This reply was modified 2 weeks, 6 days ago by eliv3n.
    #646220

    eliv3n
    Participant

    Just following up on this matter.

    #646310

    Artem Temos
    Keymaster

    Hello,

    It will be included in our 8.1.2 version.

    Kind Regards

    #647718

    eliv3n
    Participant

    Hi Artem,

    It’s been three weeks now, and the patch for this small issue has not been issued. Could you please include it in the next patch? I’ve noticed that other users facing similar issues with WPML are getting fixes through patches rather quickly.

    Additionally, please remove the domain name from post ID #642013 for privacy reasons.

    Thanks for your help!

    • This reply was modified 2 weeks ago by eliv3n.
    #647976

    Artem Temos
    Keymaster

    Hello,

    We will release this update next week. We don’t see your domain anywhere in the first reply. Could you please clarify where we need to remove it?

    Kind Regards

    #647996

    eliv3n
    Participant

    Hi Artem,

    Thank you for your help.

    It is written on the first line in the post #642013

    #647999

    Artem Temos
    Keymaster

    Oh, right, I didn’t notice that. Deleted.

    #648049

    eliv3n
    Participant

    Thank you Artem

    #648239

    eliv3n
    Participant

    Hello Artem.

    Please remove your screenshot from #643074 post.
    it contains the domain name.

    Thank you.

    #648355

    Artem Temos
    Keymaster

    Hello,

    Removed. Please let me know if there is anything else I can help you with.

    Kind Regards

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