• Resolved basile67

    (@basile67)


    Hi,

    When an input field is hidden due conditions, your code adds inline the style=”display: none;” at form-group div class, which hides the field but leaves a blank space.

    It is better to add this style inline at ppom-field-wrapper class which will completely hide the input field without any blank space.

    Thus the form will be better rendered without blank spaces.

    I will apreciate if you add this in next version.

    Thank you.

    • This topic was modified 4 years, 4 months ago by basile67.
Viewing 9 replies - 1 through 9 (of 9 total)
  • Hi,

    ok, thanks for the suggestion. We will take it.

    Thread Starter basile67

    (@basile67)

    Hi,

    As I show you didn’t include this in your last release. Pleas let me know when you will include a solution for this.
    Is there any work around that can be applied for the moment.
    Thank you very much.

    Hi,

    did you try our New conditional logic script, you can enable it from PPOM settings tab under WooCommerce->Settings.

    Thread Starter basile67

    (@basile67)

    Hi,
    Unfortunately it seems not to work well. Do I need to have the pro version for this to work?

    No, it should be working fine with Free version. Make sure you clear the cache.

    Thread Starter basile67

    (@basile67)

    Hi,

    I clear the cache but it doesn’t work. See the link bellow. When the page is loaded the width1 fild must be shown, but it doesn’t.
    When the selection in the first selection box the width2 filed is shown as it is configured.
    If you change back the first selection the width1 is not shown again.
    I double checked the configuration and is correct. So something has to do with the new script.

    https://www.trikkistrom.com/product/somies-michanismos-no1/

    Please check this. Thank you.

    @basile67, I checked your URL and found many JS errors created by your theme and this is why conditions are not working:

    init.js?ver=756114782:2023 Uncaught TypeError: Cannot read property 'height' of undefined
        at init.js?ver=756114782:2023
        at getDivChildren (init.js?ver=756114782:3018)
        at setRowHeight (init.js?ver=756114782:1958)
        at Object.initRow (init.js?ver=756114782:1161)
        at (index):766
    init.js?ver=756114782:2023 Uncaught TypeError: Cannot read property 'height' of undefined
        at init.js?ver=756114782:2023
        at getDivChildren (init.js?ver=756114782:3018)
        at setRowHeight (init.js?ver=756114782:1958)
        at Object.initRow (init.js?ver=756114782:1161)
        at (index):769
    init.js?ver=756114782:2023 Uncaught TypeError: Cannot read property 'height' of undefined
        at init.js?ver=756114782:2023
        at getDivChildren (init.js?ver=756114782:3018)
        at setRowHeight (init.js?ver=756114782:1958)
        at init.js?ver=756114782:3058
        at HTMLDocument.qtyPlusMinus (woocommerce-uncode.js?ver=756114782:315)
        at i (jquery.js?ver=1.12.4-wp:2)
        at Object.fireWith [as resolveWith] (jquery.js?ver=1.12.4-wp:2)
        at Function.ready (jquery.js?ver=1.12.4-wp:2)
        at HTMLDocument.J (jquery.js?ver=1.12.4-wp:2)
    init.js?ver=756114782:2023 Uncaught TypeError: Cannot read property 'height' of undefined
        at init.js?ver=756114782:2023
        at getDivChildren (init.js?ver=756114782:3018)
        at setRowHeight (init.js?ver=756114782:1958)
        at HTMLDocument.<anonymous> (init.js?ver=756114782:3049)
    Thread Starter basile67

    (@basile67)

    Hi, thank you for your quick response.
    I show this but the old script was working well. The only problem was that it added the hidde display style to wrong element and leaves ungly spaces.
    Is it possible to do somthing with the old script?
    Check the link bellow to see what I mean:

    https://www.trikkistrom.com/product/somies-michanismos-no1/

    Hi @basile67,

    Sorry, but now we are moving towards a new conditional logic approach which is more fast and reliable. Please remove JS errors from your site in order to new script work.

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Hidden Input Field due conditions’ is closed to new replies.