I use
- this bueditor_plus-7.x-1.0
- bueditor-7.x-1.4
- ajax_markup-7.x-1.0
- wysiwyg-7.x-2.1
- ckeditor-7.x-1.8 as library (module disabled)

wysiwyg setting:
- Full HTML: ckeditor
- Filtered HTML: none
- Plain Text: none

so Text Filter Setting:
- Full HTML: wysiwyg + ckeditor
- Filtered HTML: bueditor
- Plain Text: no editor

Step on create or edit content:
1. Filtered (default) -> bueditor showing, ckeditor gone............ nice
2. change to plain text -> bueditor gone, ckeditor gone ........... nice
3. change to Full HTML -> bueditor gone, ckeditor showing ........... nice
4. change to Filtered HTML -> bueditor gone, ckeditor gone............ <--------------BUG HERE
5. change to plain text -> bueditor gone, ckeditor gone ..................... nice
4. change to Filtered HTML -> bueditor showing, ckeditor gone............ nice

bug happened whenever ckeditor show then change back to bueditor.
so after hiding on ckeditor, bueditor wont show
except if i change to plain text mode (no editor) than to bueditor (extra stepping), it shows again.

Comments

Jamie Holly’s picture

StatusFileSize
new521 bytes

I just did a short fix to get it working. Turns out the WYSIWYG module actually hides text areas and that messes with it. Simply setting the visibility in this module fixed it. I've got the patch attached and have committed the whopping 1 line addition to the 1.x-dev branch.

Jamie Holly’s picture

Assigned: Unassigned » Jamie Holly
Status: Active » Needs review
itz_andr3’s picture

It still happened,
try your patch, and when i try show from ckeditor (Full HTML) to bueditor(Filtered HTML),
bu editor wont show..

i even try to add -> alert("a"); after your patch and it trigers to show message box (js is running).

this don't work, does wysiwyg hide bueditor after runs your js? so it got overlaped?

A litle Update here
When i show Edit summary (open summery field),
bueditor is shown, even when i change back and forward bu and ckeditor,
but only on summary field. not the body.

Jamie Holly’s picture

Out of curiosity, did you clear Drupal's cache after applying the patch?

I did play around with the JS loading order and it doesn't appear to make any difference. I've had it loading before WYSIWYG and BUEditor and after and it worked in all occurrences.

Also, could you tell me what browser you are using? That might help. I've been testing it out on Firefox.

itz_andr3’s picture

StatusFileSize
new15.71 KB
new8.8 KB
new32.54 KB
new14.13 KB
new11.36 KB

The pathc only apply to
sites\all\modules\bueditor_plus\js\bueditor-plus.js right? just adding 1 line?

i try clear drupal cache, and still happened,
and i try to add alert("a"); to test if the js is updated,
and it show "a" masage box

i attach screenshot of step i do, after apply your patch

piece of code after patch

if (self.pset[selectedFormat]){
          BUE.preset[id] = self.pset[selectedFormat];
          	$(self.body.get(0)).show(); // bugfix
          	alert ("a");
          self.editor = BUE.processTextarea(self.body.get(0), self.pset[selectedFormat]);
          if (self.summary.length){
            self.summaryEditor = BUE.processTextarea(self.summary.get(0), self.pset[selectedFormat]);
          }
        }

- im using chrome 18.
- i try on my firefox 8, same problem.
- on ie 8, same problem.

do u use ckeditor on wysiwyg to reproduce?

Jamie Holly’s picture

Yeah I'm using CKEditor + WYSIWYG. Also I am tried in the latest Chrome and it's working there too.

Are you showing any Javascript errors? One thing I notice is that in screenshot #4 your textarea is shrunk down, making me think something is erroring out before resetting the height.

itz_andr3’s picture

++++textarea is shrunk, i do on purpose to capture small screen area, to reduce image size

ok, i newly discover, that the BUEditor placed correctly on summary, not the body content

so i think the "self.editor" on js take the wrong object,

and a don't know when this happened,
i don't realy understand this whole js function getting linked,

ill try to trace it more

itz_andr3’s picture

StatusFileSize
new158.7 KB
new191.92 KB

OK, i do the debug,

i attach my debug, looks like your fix not change the way content showing.

I changing from ckeditor to bueditor, and catch on breakpoint:
- on Debug-1, after your fix $(self.body.get(0)).show(); //bug fix
the self.body.get(0).style.visibility still hiden <-------- .show not working

- on Debug-2, the parameter also return hidden, this passing false to the function

My working solution, i want to propose for release patch:

if (self.pset[selectedFormat]){
          BUE.preset[id] = self.pset[selectedFormat];				
                      $(self.body.get(0)).show();             //bug fix		
                      self.body.get(0).style.visibility = "";	//bug fix
          self.editor = BUE.processTextarea(self.body.get(0), self.pset[selectedFormat]);
          if (self.summary.length){
            self.summaryEditor = BUE.processTextarea(self.summary.get(0), self.pset[selectedFormat]);
          }
        }

adding 2 line of fix:
$(self.body.get(0)).show(); //bug fix
self.body.get(0).style.visibility = ""; //bug fix

This one working for me. (need feedback)

**but got one thing to ask,
The summary text area, is using bueditor also, along with body area, is this the default behaviour??
coz i wysiwyg don't use this kind behavior.

Jamie Holly’s picture

StatusFileSize
new540 bytes

Here's a new patch setting the visibility to 'visible' as well as doing a jquery display. See how that works out.

itz_andr3’s picture

Status: Needs review » Fixed

u make it so simple,

it confirm, it works.
thx

Jamie Holly’s picture

Awesome! I got the changes in the 7.x-1.x-dev branch. Thanks for the help!

Jamie Holly’s picture

Status: Fixed » Closed (fixed)