search

galetahub / ckeditor

Ckeditor 4.x integration gem for rails
https://ckeditor.com/ckeditor-4/
MIT License
2.23k stars 875 forks source link

In gem ckeditor v5.1.2: app/assets/config/manifest.js overrides main manifest.js application configuration (Rails 7.1.3.4) #938

Closed Sega100500 closed 4 months ago

Sega100500 commented 5 months ago

In gem ckeditor v5.1.2: app/assets/config/manifest.js overrides main manifest.js application configuration - all directives of main application are losed after mount ckeditor gem (Rails 7.1.3.4). According to the unspoken rules, the manifest file for a gem should be called: _manifest.js, i.e. in your case ckeditor_manifest.js (I don’t remember exactly where, but this topic was discussed earlier, and it was never reflected in the official Rails documentation).

galetahub commented 5 months ago

I just renamed app/assets/config/manifest.js to app/assets/config/ckeditor_manifest.js and it's worked. All tests passed.

galetahub commented 5 months ago

This will be fixed in the upcomming release 5.1.3

galetahub commented 4 months ago

Fixed in 5.1.3 release