Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#1484752 closed Tasks (fixed)

Note about Opera and TinyMCE

Reported by: till Owned by: estadtherr
Priority: 8 Milestone: 0.1-stable
Component: Documentation Version: git-master
Severity: trivial Keywords:
Cc:

Description

Should we add a note that TinyMCE is not compatible with Opera in README? We could even introduce a "known bugs" thing which we have on the wiki.

Change History (9)

comment:1 Changed 5 years ago by till

  • Component changed from Client to Documentation
  • Milestone set to 0.1-stable
  • Priority changed from 5 to 8
  • Severity changed from normal to trivial
  • Version changed from 0.1-rc2 to svn-trunk

comment:2 Changed 5 years ago by tomekp

what version of TinyMCE and what version on Opera was that based on? I've just tried TinyMCE examples on Opera 9.25 and it works without any problems

comment:3 Changed 5 years ago by till

(Of course) Based on the TinyMCE bundled with RoundCube, and 9.25. Maybe ours is way to old then?

comment:4 Changed 5 years ago by till

  • Owner set to estadtherr

comment:5 Changed 5 years ago by till

TinyMCE also just failed me completely in IE7.

I have a JavaScript? debugger which goes like "'null'" is not an object. It's in app.js and the line is kinda like (sorry a bit vague):

var editor = tinymce.getinstance(eid);
(...) = editor.(...)

I thought I had copied it, but I didn't. Shouldn't be too hard to find. Anyway, it complains that "editor" is not an object. Sounds/Looks? like a race condition to me, maybe it ("eid") is checked before the page finished loading.

Perfect example of why we need jquery. ;-)

Anyway, disabling TinyMCE allows me to send email. I am voting to maybe update it but overall set "enable" in config/main.inc.php to FALSE and reevaluate after 0.1-stable.

comment:6 Changed 5 years ago by till

I semi-fixed the issue I had - problem now is that switching between plain <> html leaves me with two TinyMCE's. I think that's because we don't find the instance.

comment:7 Changed 5 years ago by jpingle

Like tomekp, I think it there needs to be some clarification on what supposedly doesn't work.

I just updated to SVN [d5ff9276], and HTML composition works fine with Opera 9.25 on Windows and FreeBSD.

There used to be a separate ticket for the multiple HTML editor fields showing up, but a quick search doesn't find it at the moment.

comment:8 Changed 5 years ago by estadtherr

  • Resolution set to fixed
  • Status changed from new to closed

It appears that Opera is calling the "onchange" event handler on both radio buttons when toggling, whereas Firefox/Safari? only call it once. This uncovered a logic error in the event handler function that changes the editor GUI. This is now fixed.

comment:9 Changed 5 years ago by nereusz

everything is working under opera. the only problem was for me that in opera I couldn't click on any key under the composing textarea (buttons send now!, cancel and in html editor chooser). when I tabbed into this field and clicked enter it crashed. so I searched and I find out that solution:
add style="position:relative;" in <table> tag at line 134 in /skins/default/templates/compose.html and now it's working fine. I suggest to add this to the official release.

Note: See TracTickets for help on using tickets.