I thought it would be really nice to have this module work with WYSIWG editors, in much the same way that the teaser break button works in WYSIWYG API.

CommentFileSizeAuthor
#4 columns_filter-HEAD_wysiwyg.patch905 bytesdboulet
#4 plugins.zip9.17 KBdboulet

Comments

joachim’s picture

joachim commented

Definitely!
I think I saw the wysiwyg module has an API for Drupal modules to provide button plugins. This would be the way to do it so we only work with that one API and not dozens of individual editors. See http://drupal.org/node/152046

Again, I haven't time to look at this myself, but I'll gladly commit patches.

joachim’s picture

joachim commented

More details here: http://drupal.org/node/372826

dboulet’s picture

dboulet commented
Assigned: Unassigned » dboulet

Great, thanks. I think I'll attempt to tackle this later this week if I get the chance.

dboulet’s picture

dboulet commented
Version: 6.x-1.0 » 6.x-1.1
Status: Active » Needs review
StatusFileSize
new plugins.zip9.17 KB
new columns_filter-HEAD_wysiwyg.patch905 bytes

Ok, here is the patch for the WYSIWYG API plugin. I've included the patch to columns_filter.module and the entire zipped 'plugins' folder seperately—I thought it would be simpler that way. To use the plugin, just unzip the plugins.zip file to the columns_filter module directory after applying the patch.

joachim’s picture

joachim commented
Status: Needs review » Fixed

Great stuff!
Works like a charm :)
The HTML when I switch off the wysiwyg looks dreadful but I assume that's just a consequence of wysiwygs and the rendered node is all fine.

I've committed it, with just a small tweak to the hook implementation -- I doubt we will ever need $type = editors here so an if statement is easier to read than a switch.

joachim’s picture

joachim commented

New release will be up shortly.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.