800850 - Styling buttons to insert bbcode broken on http://www.security-helpzone.com/ since Firefox 16

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: French, defect)

Description

[Pascal Chevrel:pascalc]

1/ go to http://www.security-helpzone.com/
2/ create an account
3/ create a message
4/ Try using the edit message toolbar to style (bold, italic...) your text

expected result:
[b]my bold text[/b]

actual result:
[undefined=undefined]bold[/undefined]

This was reported in the French Mozilla forums here:
http://www.geckozone.org/forum/viewtopic.php?f=5&t=108673&p=708264#p708264

See the screenshot:
http://data.imagup.com/10/1164698595.png

I can confirm that with Firefox 16 and nightly builds on Linux, it works fine in Chrome.

The reporter indicates that reinstalling Firefox 15, the feature is back to normal, so that looks like a regression (should be added to RRTT page)

Comment 1

[Alice0775 White]

I can reproduce

In http://www.security-helpzone.com/jscripts/editor.js?ver=1600
I guess this problem caused by conflict with  itemType  property of HTML5 Microdata.

Steps to reproduce(anonymouse user):
1. Open http://www.security-helpzone.com/
2. Click "Désinfection Malwares" of forum
3. Click "Nouveau Sujet" blue button
4. Select text
5. Click "B" icon


Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/a7a905fd70d5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120604204045
Bad:
http://hg.mozilla.org/mozilla-central/rev/c76497029f0d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120605055317
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=a7a905fd70d5&tochange=c76497029f0d


Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/eee5cd8316af
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120604164643
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/f0f924ac58d2
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120604180843
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=eee5cd8316af&tochange=f0f924ac58d2

Suspected : 28dd33748be5	David Zbarsky — Bug 591467 - Implement HTML5 Microdata API r=bz

Comment 2

[Alex Keybl [:akeybl]]

This affects mybb latest, but doesn't affect phpbb latest. Sending over to David to investigate for a fix for FF17 (next couple of weeks), and cc'ing bz as backup.

Comment 3

[Boris Zbarsky [:bzbarsky]]

So the only fixes I can think of are to back out microdata or to get the site fixed.  Basically, this site is not compatible with the microdata spec as written.

Note that we had all this discussion in bug 763229 already, afaict.

If we're going to be evangelizing people, though, we need to do that.  David is certainly not the right person to do it....

Comment 4

[Alex Keybl [:akeybl]]

Thanks for the context in bug 763229 - hadn't seen that yet.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Note that this is not quite a duplicate.  Once mybb is fixed, this site will have to then update to the fixed version....

Someone should let them know we need to do that, of course.  :(

Comment 6

[Boris Zbarsky [:bzbarsky]]

Oh, and the someone _could_ be me, if needed, but my French is very very poor...

Comment 7

[Alice0775 White]

This site seemed to fix the problem. They used editor.js version 1608 now.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Ah, excellent.