Forum Replies Created
-
Posts
-
@ismael That did it, thank you for sticking with me through all that!!!
I meant @ismael :)
Oops, pinged the wrong username. :) @Ismael
@ishmael It seems to ignore any changes I make to the JS file. Just for testing I’ve added
console.log('blah')to thecurrentLink.on('mouseenter',function. Even deleting the JS file from the main and child theme has no effect. Is it possible I have some setting telling it to use a different menu? Thanks.Hmm. I can get the CSS working, though mine is a little different than yours.
#top .av-main-nav a:focus { text-decoration: underline!important; }As for the focus, still no luck for me. Does this look right? It doesn’t seem to have an effect on the menu’s behavior.
@mitchell Here’s how I did it:
1. Follow the docs to create a child theme: https://kriesi.at/documentation/enfold/child-theme/.
2. Copy the file fromenfold/js/avia-snippet-megamenu.jstoenfold-child/avia-snippet-megamenu.js. After you do this, you should be able to see theavia-snippet-megamenu.jsunderAppearance/Theme Editorin the right sidebar.
3. From there, make the changes that Ishmael listed above.Thanks @Ismael. I appreciate you looking into this. Unfortunately, this doesn’t help people using screen readers. The visibility is
hidden, which means it’s hidden from screen readers too. That means that blind people are unable to navigate any Enford site. That’s _exactly_ what makes _al_l of us open to lawsuits.It’s 2020. Maybe 7 years ago when this theme was created accessibility was an optional feature. It’s not anymore. This theme has been in the top sellers list for years, meaning it’s made millions and millions from us, your customers. With that kind of money, you can afford to keep your theme up to date with the most basic of legal requirements.
Well, I bought and implemented the Ubermenu and it’s totally broken. That’s a half day wasted. I’m really starting to get frustrated with this theme. Navigating a website with the keyboard may have been a “feature” several years ago, but it’s a necessity for a modern website. How is a theme this popular so behind on accessibility? Accessibility is the law in my country and this theme doesn’t meet even the most basic of standards. :(
I just recently asked the same thing here: https://kriesi.at/support/topic/accessibility-issues-with-main-nav-leaves-us-sites-in-danger-of-lawsuits/. I’m a little disappointed with the answer, given that in the US we can get sued for having non-accessible sites. :(
My jQuery isn’t great, but it seems like you could edit the avia-snippet-megamenu.js to respond to focus, as well as hover events. Is that right?
//bind event for mega menu megaItems.each(function(i){ $(this).hover( function() { delayCheck[i] = true; setTimeout(function(){megaDivShow(i); },options.delay); }, function() { delayCheck[i] = false; setTimeout(function(){megaDivHide(i); },options.delay); } ); }); // bind events for dropdown menu dropdownItems.find('li').addBack().each(function() { var currentItem = $(this), sublist = currentItem.find('ul:first'), showList = false; if(sublist.length) { sublist.css({display:'block', opacity:0, visibility:'hidden'}); var currentLink = currentItem.find('>a'); currentLink.on('mouseenter', function() { sublist.stop().css({visibility:'visible'}).animate({opacity:1}); }); currentItem.on('mouseleave', function() { sublist.stop().animate({opacity:0}, function() { sublist.css({visibility:'hidden'}); }); }); } });Thanks @ismael! So there’s no way to make it navigable with the keyboard? Is there a way to use a different menu, not the one built into the theme?
@rikard @guenni007 Thanks so much! Yeah, that looks like it’ll work. I’ll post a link once it’s done.
Thanks @Rikard! The guidelines are unfortunately a little vague, but you need to be able to navigate navigation menus without a mouse. Otherwise there’s no way for many people with disabilities to reach your content.
