Ticket #6257 (closed defect: fixed)

Opened 4 months ago

Last modified 4 months ago

[patch] test_Editor.html throws an error in IE7 (FontChoice plugin)

Reported by: BryanForbes Owned by: alex
Priority: high Milestone: 1.1
Component: Editor Version: 1.1b1
Severity: major Keywords: patch
Cc:

Description

While trying to debug some Editor anomalies, I ran across a bug in the FontChoice? plugin where the value in the updateState() method has the indexOf method run on it before checking if it's a string. IE7 returns an integer from _e.queryCommandValue(), so the indexOf call throws an error. I'm attaching a patch to fix this for IE7.

Attachments

editor_fix.diff (0.6 kB) - added by BryanForbes 4 months ago.

Change History

Changed 4 months ago by BryanForbes

Changed 4 months ago by alex

  • status changed from new to closed
  • resolution set to fixed

(In [13146]) merging patch from Bryan Forbes to fix FontChoice? exception. Fixes #6257

Changed 4 months ago by peller

(In [13154]) Don't process font size pixel map unless the value is in pixels. Fixes #6257

Note: See TracTickets for help on using tickets.