jcchavezs / cmb2-conditionals

Plugin to relate fields in a CMB2 metabox
GNU General Public License v2.0
86 stars 61 forks source link

Any reason not to support option pages? #57

Open asadkn opened 6 years ago

asadkn commented 6 years ago

There seems to be no support for option pages by default. Is there any reason why not?

It can be added by modifying the cmb2-conditionals.js file with:

CMB2ConditionalsInit('.cmb2-options-page', '.cmb2-options-page .cmb-field-list');

And by adding the cmb2-conditional.js enqueu manually in the plugins. Now adding the enqueue is easy, but the JS code should exist in cmb2-conditionals.js by default so it doesn't have to be modified.

pogla commented 6 years ago

@asadkn Change this line to:

CMB2ConditionalsInit( '.cmb-form' );

Also, you will have to change this line to something like:

if ( 'admin.php' !== $GLOBALS['pagenow'] || 'arp_options' !== $GLOBALS['plugin_page'] ) {
  return;
}

Or just remove it.

tw2113 commented 6 years ago

Sorry for the GH issue noise, but thanks for the two pointers above for getting this working on options pages. Just what I was needing.

jcchavezs commented 6 years ago

@pogla thanks for helping with the solution. Would you mind opening a PR for such a change?

Den ons. 1. aug. 2018, 01:41 skrev Michael Beckwith < notifications@github.com>:

Sorry for the GH issue noise, but thanks for the two pointers above for getting this working on options pages. Just what I was needing.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jcchavezs/cmb2-conditionals/issues/57#issuecomment-409402542, or mute the thread https://github.com/notifications/unsubscribe-auth/AC7sAr73ugA8TbQAvttgQU38SVjm2Th0ks5uMOsqgaJpZM4UK76e .

--

José Carlos

pogla commented 6 years ago

@jcchavezs Sure: #59