Hi,

With the 6.x version of WYSIWYG module, when you activate it with tinyMCE (didn't test with others editor as tinyMCE is the only one "supported". But the split summary button (one of the new functionnality of D6) don't appear anymore...

With the editor enabled, I can understand to hide it because it's not very usable to let this button separate from the others of the editor, but, the break function should provide this possibility so (it's not the case atm).

But with editor disabled, the button is still not present, this is not a normal behavior IMHO

CommentFileSizeAuthor
#20 wysiwyg-HEAD.show-summary.patch1.4 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zmove’s picture

Priority: Normal » Critical

As it break a core functionnality, I think it can be threated as critical

sun’s picture

Title: Split summary button is removed » Split summary button removed on non-wysiwyg textareas
Priority: Critical » Normal

I don't know yet whether this can be fixed at all. Leaving as a reminder.

Note: Status "critical" is reserved for issues that lead to fatal errors, render a site unusable, and the like.

zmove’s picture

By reading again my post, I have to precise something.

In fact, it is not really the split summary button that is important, because others editors often display correctly a nice separation between the teaser and the body (eg tinymce with the red break bar).

The more annoying problem is the option to not repeat the teaser in the full node, option that appear as a checkbox in drupal 6 when you split the teaser and the body and that is just removed with TinyMCE in WYSIWYG.

I think just a few modification of the break plugin would solve the problem, I still have some problem to understand well all the mechanism in WYSIWYG and integration with third party module.

zmove’s picture

Hi again,

I tested tiny tinyMCE that successfully implement a nice teaser break integrated in the editor and could let the checkbox to show the teaser in the full body (or not), so I think it's doable. Maybe you can take inpire your code from this module.

sun’s picture

If you are referring to the PageBreak plugin, this TinyMCE plugin is also included in Wysiwyg Editor, named "Break". You just need to enable the button for your wysiwyg profile.

A possible and similar solution for this conflict has been found (and committed) for Image Assist 2.x, which allows to keep Drupal's teaser split button - I hope we can adapt the code from there.

zmove’s picture

Hi,

If it can help you, to generate the body field, drupal call a new 6.x function : node_body_field.

It automatically generate a node body with the slit button and checkbox. The things is to check if it's bring conflixt (javascript or other) with the wysiwyg editors.

Anonymous’s picture

just chiming in with my 2 cents to say that a solution that included both the "break" feature and the option to not include the summary in the full view would be absolutely super!

Thanks!

Anonymous’s picture

If you are referring to the PageBreak plugin, this TinyMCE plugin is also included in Wysiwyg Editor, named "Break". You just need to enable the button for your wysiwyg profile.

Thanks for the tip. Unfortunately, I'm not finding "break" or "pagebreak" in the list of Buttons and plugins at admin/settings/wysiwyg/profile/edit/2.

I do see "pagebreak" listed as a plugin in the TinyMCE directory.

I'm running wysiwyg 6.x-0.5

Am I looking in the wrong place?

Thanks!

sun’s picture

True. The "Teaser break" plugin is only available for TinyMCE 2. Support for TinyMCE 3.x will be available when #319363: Rewrite editor plugin API is done.

zmove’s picture

Yes, but this thread is not only related to the teaser break button. It is for the checkbox that allow to not show the node teaser on the node full view. This is a new feature of drupal 6, and it would be annoying to loose it installing a WYSIWYG editor.

Anonymous’s picture

True. The "Teaser break" plugin is only available for TinyMCE 2. Support for TinyMCE 3.x will be available when #319363: Rewrite editor plugin API is done.

ah. Got it. Thanks for the clarification!

pillarsdotnet’s picture

I have the wysiwyg module installed with the rewrite-editor-plugin-api patch located at http://drupal.org/node/319363#comment-1178449

Any hints on what it would take to enable the "teaser break" plugin for tinymce-3 ?

I've tried disabling javascript aggregation and enabling the plugin, but the button still does not appear. Nor does the option to show/hide the teaser in full-node view.

If there is a javascript function that would make this button appear, can you tell me what it is so that I can try inserting it into the "none.js" file?

Tried with tinymce-2 and still can't control teasers. Perhaps excerpt module will help?

lathan’s picture

following.

lacitpo’s picture

me too

petey318’s picture

subscribing.

I need to have the ability to NOT include the teaser information in the main body of the node. This is is a great feature of D6.

petey318’s picture

subscribing.

I need to have the ability to NOT include the teaser information in the main body of the node. This is is a great feature of D6.
Edit: no idea how this comment got here twice!! (had a browser issue when I typed in the original)

Anonymous’s picture

I need to have the ability to NOT include the teaser information in the main body of the node. This is is a great feature of D6.

I totally agree. It would be awesome for the users of our sites to have both a WYSIWYG editor and the ability to have a different teaser and body.

J V’s picture

same, need to disclude the teaser from the body...

J V’s picture

Version: 6.x-1.x-dev » 6.x-1.1

Might as well change this, mabey they will pay some attention then...

sun’s picture

Title: Split summary button removed on non-wysiwyg textareas » Show summary in full view checkbox not displayed
Status: Active » Fixed
FileSize
1.4 KB

Committed attached patch to all 6.x branches.

Don't even try to re-open this issue to ask for the teaser splitter. You should rather prepare yourself that the Teaser splitter is no longer existing in Drupal 7.

petey318’s picture

You should rather prepare yourself that the Teaser splitter is no longer existing in Drupal 7.

Where could I read more about this? Perhaps I have misunderstood what was written.

sun’s picture

Status: Fixed » Closed (fixed)

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

spython’s picture

Hm, any idea how to hide this checkbox again (as it is not needed on my site), without hacking this module?

TheRec’s picture

Little bit off topic, but you could use hook_form_alter, in a custom module :

function <custom_module_name>_form_alter(&$form, $form_state, $form_id) {
  if(isset($form['body_field']['teaser_include']) && preg_match('/.+_node_form$/i',$form_id)) {
    unset($form['body_field']['teaser_include']);
  }
}

This would remove this particulat checkbox for every node add/edit form, no matter the node type. If you need it for the only a certain node type just tweak the second part of the condition.

davegan’s picture

subscribe

davegan’s picture

subscribe

SuperContraXTC’s picture

Thanx
#25 TheRec

I created a custom module using your code and it work. COOL!

function hidethisstupidshowthingy_form_alter(&$form, $form_state, $form_id) {
if(isset($form['body_field']['teaser_include']) && preg_match('/.+_node_form$/i',$form_id)) {
unset($form['body_field']['teaser_include']);
}
}

buzzman’s picture

if you insert the code in #25 in the existing node_form_alter function in node.module, it raises a new issue!

if you are using TinyMCE as your editor along with the WYSIWYG module, then it disables the editor from showing up on the "add" node (any type of node) form.

but for existing node "edit", the editor does get loaded.

i'll be testing further and feedback.

has anyone else noticed this behavior?

EDIT: i double checked, and maybe cuz of my turning options on/off in "buttons & plugins" part of config in admin the issue seems now to have gone away ... strange, but works for me :)

lotyrin’s picture

sub

asb’s picture

Am I understanding this correct: Having the 'WYSIWYG' module in D6 enabled on a site kills the teaser break button for all input formats, and there's nothing we can do about it but to completely uninstall everything based on WYSIWYG; because in D7 teasers will be handled differently, this core-functionality-breaking issue is not even considered as a bug on current D6 sites?

doitDave’s picture

Hi. I am not reopening this, Daniel, but instead just about to write a small add-on module which probably helps out at least a little bit. (I am talking about D6 only here since I am still using it for quite a while.)

Basic thought is here: Either a user wants to use a Rich Text Editor - then he'll be more than satisfied with the existing solutions. Or he does not like rich text at all, so we can carry him around the "critical" point (i.e. modifications wysiwyg.module does in hook_form_alter).

I hope you won't mind me coding and providing this, sun. My only request would be "please do not change the way wysiwyg.module hooks into the D6 form generation anymore" :-)

The core functionality is already working. As soon as I have implemented the configuration options (for now, I will restrict them to "per user" and probably "per node type"), I will provide it.

Hopefully that might serve the one or the other one's desires a bit.

BTW, thanks a lot for the great work on wysiwyg.module. Simply magnificent plugin. Honestly!

jawad.shah’s picture

Ref #25
thanks TheRec it works for me as well. . :-)

Zeropoint.it