It's great to be able to customize FCKeditor through fckeditor.config.js, but the placement of the file seems to me to make it very easy to overwrite by mistake (like when updating the module to a new version). Apparently Drupal's settings.php file had that problem. I know I overwrote a few of those with the default settings.php in Drupal 5. Now in Drupal 6 the file is called default.settings.php and renamed settings.php by the installer. That way you can't overwrite your server settings by uploading a fresh distribution of core. I wonder if a similar approach couldn't be adopted with fckeditor.config.js. Call it default.fckeditor.config.js in the download package and have the installation routine rename it. Or--an option I'm even fonder of--what if the module checked for the presence of fckeditor.config.js in the theme folder and used it if it existed the way the templating system does for tpl.php files? That would keep things all nicely packaged and portable. Whatever the final solution, I think it would be good to do something.
Comments
Comment #1
Jorrit CreditAttribution: Jorrit commentedNice suggestion, I'll add it to our todo list!
Comment #2
JohnAlbinYay! Drupal.org hit 300,000 nodes! Happy nodeday, Drupal! ;-)
Comment #3
Jorrit CreditAttribution: Jorrit commentedI would fix this bug just so I could put this nice number in our changelogs.
Comment #4
TravisCarden CreditAttribution: TravisCarden commentedHaha. Sweet! :-P I'm glad you like the suggestion, too. Thanks for adding it to the todo list. :-)
Comment #5
kenorb CreditAttribution: kenorb commentedGratulation of 300k!;)
Comment #6
chrisschaub CreditAttribution: chrisschaub commentedYou can change the location of the config file. You set the location under the advanced options in the fckeditor drupal config area. This way it can be in your theme directory or wherever. I create a fckeditor directory in my theme directory and locate other fckeditor custom files and plugins there. You can localize all fckeditor settings this way.
Under advanced options:
CustomConfigurationsPath = '/sites/all/themes/sometheme/fckeditor/fckeditor.config.js';
Hope it helps.
Comment #7
TravisCarden CreditAttribution: TravisCarden commentedThis issue is being tackled as #370019: fckeditor.config.js in theme path not recognized in admin