Support » Plugin: Campaign Monitor Forms by Optin Cat » Bugs/Corrections/Suggestions

  • Resolved david-fanin

    (@david-fanin)


    Hello,

    Thanks for your plugin and the saved time!

    Nevertheless, I found and fixed a bug and i have suggestions to improve your plugin..

    When I validated the form without entering the field, I had no error message.
    The bug was in the file: /assets/script.js.
    I replaced it:

    fca_eoi.invalid_email = $this.children('.fca_eoi_error_texts_email').val();
    fca_eoi.field_required = $this.children('.fca_eoi_error_texts_required').val();

    By:

    fca_eoi.invalid_email = $this.find('.fca_eoi_error_texts_email').val();
    fca_eoi.field_required = $this.find('.fca_eoi_error_texts_required').val();

    In the same file, it would replace the following line:

    $( '.fca_eoi_form_button_element', $this ).attr( 'disabled', 'disabled' );

    By:

    jQuery( '.fca_eoi_form_button_element', $this ).attr( 'disabled', 'disabled' );

    Indeed, for reasons of compatibility, the shortcut ‘$’ jQuery can be disabled.

    In the file ‘/assets/style-new.css’ : I suggest you to remove ‘!important’ rules because it’s difficult to customize the affected properties.

    Bye!

    https://wordpress.org/plugins/campaign-monitor-wp/

Viewing 1 replies (of 1 total)
  • Plugin Author fatcatapps

    (@fatcatapps)

    Hi David,
    Thanks! for your suggestions and method of fixes. We will definitely look into these issues and apply the patches in our future updates.

    Cheers!

Viewing 1 replies (of 1 total)
  • The topic ‘Bugs/Corrections/Suggestions’ is closed to new replies.