Project

General

Profile

Backport(バックポート) #2370

IE9 で文字装飾のプレビューモードが正常に動作しない( tinymce のバージョンを上げる)

Added by Yuma Sakata over 11 years ago. Updated over 11 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2011-08-22
Due date:
% Done:

100%


Description

概要

TinyMCE (tiny_mce) の文字装飾機能について、IE9 ではプレビューモードが正常に動作しない。なお、この問題は OpenPNE 2 でも発生する。

  • 太字・斜体のような文字装飾ボタンを押しても装飾が機能しない
  • 文字色パレット・絵文字パレットは表示されるが、色や絵文字を選んで押下しても装飾が反映されない
  • テキストモードに戻せない
  • 文字の表示サイズが他のブラウザ(IE8 など)に比べて小さい

これらは後述する原因によるものであり、よりマクロな問題は「IE9 で文字装飾機能(TinyMCE)が動作しない」というものである。

IE9のスクリプトデバッグモードを使用した際に表示されるエラー

OpenPNE 3 の場合、テキストモード→プレビューモードの切替時に以下のエラーが表示される。

SCRIPT438: オブジェクトは 'recalc' プロパティまたはメソッドをサポートしていません。 
tiny_mce.js, 行 1 文字19997

このエラーが表示された直後にブラウザがフリーズするため、文字装飾ボタンをクリックした場合などのエラーは確認していない。

(蛇足)OpenPNE 2 の場合は上と同じタイミングで以下のエラーが表示される。

SCRIPT438: オブジェクトは 'parentElement' プロパティまたはメソッドをサポートしていません。 
tiny_mce.js?r7140, 行 1 文字37913

対象バージョン

OpenPNE 3 系の全てのバージョン。

この問題は TinyMCE 側にあり、 IE9 独自の問題に対応したバージョンの TinyMCE を用いていないことが OpenPNE でこの問題が生じる原因である。ただし OpenPNE 側にも問題があるので併せて修正する。

修正方針

2009年10月14日に OpenPNE-3.1.4 で TinyMCE-3.2.7 が組み込まれている( e24edf33 )。 TinyMCE-3.2.7 では IE9 に対応しておらず、上記のような問題が生じる。

2011年8月現在、 TinyMCE は 3.4.4 が最新版として公開されており、このリリースまでに IE9 に対する修正がいくつか取り込まれている。

  • TinyMCE を 3.4.4 にバージョンアップすることで対応する。
  • ただし、 TinyMCE-3.2.7 の実装を前提に記述した OpenPNE 側のコードがあるため、バージョンアップ後に OpenPNE 側で拡張した部分を修正する必要がある。

Related issues

Related to OpenPNE 3 - Bug(バグ) #2362: IE9 で文字装飾のプレビューモードが正常に動作しない( tinymce のバージョンを上げる) Fixed(完了) 2011-08-22

Associated revisions

Revision 4b4de01d (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) updated tinymce to 3.4.4

Revision aaa362f5 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixes tinymce-plugins/openpne for tinymce-3.4.4

Revision f6539963 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) changed emoji-palette position.

Revision 53965b1e (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixes buttons spacing of tinymce-toolbar for IE6/7

Revision cc9456f7 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixed the color-palette of text-mode not to depend on position-property-value of element of ancestors.

Revision c9d6c6e8 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixes tinymce-plugins/openpne for tinymce-3.4.4; fixes hideMenu of the emoji-palette.

Revision a4952ff3 (diff)
Added by Minoru Takai over 11 years ago

Revert f6539963 "(refs #2370, BP from #2362) changed emoji-palette position."

Revision 4d7c8509 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) changed to display the emoji-palette on the upper part of the button.

Revision 583789d0 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) added inline-block to the button, that is "a" (anchor) element, to exactly calculate the height.

Revision c4ce8253 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixed _previewToText() method to convert the "strong", "strike" and "em" element in Webkit (Safari, Chrome)

Revision a06bf3f6 (diff)
Added by Minoru Takai over 11 years ago

(refs #2370, BP from #2362) fixed _previewToText() method not to output many line-break.

History

#1 Updated by Minoru Takai over 11 years ago

  • Subject changed from IE9 で文字装飾のプレビューモードが正常に動作しない to IE9 で文字装飾のプレビューモードが正常に動作しない( tinymce のバージョンを上げる)
  • Description updated (diff)

#2 Updated by Minoru Takai over 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Minoru Takai

BP対応します。

#3 Updated by Minoru Takai over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

stable-3.6.x ブランチに親チケットでの修正を全て取り込みました。

#4 Updated by Minoru Takai over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Accepted(着手)

親チケットで差し戻しを行います。

#5 Updated by Minoru Takai over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

note-4 に対する修正を取り込みました。親チケット http://redmine.openpne.jp/issues/2362#note-19 で示している 9 個の修正を取り込んでいます。

#6 Updated by Yuma Sakata over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

先にテスト実施しましたが、気になる点がありましたので確認お願いします。

プレビューモードでテキストモードに戻せるか Safari で確認

  • 試験手順
    1.SNSログイン後、日記作成ページ(/diary/new)にアクセスする
    2.プレビューモード選択後、文字入力(例:OpenPNE)および文字装飾(例:太字)する
    3.テキストモードに戻す
  • 試験結果
    「<strong>OpenPNE</strong>」と表示され、プレビューモードに戻しても太字が反映されず、HTML形式になる
  • 修正方針
    テキストモードに戻したときに「<op:b>OpenPNE</op:b>」と表示され、プレビューモードに戻しても太字が反映されるように修正お願いします。
  • 備考
    • 上記の問題は Safari のみで発生し、他のブラウザでは問題ありませんでした。

#7 Updated by Minoru Takai over 11 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

note-6 の差し戻しについて、 MacOS Safari と Chrome でも不具合を確認しました。対応します。

#8 Updated by Minoru Takai over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

note-6 に対する修正を c4ce8253 で取り込みました。

#9 Updated by Minoru Takai over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

これまでこのチケットでレビューを行なっていませんでしたが、まとめてレビューを行いました。

master ブランチと同一の修正が stable-3.6.x ブランチに全て(note-8 までの10個)取り込まれていることを確認しました。レビューを通します。

#10 Updated by Yuma Sakata over 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#11 Updated by Minoru Takai over 11 years ago

  • Status changed from Fixed(完了) to Accepted(着手)

テスターテストが通っていますが、プレビューモードからテキストモードに変えたときに改行が倍になるという問題が見つかっているのでこれを対応します。

#12 Updated by Minoru Takai over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 100 to 50

note-11 の問題を対応した 11 個目の修正 a06bf3f6 を取り込みました。

#13 Updated by Minoru Takai over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

note-12 が stable-3.6.x ブランチに取り込まれていることを確認しました。

#14 Updated by Minoru Takai over 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

note-10 でテスターテストが通っており、11個目の修正については実装者テストを完了しているため、これを以てテストOKとし、ステータスを完了にします。

Also available in: Atom PDF