Right now the validation rules uses it's own custom UI for administering the rules. I believe it should split off the 'Add new validation' part of the page to a local action, and then use the default CTools UI for exports, which means that site builders can easily disable, enable, override, etc the validations. This should also help the module be a little more forward compatible with Drupal 8.

CommentFileSizeAuthor
#2 Validation | drupal7.dev (mysql) - Chromium_036.png24.98 KBDave Reid
#2 Body | drupal7.dev (mysql) - Chromium_037.png69.76 KBDave Reid
#1 1973958-add-local-action.patch2.89 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
1973958-add-local-action.patch2.89 KB

Initial patch for converting the /add callback to a local action.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
FileSize
Body | drupal7.dev (mysql) - Chromium_037.png69.76 KB
Validation | drupal7.dev (mysql) - Chromium_036.png24.98 KB

Screenshots of #1.

g089h515r806’s picture

g089h515r806 CreditAttribution: g089h515r806 commented

There are two UIs for this Project, one is based on Ctools Export UI, the other one is copied from webform validation.
After support Ctools Export UI, I did not delete the old UI and did not improve it.
It is ok for me if you want to Improve the old UI.

Jorrit’s picture

Jorrit CreditAttribution: Jorrit at nCode for DOM Digital Online Media GmbH commented
Issue summary: View changes
Status: Needs review » Needs work

The patch no longer applies.