Pilot-in / PiloPress

The most advanced WordPress Page Builder using Advanced Custom Fields & TailwindCSS
https://www.pilopress.com
62 stars 11 forks source link

Updating a Pilo'Press layout field group erase TailwindCSS overrided config #225

Closed DamChtlv closed 1 year ago

DamChtlv commented 1 year ago

Describe the bug Updating a Pilo'Press layout field group erase TailwindCSS override config

To Reproduce

Expected behavior No impact on TailwindCSS config when updating a Pilo'Press layout field group

Suggested fix Add:

if ( !$tailwind_config ) {
  return;
}

before if ( $override_config ) : statement in here: https://github.com/Pilot-in/PiloPress/blob/679475a3f9eb3c0eb8f1a82c2cb7d2c695e604cc/includes/classes/admin/modules/class-tailwind.php#L310-L317

Screenshots image(7)

DamChtlv commented 1 year ago

Already fixed in: https://github.com/Pilot-in/PiloPress/commit/3cadf2dfd09129b398ae17c44658365a2c47532e