385519 - TinyMCE editor not working on Evite site

Status: RESOLVED INCOMPLETE

(Tech Evangelism Graveyard :: English US, defect)

Description

[cmtalbert]

The WYSIWYG editor on the Evite new invitation site does not work with Minefield.  It works in 2.0.0.4.

I am using Minefield on Mac: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a6pre) Gecko/20070622 Minefield/3.0a6pre

Also confirmed this on Windows XP: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a6pre) Gecko/20070622 Minefield/3.0a6pre

== Steps ==
1. Start Minefield
2. Go to evite's "new invitation" site:
http://www.evite.com/app/invitations/create/event.do?template=wild30&customDesign=yes&productPos=1%3A1
3. Scroll down to the "Message To Guests" box.  This is an inactive tinyMCE control, and is supposed to become active when it receives an "onclick".
4. Attempt to write a message in this TinyMCE control box.

== Actual Results ==
You can click on it, double click on it, type on it, and the control does not activate.  Note that if you start typing, the focus runs immediately to the find as you type search, so that suggests this may be a focus issue.

== Expected Results == 
The TinyMCE box will become activated, displaying a style toolbar (with bold, underline etc) and you will be able to type a message in the box.  You can verify this behavior with Firefox 2.0.0.4

== Note ==
You can verify that the normal TinyMCE control works OK on Minefield here: http://wiki.moxiecode.com/examples/tinymce/installation_example_00.php
So, it seems to point to an issue with either focus or with the handling of the onclick event to unhide the control.

Comment 1

[Martijn Wargers (dead)]

This regressed between 2006-03-07 and 2006-03-08:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-03-07+04&maxdate=2006-03-08+09&cvsroot=%2Fcvsroot
So I guess this is somehow a regression from bug 234455.
I have some partly stripped down testcase, but it is still huge, because the site uses tons of javascript libraries.
Maybe Olli has a clause what's going on here.

Comment 2

[Martijn Wargers (dead)]

Attachment: zipped testcase

Like I said, still very huge :(

Comment 3

[Martijn Wargers (dead)]

Attachment: testcase

Ok, I looked at the code a bit more.
I think it relies on the fact Mozilla used to cancel events when the element that was clicked upon was removed.
That's not the case anymore with bug 234455 fixed.
So I think this bug is invalid or a tech evangelism bug.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #3)
> Ok, I looked at the code a bit more.
Thanks!

> I think it relies on the fact Mozilla used to cancel events when the element
> that was clicked upon was removed.
Or more precisely because 1.8 doesn't have event target chain, event 
dispatching may have this kind of side effects if DOM is mutated.

> That's not the case anymore with bug 234455 fixed.
Exactly. Now event target chain is created before event dispatching 
and the chain is kept alive until the end of dispatch.
(http://www.w3.org/TR/2000/REC-DOM-Level-2-Events-20001113/events.html#Events-flow)

> So I think this bug is invalid or a tech evangelism bug.
> 
I agree.

Comment 5

[Martijn Wargers (dead)]

Moving over to tech evangelim.
I guess evite just needs to update their TinyMCE version, as it seems that it works just fine in the TinyMCE demo.

Comment 7

[cmtalbert]

(In reply to comment #1)
> I have some partly stripped down testcase, but it is still huge, because the
Martijn: How did you go about creating this test case?  I looked about for a way to do this, playing with liveHeaders, but I didn't understand if that was sufficient.  Thanks for the edification.

Comment 8

[Martijn Wargers (dead)]

Iirc, I visited the page with javascript turned off, and then saved it to my computer. (then turned js on again)
I then had a page that worked on branch, but not on trunk.
Then it was more or less just a matter of removing manually a lot of the code.

Comment 10

[Bret Reckard]

I'm working with a Director of Engineering at Evite to find a resolution to this bug.  Would be great to have the Evite service available to our FFx 3.0 users.

Comment 12

[Dietrich Ayala (:dietrich)]

Ugh, this forced my wife to use IE for planning our party on the 14th (you're all invited, btw).

Comment 13

[Bret Reckard]

A special thanks to Branden Dyke and Jeff Bailey at Evite for working through this bug.  They are working on a more permanent fix, but the current work around is functioning properly in Firefox 3.0.  Thanks Evite!

Comment 14

[Bret Reckard]

Oh, and I should mention Evite worked hard to get this in place for the launch of FFx 3.0.  The current 21M+ downloads are working well for our Evite users.  Now send out those 4th of July BBQ Evites.

Comment 15

[Tony Chung [:tchung]]

i noticed the HTML edit controls are missing on Fx3 builds for the message body fields, yet they exist on Fx2.  (eg. Bold, underline, indent, etc..)   Can you confirm this is true?

Comment 16

[Martijn Wargers (dead)]

Good question. 

It seems to me this tech evangelism bug is still there.

Comment 17

[Bret Reckard]

Quote from Evite:

"Actually we disabled tinymce for firefox for the time being.

We are working on a new create process that uses a newer version of tinymce that doesn't exhibit the bug described."

Perhaps I was premature in closing the bug, but they are on the path to resolving the issue, and the current solution allows you to complete the Evite process and edit the "message to guests" field.  Albeit, without the more feature rich TinyMCE editor that allows HTML edit controls for Bold, underline, indent, etc..

Thoughts?

Comment 18

[Martijn Wargers (dead)]

(In reply to comment #17)
> Quote from Evite:
> 
> "Actually we disabled tinymce for firefox for the time being.

They disabled it for Firefox 3, you mean? Because it works fine in Firefox 2.
If that's the case, then it's fine for me to close this bug again.

Comment 19

[uxorious]

This is also broken in FF3.1 b1 and b2.

Comment 20

[Bret Reckard]

This seems to work properly in my nightly build on Mac.

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20081211 Shiretoko/3.1b3pre

Anyone else seeing the problem again?

Comment 21

[Chris Lawson (gone)]

INCOMPLETE due to lack of activity since the end of 2009.

If someone is willing to investigate the issues raised in this bug to determine whether they still exist, *and* work with the site in question to fix any existing issues, please feel free to re-open and assign to yourself.

Sorry for the bugspam; filter on "NO MORE PRE-2010 TE BUGS" to remove.