Thanks for this module! It makes it possible to diverts customers from WYSIWYG editors ;-). I have built a small module that adds Markdown quicktags, very easy thanks to the hooks.
On the site I'm working on now I needed some way to turn of quicktags for some roles and some contexts. The patch that I have attached includes the following:
* Adds the permission "access quicktags".
* Adds the display options "Everywhere/Only nodes/Nodes and comments".
* Use drupal_set_html_head(theme_stylesheet_import()) to add stylesheet, making is possible for themes to override the styles.
* Tried to make the code follow Drupals coding style guidelines more closely.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | commentrss-2.patch | 4.36 KB | frjo |
| quicktags-1.patch | 4.16 KB | frjo |
Comments
Comment #1
Bèr Kessels commentedA nice patch, and good features, but I dislike one part.
You introduce " 'Everywhere', 'Only nodes' 'Nodes and comments'" which is a fine starting point, but should certainly not be "the end". We might want to extend this with "only one certain pages", or "if evaled PHP retrurns true".
That is too much to ask from this patch, but I would like to leave that "door" open in this patch.
Hence I would like to see a "quicktags_show_quicktags()" function, wich is called to evaluate if we want to show the quicktags. should simply return TRUE or false. for now all you need to do, is move the permission check and the three switch case statments into there.
Thanks for the hard work and the good cleaning job!
Comment #2
frjo commentedSomething like in this updated patch?
Is there any way to get around the need to turn off "resizable"? It's a neet function.
Comment #3
Bèr Kessels commentedThat is about it. I will commit it tomorrow after some sleep and then some testing :)
About the resizer: a new issue about that is here: http://drupal.org/node/76115
Comment #4
Bèr Kessels commentedCommitted a modified version
Comment #5
(not verified) commented