Forum Replies Created

Viewing 30 posts - 1 through 30 (of 35 total)
  • Author
    Posts
  • in reply to: The Events Calendar Short Codes & Child Theme #1436408

    Thank you for the reply Ismael.
    From what I can finally gather from the plugin provider’s website, they no longer support short code use with their non-pro version of the plugin.
    I ended up modifying the template directly.
    Thanks again.

    Mitchell

    in reply to: Fullwidth Easy Slider #1334813

    Hi Mike,

    It was SiteGround’s plugin.

    Mitch

    in reply to: Fullwidth Easy Slider #1334720

    I found the answer in another post about lazy loading. I turned it off in another plugin and the slider displayed as normal.

    Is this going to be addressed in future releases so we can utilize lazy loading?

    in reply to: Fullwidth Easy Slider #1334718

    I’m seeing this issue as well where there is white space below the first slide on initial load along with the nav controls. After the first slide rotates out, everything displays as normal.
    This started after upgrading to 4.8.8.1
    Lazy loading is turned off and cache has been cleared on server and the client.

    Mitch

    in reply to: Blog Listings as Home Page #1292574

    Thanks Mike,

    I had fixed the issue and forgot to come back here and clean up the thread. Appreciate the help.

    Mitch

    Yigit,

    Thank you. That fixed the problem.

    Mitch

    in reply to: jQuery Issue with WP 5.5 & Enfold 4.7.6.3 #1240558

    Please go ahead and close this ticket.
    Thank you.

    in reply to: jQuery Issue with WP 5.5 & Enfold 4.7.6.3 #1240372

    Victoria,

    Thank you. That did the trick for now.

    Mitch

    Thank you for your help Ismael,

    I got the following to work for me:

    
    
    wp_dequeue_script('avia-megamenu');
    wp_enqueue_script('avia-megamenu', get_stylesheet_directory_uri().'/js/avia-snippet-megamenu.js', array('avia-default'), '', true);

    dfpg,

    The only way I could get it to work was to modify the parent theme file. Recreating the structure/file in my child theme wouldn’t take. The supplied code works. Now it’s just a parent/child issue.

    Mitch

    Thank you Ismael,

    How can I get this into a child theme so the changes aren’t overwritten with the next release?

    Mitch

    in reply to: Accessibility – Skip Navigation #1226214

    Mike,

    There is a growing need to have a WCAG accessible base theme. If you could pass on the need for a skip link and this thread https://kriesi.at/support/topic/wcag-2-1-level-aa-and-landmarks/ to the developers, I would appreciate it. The base theme is already almost there. Just a handful of changes needed….

    Yes. Please go ahead an close this thread.

    Thanks again,

    Mitch

    Ismael,

    What we are looking for is for the sub menu to display when the top level menu item is tabbed to using tab on the keyboard. So if the main menu item has items underneath it, the list of sub-menu items should display just as if I hovered a mouse over the main menu item. From there I can simply keep tabbing down the list.

    To see a working example, go to this page and use tab on your keyboard to cycle through the menu.
    https://open.berkeley.edu/

    Mitch

    in reply to: Accessibility – Skip Navigation #1226144

    Thank you Mike. I had to modify the header.php file anyway due to other non-accessible issues in the base enfold theme.

    I just added the .insertAfter portion of the script to my existing script and will move on…

    Thanks again.

    Mitch

    in reply to: Accessibility – Skip Navigation #1226061

    Mike,

    Thanks but that doesn’t really help. Is the markup in the functions-enfold.php file in the avia_title function?

    Mitch

    in reply to: Accessibility – Skip Navigation #1225651

    Hello,

    Here is a good example of how it should behave without having to add any JavaScript:
    https://www.davidmacd.com/test/skipnav-tabindex.html

    The only way I can get the tab order correct is to do the following:

    1) Target an anchor tag with a name attribute and tabindex set:
    <a id="maincontent" name="maincontent" tabindex="-1"></a>

    2) Add some jQuery (as suggested) to set the input focus, not just the visual focus.

    
    
    $(document).ready(function () {        
            $(".jsf-skip-link").click(function(){           
                $("#maincontent").focus();            
            });
        });

    Targeting the #av_section_1 div with the above did not work.
    Where can I inject my anchor to have it below the breadcrumb section?

    Thank you.

    Mitch

    in reply to: Expand Sub-Menu on Tab Focus #1225622

    Thanks. I’ll take a look at your post. I may have to opt buying the other menu if the theme won’t support tab navigation.

    in reply to: Accessibility – Skip Navigation #1225238

    Mike,

    I targeted the DIV originally with no success. I set it back to target #main so you can see.

    Mitch

    in reply to: Accessibility – Skip Navigation #1224891

    Mike,
    I’ve included a link to the site in the private content area.

    This site is what I modeled the skip link setup after:
    https://knowbility.org/blog/2019/skip-links/

    I’ve included the following link directly after the opening body tag:

    <a href="#maincontent" class="jsf-button jsf-skip-link">Skip to main content</a>
    

    I have CSS applied to hide the link until on focus (after tabbing from address bar).

    I’ve tried different targets (the div, a dedicated anchor) and adding tabindex=’-1′ with no success.

    in reply to: Accessibility – Skip Navigation #1224617

    I’m having the same issue. Did this ever get solved?

    in reply to: WCAG 2.1 Level AA and landmarks #1220909

    To summarize my changes for others with the same issue, the following was made to Enfold version 4.7.5 to address nested landmarks and non-unique landmarks in my child theme.

    File: page.php
    Changed line 26 from
    <main class='template-page content <?php avia_layout_class( 'content' ); ?> units' <?php avia_markup_helper(array('context' => 'content','post_type'=>'page'));?>>
    to
    <div class='template-page content <?php avia_layout_class( 'content' ); ?> units' <?php avia_markup_helper(array('context' => 'content','post_type'=>'page'));?>>
    and on line 39, changed the closing </main> tag to a </div> tag.

    ——————————–
    File: header.php
    Changed line 124 to have a role=’main’ attribute added:
    <div id='main' role='main' class='all_colors' data-scroll-offset='<?php echo avia_header_setting('header_scroll_offset'); ?>'>

    ——————————–
    File: includes/helper-main-menu.php
    Changed line 71 to add an id and aria-label to the <nav> element to make it unique from the main site nav.
    echo "<nav id='my_unique_id' aria-label='my_aria_label' class='sub_menu' ".avia_markup_helper(array('context' => 'nav', 'echo' => false)).">";

    ———————————
    File: config-templatebuilder/avia-shortcodes/section.php
    Created a ‘shortcodes’ folder in my child theme and copied the section.php file into the folder.

    Changed line 1336 from
    $markup = 'main ' . avia_markup_helper( array( 'context' => 'content', 'echo' => false, 'custom_markup' => $custom_markup ) );
    to
    $markup = 'div ' . avia_markup_helper( array( 'echo' => false, 'custom_markup' => $custom_markup ) );

    Changed line 1366 from
    $close_markup = '</main><!-- close content main element -->';
    to
    $close_markup = '</div><!-- close content main element -->';

    Finally, added the following function to the child theme functions.php file.

    function avia_include_shortcode_template($paths)
    {
      $template_url = get_stylesheet_directory();
          array_unshift($paths, $template_url.'/shortcodes/');
    
      return $paths;
    }
    add_filter('avia_load_shortcodes', 'avia_include_shortcode_template', 15, 1);
    

    ———————————

    Mitchell Adams

    in reply to: WCAG 2.1 Level AA and landmarks #1220900

    Hi Ismael,

    Thank you for all your help. I had caught the syntax error and corrected it in my code. At any rate, it still doesn’t work so I’ll just stick with the hard code in the section.php file.

    Is there any chance the landmark issue will be addressed in a future release? There is a large need for WCAG compliant WordPress themes out there. I would imagine there are also business opportunities with companies like Deque.

    Mitch

    in reply to: WCAG 2.1 Level AA and landmarks #1220651

    Ismael,
    Thanks for the link to the shortcodes documentation. I have that piece working.

    The ‘avf_markup_helper_attributes_modified’ function you sent over doesn’t want to take (I removed the extra parenthesis).

    The only way I could get the ‘role’ attribute removed was to modify line 1336 in the section.php file and taking out the ‘context’ => ‘content’ array item.

    Changing
    $markup = 'main ' . avia_markup_helper( array( 'context' => 'content', 'echo' => false, 'custom_markup' => $custom_markup ) );

    To
    $markup = 'div ' . avia_markup_helper( array( 'echo' => false, 'custom_markup' => $custom_markup ) );

    Should I just go that route instead of a function since I already have to modify section.php or will it introduce other complications?

    Thanks

    Mitch
    Mitch

    in reply to: WCAG 2.1 Level AA and landmarks #1219209

    Thank you Ismael,

    I’ve copied the directory structure into my child theme but it doesn’t want to pick up the changes. Only modifying the parent theme file works. Are there other steps I need to take to be able to use the changes in a child theme so changes don’t get overwritten after an update?

    Also, it appears the function ‘avia_markup_helper’ in the file ‘/includes/helper-markup.php’ is injecting ‘role=main’ which is also going to cause a conflict. Can line 69 of that file simply be commented out?

    case 'content':
                    $attributes['role']     = 'main';
                    $attributes['itemprop'] = 'mainContentOfPage';

    Or should I delete the args from the section.php file?

    Thank you.

    Mitch

    in reply to: WCAG 2.1 Level AA and landmarks #1218828

    Hello Ismael,
    Thank you for the help. I’ve applied those changes.

    The example page has a Color Section followed by a 1/1 column. It seems both of those generate HTML containing another <main> element which ends up forming nested <main> elements.

    This HTML getting output is:
    <main role="main" itemprop="mainContentOfPage" class="template-page content av-content-full alpha units">

    See this documenation:
    https://dequeuniversity.com/rules/axe/3.5/landmark-main-is-top-level

    Which files need to be modified to ensure only one <main> or <div role=’main’> element is present on the page?

    Thank you.

    Mitch

    P.S. I’ve copied the page.php file into my child theme and it doesn’t appear to be used.

    • This reply was modified 4 years, 3 months ago by Mitchell Adams. Reason: I noticed the page.php file in my child theme was not getting used
    in reply to: Enfold Update on Level AA of WCAG 2.0 #1215538

    Hello Rikard,

    Thank you for the reply. Yes. I am using the latest version. I started a new thread with the issues I’m working to address.

    Mitchell

    in reply to: Enfold Update on Level AA of WCAG 2.0 #1215092
    This reply has been marked as private.
    in reply to: Easy Slider Captions Hidden Until Mouseover #1113975

    Thank you. That did the trick. Was this unique to our instance or will it be included in the next release?

    Thanks again.

    in reply to: Easy Slider Captions Hidden Until Mouseover #1113308

    Hello,
    I’ve included links to before and after images.
    I’m viewing on Windows 10.
    Chrome version Version 75.0.3770.100 (Official Build) (64-bit)
    Edge version 44.17763.1.0
    Firefox version 67.0.4 (64-bit)

    Chrome and Firefox behave the same way with no caption showing until a mouseover of the image.
    Edge will show a caption but is not sized correctly until a mouseover.

    NOTE: If I remove the link on the slide, the caption displays normally.

    in reply to: Easy Slider Captions Hidden Until Mouseover #1112740
    This reply has been marked as private.
Viewing 30 posts - 1 through 30 (of 35 total)