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

RESOLVED FIXED

Status

Tech Evangelism Graveyard
French
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: pascalc, Assigned: dzbarsky)

Tracking

({regression})

Details

(Whiteboard: [mozfr])

(Reporter)

Description

5 years ago
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)
(Reporter)

Updated

5 years ago
Whiteboard: [mozfr]

Updated

5 years ago
Version: Trunk → 16 Branch

Comment 1

5 years ago
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
Blocks: 591467
tracking-firefox16: --- → ?
tracking-firefox17: --- → ?
tracking-firefox18: --- → ?
tracking-firefox19: --- → ?
Component: General → DOM
Keywords: regressionwindow-wanted
Product: Firefox → Core

Comment 2

5 years ago
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.
Assignee: nobody → dzbarsky
tracking-firefox16: ? → -
tracking-firefox17: ? → +
tracking-firefox18: ? → +
tracking-firefox19: ? → +
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

5 years ago
Thanks for the context in bug 763229 - hadn't seen that yet.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
tracking-firefox17: + → -
tracking-firefox18: + → -
tracking-firefox19: + → -
Resolution: --- → DUPLICATE
Duplicate of bug: 763229
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.  :(
Status: RESOLVED → REOPENED
Component: DOM → French
Product: Core → Tech Evangelism
Resolution: DUPLICATE → ---
Version: 16 Branch → unspecified
Oh, and the someone _could_ be me, if needed, but my French is very very poor...

Comment 7

5 years ago
This site seemed to fix the problem. They used editor.js version 1608 now.
Ah, excellent.
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.