danyaPostfactum / modx-ace

ACE editor integration into MODx
24 stars 23 forks source link

Ace inits when not used #43

Open theboxer opened 9 years ago

theboxer commented 9 years ago

Ace inits in resource panel even if not set as RTE...

danyaPostfactum commented 9 years ago

Is this bad? Ace inits when RTE is disabled. Ace can't be set as RTE. This allows to use RTE for specific resources and Ace for others (say no to lame textareas).

theboxer commented 9 years ago

Then I would add an option to load Ace for resource, but loading that automatically doesn't seem right to me

danyaPostfactum commented 9 years ago

Why not? Ace is just a textarea replacement. You can try to uncheck OnDocFormPrerender event on plugin settings page.

theboxer commented 9 years ago

I still don't get why you force user to use it on all textareas

danyaPostfactum commented 9 years ago

As I said, this way allows to use both Ace and RTE for different resources. As for an setting for this, I don't like to add extra settings. I just don't see why syntax highlighting is worse than plain textarea.