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.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | wysiwygfix-1518222-9.patch | 540 bytes | Jamie Holly |
| #8 | Debug-2.png | 191.92 KB | itz_andr3 |
| #8 | Debug-1.png | 158.7 KB | itz_andr3 |
| #5 | 1.png | 11.36 KB | itz_andr3 |
| #5 | 2.png | 14.13 KB | itz_andr3 |
Comments
Comment #1
Jamie Holly commentedI 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.
Comment #2
Jamie Holly commentedComment #3
itz_andr3 commentedIt 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.
Comment #4
Jamie Holly commentedOut 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.
Comment #5
itz_andr3 commentedThe pathc only apply to
sites\all\modules\bueditor_plus\js\bueditor-plus.jsright? 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
- 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?
Comment #6
Jamie Holly commentedYeah 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.
Comment #7
itz_andr3 commented++++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
Comment #8
itz_andr3 commentedOK, 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 fixthe
self.body.get(0).style.visibilitystill 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:
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.
Comment #9
Jamie Holly commentedHere's a new patch setting the visibility to 'visible' as well as doing a jquery display. See how that works out.
Comment #10
itz_andr3 commentedu make it so simple,
it confirm, it works.
thx
Comment #11
Jamie Holly commentedAwesome! I got the changes in the 7.x-1.x-dev branch. Thanks for the help!
Comment #12
Jamie Holly commented