• Resolved atb83

    (@atb83)


    Hello
    The plugin works well and is very useful ??
    Some small improvements could be made, such as the name of the filters of the image sizes (when using the base module and yours, and that we do not see the same sizes this little problem posed).
    For example,
    $ Width = (int) apply_filters (‘ACME_et_pb_portfolio_image_width’, 510);
    $ Height = (int) apply_filters (‘ACME_et_pb_portfolio_image_height’, 382);

    A very useful improvement would also be to be able to select blog articles at the same time as customs type post.

    Is this easy to do?

    Thanks for your plugin ??

Viewing 12 replies - 1 through 12 (of 12 total)
  • Plugin Author acmemediakits

    (@acmemediakits)

    Hi! Thank you for using my plugin.
    As per the first question, let me investigate how can I implement this function and I’ll get back to you.

    For the second issue, I don’t understand if you need to display both articles and custom post types in the same module/section or in the same page. Could you please clarify your question, please?

    Take care
    Mirko

    Thread Starter atb83

    (@atb83)

    Thank you ??

    In fact I would like to use the module ‘slider post fullwidth’ with the post classical and articles of the post type ‘project’ (the 2 in the same slider).

    I do not know if I explain correctly?

    Plugin Author acmemediakits

    (@acmemediakits)

    The query that calls the portfolio elements can work only for a post_type at once.

    Anyway you can workaround this issue with the “override permalink” option.

    How to manage this.

    Let’s say you have 3 projects and 1 article you want to display in your portfolio:
    The 3 project run under category “carousel”
    The article url is: https://somesite.dom/my_featured_article/

    Now we will add a new project with the category “carousel”: name it whatever you want, add a featured image and everything you want to be displayed in the carousel/portfolio.
    Now scroll your page to “Custom Fields”, add a new custom field named, for instance, ‘slider_url’ and set it to: ‘https://somesite.dom/my_featured_article/’

    Now go back to Acme Divi Modules settings, open your portfolio preset and fill the field “override permalink” with the value: ‘slider url’

    You can use this workaround to link portfolio items from different post types.

    For SEO purposes, add no-index/no-follow to the fake content page.

    Hope this is clear, let me know if you need further assistance.
    Take care
    m

    Thread Starter atb83

    (@atb83)

    Ok, find good ??
    I would never think of doing that.
    It seems clear to me, I will try and I come back if there is a problem.

    But it should work ??
    thank you very much

    Plugin Author acmemediakits

    (@acmemediakits)

    Hi! Glad to know it is clear ??

    If you like my plugin, please share your love a give it some stars!

    Anyway I will keep this thread open still for some days if you encounter any issue.

    About the image size, I think I will add the feature you suggested in the next release: thank you for your hint.

    Have a good job
    Mirko

    Thread Starter atb83

    (@atb83)

    OK, I will note the plugin no problem ??

    I have a little worry, I have the trick.
    But I have a problem, the link appears well, but I do not have the article asked, I come across the fake project.

    A second question, is it possible to add a Custom Fields, so that the plugin shows or not the project?
    Thanks for your replies ??

    Plugin Author acmemediakits

    (@acmemediakits)

    Please double check you filled the right informations, I have this trick working on a live environments:

    Project Meta Field:
    Project Meta Fields Value

    Preset setting:
    Acme Divi Modules Preset Setting

    As per your second question: yes it is possible to add it, but I think it is easier for you to set up a proper category so you can include only the projects you want to display.

    Thread Starter atb83

    (@atb83)

    Thank you it works ??

    I think I saw a small CSS bug
    When I resize the window, the number of columns I have indicated (to have only one column at any resolution), does not apply.
    I have 1 column at 1600 and higher, only one at 320, but the intermediate resolutions gives me 4, 3 then 2 columns.
    How do you fix that?

    Plugin Author acmemediakits

    (@acmemediakits)

    Hi!
    Those settings are driven by a JS that overrides Divi defaults.

    You are supposed to fill the value for all the 5 resolution breakpoints.
    In order to set only 1 column for all breakpoints you should fill this value:

    View post on imgur.com

    Here you can find a quick tutorial:
    https://dev.acmemk.com/fw-portfolio-products/

    Thread Starter atb83

    (@atb83)

    Yes that’s what I did.
    But, the 3 intermediate resolutions, does not work, and an info with 3 and 2 columns is sent.

    Plugin Author acmemediakits

    (@acmemediakits)

    Hi.
    I guess the override function is triggered too early in your environment.
    You may try to trigger the resize action later.

    You can see the same setup running in this example:

    Anyway, how to increase the function timeout?

    1. open the file /wp-content/plugins/acme-divi-modules/public/js/acme-divi-modules-public.js

    Starting at row 32 we have the following functions:

    
    /**
         * ONLOAD Launcher
         */
        $(document).ready(function () {
            waitForFinalEvent(function () {
                if(false==isFrontBuilder())
                    masonry_setup($('.acme_grid'));
                if($('.acme_carousel').length > 0) {
                    var $et_pb_fullwidth_portfolio = $('.acme_carousel');
                    $et_pb_fullwidth_portfolio.each(function () {
                        var set_container_height = $(this).hasClass('et_pb_fullwidth_portfolio_carousel') ? true : false;
                        set_fullwidth_portfolio_columns($(this), set_container_height);
                    });
                }
            }, 40, 'callbackID');
        });
    
        /**
         * ON WINDOW RESIZE Launcher
         */
        $(window).resize(function () {
            waitForFinalEvent(function () {
                if(false==isFrontBuilder())
                    masonry_setup($('.acme_grid'));
                if($('.acme_carousel').length > 0) {
                    var $et_pb_fullwidth_portfolio = $('.acme_carousel');
                    $et_pb_fullwidth_portfolio.each(function () {
                        var set_container_height = $(this).hasClass('et_pb_fullwidth_portfolio_carousel') ? true : false;
                        set_fullwidth_portfolio_columns($(this), set_container_height);
                    });
                }
            }, 40, 'callbackID');
        });
    

    At both row 46 and row 63 you will find this statement:
    }, 40, 'callbackID');

    The number 40 is currently the timeout value in milliseconds.
    You can increase this value to an arbitrary time. I suggest you to try with 1000 and if the value is working for you, you can decrease that value to the minimum suitable.

    As long as I can’t see your site, I can only guess.

    Take care
    mirko

    Thread Starter atb83

    (@atb83)

    Hello
    I managed to make it work.
    All works ??
    It was a stupid mistake, I had put ‘|’ In place of ‘,’ I had made a copy paste from the example.
    Thanks for the effective help ??

Viewing 12 replies - 1 through 12 (of 12 total)
  • The topic ‘name of filter’ is closed to new replies.