Merge pull request #261 from mailpoet/form_editor_round_1

Form editor round 1

assets/css/src/admin.styl

@@ -5,13 +5,15 @@
 @require 'common'
 @require 'modal'
 @require 'notice'
-@require 'parsley'
 
 @require 'form_editor'
 @require 'listing'
 @require 'box'
 @require 'breadcrumb'
+
 @require 'form'
+@require 'parsley'
+@require 'form_validation'
 
 @require 'settings'
-@require 'progress_bar'
+@require 'progress_bar'

assets/css/src/form_editor.styl

@@ -4,6 +4,9 @@
 icons = '../img/form_editor_icons.png'
 handle_icon = '../img/handle.png'
 
+#mailpoet_form_name
+  font-size: 23px
+
 #mailpoet_form_history
   display: none
 
@@ -99,6 +102,7 @@ handle_icon = '../img/handle.png'
 /* MailPoet Form wrapper */
 #mailpoet_form_wrapper
   position: relative
+  margin: 20px 0 0 0
 
 /* MailPoet Form container */
 #mailpoet_form_container

assets/css/src/form_validation.styl

@@ -0,0 +1,6 @@
+.parsley-errors-list
+  margin-top: 8px
+
+.parsley-required
+.parsley-custom-error-message
+  color: #b94a48

assets/css/src/public.styl

@@ -1,3 +1,4 @@
 @import 'nib'
 
 @require 'parsley'
+@require 'form_validation'

assets/js/src/form_editor/form_editor.js

@@ -231,7 +231,7 @@ var WysijaHistory = {
 
 /* MailPoet Form */
 var WysijaForm = {
-  version: '0.6',
+  version: '0.7',
   options: {
     container: 'mailpoet_form_container',
     editor: 'mailpoet_form_editor',
@@ -317,6 +317,7 @@ var WysijaForm = {
   save: function() {
     var position = 1,
       data = {
+        'name': $F('mailpoet_form_name'),
         'settings': $('mailpoet_form_settings').serialize(true),
         'body': [],
         'styles': (MailPoet.CodeEditor !== undefined) ? MailPoet.CodeEditor.getValue() : null

assets/js/src/public.js

@@ -20,7 +20,10 @@ function(
       $('form.mailpoet_form').each(function() {
         var form = $(this);
 
-        form.parsley().on('form:submit', function(parsley) {
+        form.parsley({
+          errorsWrapper: '<p></p>',
+          errorTemplate: '<span></span>'
+        }).on('form:submit', function(parsley) {
 
           var data = form.serializeObject() || {};
 

assets/js/src/segments/form.jsx

@@ -42,11 +42,7 @@ define(
         return (
           <div>
             <h2 className="title">
-              Segment <a
+              Segment
-                href="javascript:;"
-                className="add-new-h2"
-                onClick={ this.history.goBack }
-              >Back to list</a>
             </h2>
 
             <Form

assets/js/src/subscribers/form.jsx

@@ -70,11 +70,7 @@ define(
         return (
           <div>
             <h2 className="title">
-              Subscriber <a
+              Subscriber
-                href="javascript:;"
-                className="add-new-h2"
-                onClick={ this.history.goBack }
-              >Back to list</a>
             </h2>
 
             <Form

lib/Config/Initializer.php

@@ -28,6 +28,7 @@ class Initializer {
     $this->setupChangelog();
     $this->setupPublicAPI();
     $this->runQueueSupervisor();
+    $this->setupShortcodes();
     $this->setupHooks();
     $this->setupImages();
   }
@@ -125,6 +126,10 @@ class Initializer {
     $changelog->init();
   }
 
+  function setupShortcodes() {
+    $shortcodes = new Shortcodes();
+    $shortcodes->init();
+  }
   function setupHooks() {
     $hooks = new Hooks();
     $hooks->init();

lib/Config/Menu.php

@@ -374,7 +374,8 @@ class Menu {
         ->findArray(),
       'styles' => FormRenderer::getStyles($form),
       'date_types' => Block\Date::getDateTypes(),
-      'date_formats' => Block\Date::getDateFormats()
+      'date_formats' => Block\Date::getDateFormats(),
+      'month_names' => Block\Date::getMonthNames()
     );
 
     echo $this->renderer->render('form/editor.html', $data);

lib/Config/Shortcodes.php

@@ -0,0 +1,26 @@
+<?php
+namespace MailPoet\Config;
+
+class Shortcodes {
+  function __construct() {
+  }
+
+  function init() {
+    // form widget shortcode
+    add_shortcode('mailpoet_form', array($this, 'formWidget'));
+    add_shortcode('wysija_form', array($this, 'formWidget'));
+  }
+
+  function formWidget($params = array()) {
+    // IMPORTANT: fixes conflict with MagicMember
+    remove_shortcode('user_list');
+
+    if(isset($params['id']) && (int)$params['id'] > 0) {
+      $form_widget = new \MailPoet\Form\Widget();
+      return $form_widget->widget(array(
+        'form' => (int)$params['id'],
+        'form_type' => 'shortcode'
+      ));
+    }
+  }
+}

lib/Form/Block/Base.php

@@ -29,7 +29,13 @@ abstract class Base {
       }
     }
 
-    $validation = '';
+    if($block['type'] === 'radio') {
+      $rules['group'] = 'custom_field_'.$block['id'];
+      $rules['errors-container'] = '.mailpoet_error_'.$block['id'];
+      $rules['required-message'] = __('You need to select at least one option.');
+    }
+
+    $validation = array();
 
     if(!empty($rules)) {
       $rules = array_unique($rules);
@@ -37,10 +43,10 @@ abstract class Base {
         if(is_bool($value)) {
           $value = ($value) ? 'true' : 'false';
         }
-        $validation .= 'data-parsley-'.$rule.'="'.$value.'"';
+        $validation[] = 'data-parsley-'.$rule.'="'.$value.'"';
       }
     }
-    return $validation;
+    return join(' ', $validation);
   }
 
   protected static function renderLabel($block) {
@@ -86,7 +92,11 @@ abstract class Base {
 
   // return field name depending on block data
   protected static function getFieldName($block = array()) {
-    return $block['id'];
+    if((int)$block['id'] > 0) {
+      return 'cf_'.$block['id'];
+    } else {
+      return $block['id'];
+    }
   }
 
   protected static function getFieldLabel($block = array()) {
@@ -98,6 +108,6 @@ abstract class Base {
   protected static function getFieldValue($block = array()) {
     return (isset($block['params']['value'])
             && strlen(trim($block['params']['value'])) > 0)
-            ? trim($block['params']['value']) : '';
+            ? esc_attr(trim($block['params']['value'])) : '';
   }
 }

lib/Form/Block/Radio.php

@@ -9,14 +9,12 @@ class Radio extends Base {
     $field_name = static::getFieldName($block);
     $field_validation = static::getInputValidation($block);
 
-    // TODO: check if it still makes sense
-    // create hidden default value
-    // $html .= '<input type="hidden"name="'.$field_name.'" value="0" '.static::getInputValidation($block).'/>';
-
     $html .= '<p class="mailpoet_paragraph">';
 
     $html .= static::renderLabel($block);
 
+    $html .= '<span class="mailpoet_error_'.$block['id'].'"></span>';
+
     foreach($block['params']['values'] as $option) {
       $html .= '<label class="mailpoet_radio_label">';
 
@@ -24,13 +22,13 @@ class Radio extends Base {
 
       $html .= 'name="'.$field_name.'" ';
 
-      $html .= 'value="'.$option['value'].'" ';
+      $html .= 'value="'.esc_attr($option['value']).'" ';
 
       $html .= (isset($option['is_checked']) && $option['is_checked'])
                 ? 'checked="checked"' : '';
       $html .= $field_validation;
 
-      $html .= ' />&nbsp;'.$option['value'];
+      $html .= ' />&nbsp;'.esc_attr($option['value']);
 
       $html .= '</label>';
     }

lib/Form/Widget.php

@@ -36,9 +36,9 @@ class Widget extends \WP_Widget {
 
     return parent::__construct(
       'mailpoet_form',
-      __("MailPoet Subscription Form"),
+      __('MailPoet Form'),
       array(
-        'title' => __("Newsletter subscription form"),
+        'description' => __('Add a newsletter subscription form.')
       )
     );
   }
@@ -196,24 +196,6 @@ class Widget extends \WP_Widget {
   }
 }
 
-// mailpoet shortcodes
-// form shortcode
-add_shortcode('mailpoet_form',  'mailpoet_form_shortcode');
-add_shortcode('wysija_form',  'mailpoet_form_shortcode');
-
-function mailpoet_form_shortcode($params = array()) {
-  // IMPORTANT: this is to make sure MagicMember won't scan our form and find [user_list] as a code they should replace.
-    remove_shortcode('user_list');
-
-    if(isset($params['id']) && (int)$params['id'] > 0) {
-        $form_widget = new \MailPoet\Form\Widget();
-    return $form_widget->widget(array(
-      'form' => (int)$params['id'],
-      'form_type' => 'shortcode'
-    ));
-    }
-}
-
 // set the content filter to replace the shortcode
 if(isset($_GET['mailpoet_page']) && strlen(trim($_GET['mailpoet_page'])) > 0) {
   switch($_GET['mailpoet_page']) {

lib/Mailer/MailPoet.php

@@ -70,7 +70,7 @@ class MailPoet {
         'Content-Type' => 'application/json',
         'Authorization' => $this->auth()
       ),
-      'body' => json_encode($body)
+      'body' => $body
     );
   }
 }

lib/Router/Forms.php

@@ -139,6 +139,7 @@ class Forms {
 
   function saveEditor($data = array()) {
     $form_id = (isset($data['id']) ? (int)$data['id'] : 0);
+    $name = (isset($data['name']) ? $data['name'] : array());
     $body = (isset($data['body']) ? $data['body'] : array());
     $settings = (isset($data['settings']) ? $data['settings'] : array());
     $styles = (isset($data['styles']) ? $data['styles'] : array());
@@ -187,6 +188,7 @@ class Forms {
 
     $form = Form::createOrUpdate(array(
       'id' => $form_id,
+      'name' => $name,
       'body' => $body,
       'settings' => $settings,
       'styles' => $styles

lib/Router/Subscribers.php

@@ -4,6 +4,7 @@ namespace MailPoet\Router;
 use MailPoet\Listing;
 use MailPoet\Models\Subscriber;
 use MailPoet\Models\SubscriberSegment;
+use MailPoet\Models\SubscriberCustomField;
 use MailPoet\Models\Segment;
 use MailPoet\Models\Setting;
 use MailPoet\Models\Form;
@@ -140,32 +141,37 @@ class Subscribers {
         ? 'unconfirmed' : 'subscribed'
       );
 
-      // // set custom fields
+      // custom fields
-      // $meta_fields = $mailpoet->getOption('mailpoet_subscriber_meta', array());
+      $custom_fields = array();
-      // if(!empty($meta_fields)) {
+      foreach($data as $key => $value) {
-      //   // loop through data to see if any meta field has been passed
+        if(strpos($key, 'cf_') === 0) {
-      //   foreach($meta_fields as $field => $field_data) {
+          $custom_fields[substr($key, 3)] = $value;
-      //     // check if it's a mandatory field
+          unset($data[$key]);
-      //     $is_required = (isset($field_data['params']['required']) && (bool)$field_data['params']['required'] === true);
+        }
-
+      }
-      //     if(array_key_exists($field, $data)) {
-      //       // check if it's a mandatory field
-      //       if($is_required === true && empty($data[$field])) {
-      //         // if it's missing, throw an error
-      //         $errors[] = sprintf(__('"%s" is required'), $field_data['name']);
-      //       } else {
-      //         // assign field to subscriber
-      //         $subscriber[$field] = $data[$field];
-      //       }
-      //     }
-      //   }
-      // }
 
       // insert new subscriber
       $subscriber = Subscriber::createOrUpdate($data);
 
       if($subscriber === false || !$subscriber->id()) {
         $errors = array_merge($errors, $subscriber->getValidationErrors());
+      } else {
+        // add custom fields
+        if(!empty($custom_fields)) {
+          foreach($custom_fields as $custom_field_id => $value) {
+            if(is_array($value)) {
+              // date
+              $value = mktime(0, 0, 0, $value['month'], $value['day'], $value['year']);
+            }
+            $subscriber_custom_field = SubscriberCustomField::create();
+            $subscriber_custom_field->hydrate(array(
+              'subscriber_id' => $subscriber->id(),
+              'custom_field_id' => $custom_field_id,
+              'value' => $value
+            ));
+            $subscriber_custom_field->save();
+          }
+        }
       }
     } else {
       $subscriber->set('status', (

views/form/editor.html

@@ -2,17 +2,12 @@
 
 <% block title %>
   <h2 class="title">
-    <span id="mailpoet_form_name"><%= form.name %></span>
+    <input
-    <input id="mailpoet_form_name_input" type="text" value="" style="display:none;" />
+      type="text"
-    <a
+      placeholder="<%= __('Click to change the form name!') %>"
-      id="mailpoet_form_edit_name"
+      id="mailpoet_form_name"
-      class="add-new-h2"
+      value="<%= form.name %>"
-      href="javascript:;"
+    />
-    ><%= __('Edit name' ) %></a>
-    <a
-      href="<%= admin_url('admin.php?page=mailpoet-forms') | raw %>"
-      class="add-new-h2"
-    ><%= __('List of forms' ) %></a>
   </h2>
 <% endblock %>
 
@@ -122,29 +117,36 @@
       <!-- Toolbar: Shortcodes / Export -->
       <div class="mailpoet_toolbar_section closed" data-section="shortcodes">
         <a href="javascript:;" class="mailpoet_toggle"><br /></a>
-        <h3><%= __('Shortcodes') %></h3>
+        <h3><%= __('Form Placement') %></h3>
 
         <div>
           <!-- Form export links -->
           <p>
-            <%= __("You can easily add this form to your theme's in the [link]Widgets area[/link]")
+            <%= __("Add this form to your sidebar or footer in the [link]Widgets[/link].")
               | replace({
                   '[link]': '<a href="widgets.php" target="_blank">',
                   '[/link]': '</a>'
                 })
               | raw
             %>
-         </p>
+          </p>
           <p>
-            <%= __('%sHTML%s, %sPHP%s, %siframe%s and %sshortcode%s versions are also available.', 'wysija-newsletters')
+            <%= __("Copy and paste this [link]shortcode[/link] into a post or page alternatively.")
+              | replace({
+                  '[link]': '<a href="javascript:;" class="mailpoet_form_export_toggle" data-type="shortcode">',
+                  '[/link]': '</a>'
+                })
+              | raw
+            %>
+          </p>
+          <p>
+            <%= __('%sHTML%s, %sPHP%s and %siframe%s versions are also available.', 'wysija-newsletters')
               | format(
                 '<a href="javascript:;" class="mailpoet_form_export_toggle" data-type="html">',
                 '</a>',
                 '<a href="javascript:;" class="mailpoet_form_export_toggle" data-type="php">',
                 '</a>',
                 '<a href="javascript:;" class="mailpoet_form_export_toggle" data-type="iframe">',
-                '</a>',
-                '<a href="javascript:;" class="mailpoet_form_export_toggle" data-type="shortcode">',
                 '</a>'
               )
               | raw
@@ -254,6 +256,14 @@
     ];
 
     jQuery(function($) {
+      function mailpoet_form_toggle_segments() {
+        // hide list selection if a list widget has been dragged into the editor
+        $('mailpoet_settings_segment_selection')[
+          (($$('#' + WysijaForm.options.editor + ' [wysija_id="segments"]').length > 0) === true)
+          ? 'hide' : 'show'
+        ]();
+      }
+
       function mailpoet_form_fields() {
         // form editor: default fields
         var template = Handlebars.compile(jQuery('#form_template_fields').html());
@@ -269,8 +279,13 @@
           if(response !== false) {
             data.fields  = $.merge(response, data.fields);
           }
+
           // render toolbar
           jQuery('#mailpoet_toolbar_fields').html(template(data));
+
+          setTimeout(function() {
+            WysijaForm.init();
+          }, 1);
         });
       }
       window.mailpoet_form_fields = mailpoet_form_fields;
@@ -299,7 +314,7 @@
       });
 
       // toolbar: open default section
-      $('.mailpoet_toolbar_section[data-section="settings"]')
+      $('.mailpoet_toolbar_section[data-section="fields"]')
         .removeClass('closed');
 
       // form: edit name (in place editor)
@@ -385,6 +400,14 @@
           return false;
         });
 
+        // edit name
+        $('#mailpoet_form_name').on('keyup', function(e) {
+          if(e.which === 13) {
+            $('#mailpoet_form_save').trigger('click');
+            this.blur();
+          }
+        });
+
         // preview form
         $(document).on('click', '#mailpoet_form_preview', function() {
           //mailpoet_form_save(mailpoet_form_preview);
@@ -465,7 +488,7 @@
             action: 'exportsEditor',
             data: $('#mailpoet_form_id').val()
           }).done(function(response) {
-            if(response !== false) {
+            if(response.result !== false) {
               $('#mailpoet_form_export').html(template({ exports: response }));
             }
           });
@@ -483,11 +506,7 @@
           // open popup
           MailPoet.Modal.popup({
             title: "<%= __('Add new field') %>",
-            template: $('#form_template_field_new').html(),
+            template: $('#form_template_field_new').html()
-            onSuccess: function(data) {
-              // toggle widgets
-              WysijaForm.toggleWidgets();
-            }
           });
 
           return false;
@@ -502,7 +521,7 @@
             action: 'get',
             data: id
           }).done(function(response) {
-            if(response !== false) {
+            if(response.result !== false) {
               MailPoet.Modal.popup({
                 title: "<%= __('Edit field') %>",
                 template: $('#form_template_field_new').html(),
@@ -519,14 +538,14 @@
           var name = $(this).siblings('.mailpoet_form_field').attr('wysija_name');
 
           if(window.confirm(
-            "<%= __('Do you really want to delete this custom field?') %>"
+            "<%= __('Deleting this field here will delete the data associated to it for all you users. Confirm you want to delete.') %>"
           )) {
             MailPoet.Ajax.post({
               endpoint: 'customFields',
               action: 'delete',
               data: id
-            }).done(function(response) {
+            }).done(function(result) {
-              if(response === true) {
+              if(result === true) {
                 item.remove();
                 mailpoet_form_fields();
                 MailPoet.Notice.success(
@@ -546,15 +565,13 @@
         });
 
         // get form fields
-        mailpoet_form_fields().done(function() {
+        mailpoet_form_fields();
-          WysijaForm.init();
-        });
 
         // toolbar: segment selection
         var selected_segments = <%= form.settings.segments | json_encode | raw %>;
 
         //  enable select2 for segment selection
-        $('#mailpoet_form_segments').select2({
+        var select2 = $('#mailpoet_form_segments').select2({
           width:'100%',
           templateResult: function(item) {
             if(item.element && item.element.selected) {
@@ -563,7 +580,23 @@
               return item.text;
             }
           }
-        }).select2('val', <%= form.settings.segments | json_encode | raw %>);
+        });
+
+        var hasRemoved = false;
+        select2.on('select2:unselecting', function(e) {
+          hasRemoved = true;
+        });
+        select2.on('select2:opening', function(e) {
+          if(hasRemoved === true) {
+            hasRemoved = false;
+            e.preventDefault();
+          }
+        });
+
+        // set selected values
+        $('#mailpoet_form_segments')
+          .val(<%= form.settings.segments | json_encode | raw %>)
+          .trigger('change');
       });
     });
   </script>

views/form/templates/blocks/date_days.hbs

@@ -1,10 +1,10 @@
-<% set currentDay = 'now'|date("d") %>
+<% set currentDay = 'now' | date('d') | number_format %>
-
 <select id="{{ id }}_days">
   <% for day in 1..31 %>
     <option
-      {{#if params.is_default_today}}
+    <% if(currentDay == day) %>
-        {{#ifCond day "==" "<%= currentDay %>"}}selected="selected"{{/ifCond}}
+      {{#if params.is_default_today}}selected="selected"{{/if}}
-      {{/if}}><%= day %></option>
+    <% endif %>
+    ><%= day %></option>
   <% endfor %>
 </select>

views/form/templates/blocks/date_months.hbs

@@ -1,11 +1,12 @@
 <% set currentMonth = 'now'|date('n') %>
-
 <select id="{{ id }}_months">
   <% for month in 1..12 %>
     <option
-      {{#if params.is_default_today}}
+      <% if(currentMonth == month) %>
-        {{#ifCond month "==" "<%= currentMonth %>"}}selected="selected"{{/ifCond}}
+      {{#if params.is_default_today}}selected="selected"{{/if}}
-      {{/if}}
+      <% endif %>
-    ><%= month |date('1984-' ~ month ~ '-01') |date('F') %></option>
+    >
+    <%= month_names[month - 1] %>
+    </option>
   <% endfor %>
 </select>

views/form/templates/blocks/date_years.hbs

@@ -4,8 +4,9 @@
 <select id="{{ id }}_years">
   <% for year in currentYear..minYear %>
     <option
-      {{#if params.is_default_today}}
+      <% if(currentYear == year) %>
-        {{#ifCond year "==" "<%= year %>"}}selected="selected"{{/ifCond}}
+      {{#if params.is_default_today}}selected="selected"{{/if}}
-      {{/if}}><%= year %></option>
+      <% endif %>
+    ><%= year %></option>
   <% endfor %>
 </select>

views/form/templates/blocks/html.hbs

@@ -1,9 +1,7 @@
 {{#if params.text}}
-  <p>
+  {{#ifCond params.nl2br '==' '1'}}
-    {{#if params.nl2br}}
+    {{{ nl2br params.text }}}
-      {{{ nl2br params.text }}}
+  {{else}}
-    {{else}}
+    {{{ params.text }}}
-      {{{ params.text }}}
+  {{/ifCond}}
-    {{/if}}
-  </p>
 {{/if}}

views/form/templates/blocks/submit.hbs

@@ -1 +1 @@
-<input type="submit" value="{{ params.label }}" />
+<input type="submit" value="{{ params.label }}" class="button" />

views/form/templates/settings/field_new.hbs

@@ -1,9 +1,20 @@
 
-<form id="form_field_new" name="form_field_new" action="" method="post">
+<form
+  id="form_field_new"
+  name="form_field_new"
+  action=""
+  method="post"
+  data-parsley-validate="true"
+>
   {{#if id}}<input type="hidden" id="field_id" name="id" value="{{ id }}" />{{/if}}
   <p>
     <label for="field_type"><%= __('Select a field type:') %></label>
-    <select id="field_type" name="type">
+    <select
+      id="field_type"
+      name="type"
+      data-parsley-required="true"
+      data-parsley-required-message="<%= __('You need to specify a type') %>"
+    >
       <option value="">--</option>
       <option
         {{#ifCond type '==' 'input'}}selected="selected"{{/ifCond}}
@@ -33,7 +44,13 @@
   </p>
   <p>
     <label><%= __("Field's name:") %></label>
-    <input type="text" name="name" value="{{ name }}" />
+    <input
+      type="text"
+      name="name"
+      value="{{ name }}"
+      data-parsley-required="true"
+      data-parsley-required-message="<%= __('You need to specify a name') %>"
+    />
   </p>
   <hr />
 
@@ -49,41 +66,41 @@
 
     $(function() {
       loadFieldForm();
-    });
 
-    $('#form_field_new').on('submit', function(e) {
+      $('#form_field_new').parsley().on('form:submit', function(parsley) {
-      e.preventDefault();
+        // get data
+        var data = $(this.$element).serializeObject();
 
-      // get data
+        // save custom field
-      var data = $(this).serializeObject();
+        MailPoet.Ajax.post({
-      MailPoet.Ajax.post({
+          endpoint: 'customFields',
-        endpoint: 'customFields',
+          action: 'save',
-        action: 'save',
+          data: data
-        data: data
+        }).done(function(response) {
-      }).done(function(response) {
+          if(response.result === true) {
-        if(response.result === true) {
+            mailpoet_form_fields();
-          mailpoet_form_fields();
+            if(data.id) {
-          if(data.id) {
+              MailPoet.Notice.success(
-            MailPoet.Notice.success(
+                "<%= __('Updated custom field “"+data.name+"“') %>"
-              "<%= __('Updated custom field “"+data.name+"“') %>"
+              );
-            );
+            } else {
+              MailPoet.Notice.success(
+                "<%= __('Added custom field “"+data.name+"“') %>"
+              );
+            }
+
+            // close popup
+            MailPoet.Modal.success();
           } else {
-            MailPoet.Notice.success(
+            if(response.errors.length > 0) {
-              "<%= __('Added custom field “"+data.name+"“') %>"
+              $(response.errors).each(function(i, error) {
-            );
+                MailPoet.Notice.error(error);
-          }
+              });
-
-        } else {
-          if(response.errors.length > 0) {
-            for(error in response.errors) {
-              MailPoet.Notice.error(error);
             }
           }
-        }
+        });
+        return false;
       });
-
-      // close popup
-      return MailPoet.Modal.success();
     });
 
     $('#form_field_new #field_type').on('change', function() {

views/form/widget.html

@@ -15,7 +15,6 @@
       #>
       class="mailpoet_form mailpoet_form_<%= form_type %>"
       novalidate
-      data-parsley-validate
     >
       <div class="mailpoet_message"></div>