• Resolved frangoaoalho

    (@frangoaoalho)


    Dear Kris & Fibosearch Team, I hope to find you well.

    Regardless of the setting setup of the plugin, at resolutions between 767px and 784px, even if I display the search as a typing bar instead of as an icon, when someone clicks the typing bar, it triggers the overlaying search table.

    Awaiting feedback,

    The page I need help with: [log in to see the link]

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Support Kris

    (@c0nst)

    Hi @frangoaoalho!

    If you used a shortcode to display FiboSearch in this result you can disable mobile overlay with the?mobile_overlay?attribute. Here is an example:?[fibosearch mobile_overlay="0"]

    Also, you can use mobile_overlay_breakpoint attribute displays the overlay only on selected devices.
    Example: [fibosearch mobile_overlay_breakpoint =”480″] will display mobile overlay on devices with smaller screen width than 480px.

    Here you can learn more about shortcode attributes that you can use to customize FiboSearch.

    Regards,
    Kris

    Thread Starter frangoaoalho

    (@frangoaoalho)

    So, as I am using Elementor and the Fibosearch as a widget and not a short code, can I do like:


    mobile_overlay_breakpoint|480 ?

    • This reply was modified 5 months, 1 week ago by frangoaoalho.
    Thread Starter frangoaoalho

    (@frangoaoalho)

    The one above is not working at all.

    Also, not working:

    mobile_overlay|1
    mobile_overlay_breakpoint|784

    And also, not working with Snippets:

    function modify_fibosearch_widget_attributes($atts) {
    // Modify attributes for FiboSearch widget
    $atts[‘mobile_overlay’] = ‘1’;
    $atts[‘mobile_overlay_breakpoint’] = ‘784’;

    return $atts;

    }

    add_filter(‘dgwt/wcas/widget/shortcode_atts’, ‘modify_fibosearch_widget_attributes’);

    • This reply was modified 5 months, 1 week ago by frangoaoalho.
    Plugin Support Kris

    (@c0nst)

    Hi @frangoaoalho!

    Yes, you can do it with a filter, but you should use dgwt/wcas/shortcode/args instead dgwt/wcas/widget/shortcode_atts . Here is an example:

    add_filter( 'dgwt/wcas/shortcode/args', function( $args ) {
    $args['mobile_overlay_breakpoint'] = 766; // or 480
    $args['layout_breakpoint'] = 766; // or 480
    $args['layout'] = 'flex-icon-on-mobile';

    return $args;
    }, 10 );

    You have two ways to add this code to your theme:

    1. Open the?functions.php?in your child theme and add the code at the end.
    2. ?or install the?Code Snippets?plugin and apply this code as a snippet.

    Regards,
    Kris

    Thread Starter frangoaoalho

    (@frangoaoalho)

    Dear Kris, that is not working at all.
    not even with short code.

    Resolutions are definitely messed up in the php core of the plugin, and set for mobile overlying at 784px regardless of the setup.

    Also the setup available within the Fibosearch menu should be working regardless.

    Plugin Support Kris

    (@c0nst)

    Hi @frangoaoalho!

    You are correct. The isBreakpointReached function in search.js check if the breakpoint has been exceeded using the following logic: $(window).width() <= breakpoint. By setting the breakpoint for the mobile view at 800px, and when changing the viewport width in browser developer tools, it still returned true at 815px, but starting from 816px, clicking the magnifying glass no longer triggers the overlay for mobile devices.
    The solution is replacing $(window).width() with window.innerWidth which works correctly

    We acknowledge the issue and will fix it shortly. Thank you for bringing this to our attention.

    Regards,
    Kris

    Thread Starter frangoaoalho

    (@frangoaoalho)

    Dear kris, I appreciate your prompt response.

    Will be attentively awaiting for the plugin update then ??

    If you want to hire a QA let me know ??

    Marking it as resolved for now. Cheers

    Thread Starter frangoaoalho

    (@frangoaoalho)

    Hi Team, hope to find you well.
    Just to let you know that after 3 months this is still not resolved.

    Any news?

Viewing 8 replies - 1 through 8 (of 8 total)
  • You must be logged in to reply to this topic.