一部のブラウザでアルバム小窓がカーソルの位置に挿入されない

[imamura623]

■現象

PC版において、一部のブラウザでアルバム小窓を使った際にカーソル位置にテキストが挿入されず、テキストエリアの一番上に挿入されてしまう。

■環境

• OS : WindowsXP SP2, Windows2000 Professional
• ブラウザ : IE6, IE7

■原因

IEでは他のブラウザのようにカーソル位置を取得できないため。

■修正内容

実行ブラウザがIEの場合のみ、別の処理をさせるようにした。ブラウザの判定にはプロパティの有無を使った。

184 	    if (undefined !== window.execScript) {

execScriptプロパティはIEにしか実装されていないため、IEだけ処理を分けることができる。

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=16662 より転記

アルバムから貼り付けするときにどうしてもタグが上に貼ってしまう。。。
（とりあえず、カット＆ペーストで対応していますが、できることならカーソルの位置でペーストできるとベスト） 

[imamura623]

報告された方の環境は下記のとおりです

• OS : WindowsXP SP2
• ブラウザ: IE7

[nakasone]

以下のような確認を行ってみたところ再現できました。 また、FireFox2.0では正常に動作しました。

■確認ブラウザ

• IE6

■確認バージョン

• trunk

■手順

1. 日記作成画面
2. タイトル・本文を入力し、適当な位置にカーソルを合わせる
3. アルバム小窓からアルバムを挿入
4. アルバムが挿入されている位置を確認

[ebihara]

やります。

[ebihara]

IEだと別ウィンドウを開いたときにテキストエリアのカーソル位置情報が消えるのが原因っぽいですね。

調査・対応が難しいので優先度低めで取り組みたいです。

[ebihara]

他のバグ修正を優先します。手放します。

[ebihara]

次のリリースに回します

[ebihara]

フォーカスが外れる直前にカーソル位置をBookmarkしておくという策は採れるかもしれません。

[ebihara]

IE の場合のみ、 textarea に対して、

     document.getElementById("mce_editor_textarea").onblur = function() {
        var rng = this.createTextRange();
        var bk = rng.getBookmark();
        this.focus();
        rng.moveToBookmark(bk);
     };

このようなコードを書くことで、フォーカスが外れた瞬間に現在のカーソル位置を記憶し、戻すことができるようになります。

これを活用していく方向で実現したいです。

[ebihara]

チャレンジしてみます。

[ebihara]

window.open() によってダイアログを表示するのをやめるというのがシンプルな対処方法かもしれません。

[ebihara]

最初は開発版のみで対応します。

[kiwa]

#2899のテスト中に同様の現象が発生しました。

[kiwa]

ちょっと優先度あげたいです。

[fukamachi]

修正します。とりあえずは仕様変更ではなく、comment:13の海老原さんの方法で修正したいと思います。

[fukamachi]

r11274、r11275で修正しました。

[fukamachi]

イベントが正しく追加されなかったため、windowロード後に追加するよう修正

r11278

[fukamachi]

branch上でr11280で修正しました。

[fukamachi]

r11284で修正しました。
public_html/js/pne_decoration.js の関数に、IE用の動作部分を追加しました。

[fukamachi]

r11286 で追加修正
.js.srcで修正されていない箇所があったので修正

[fukamachi]

Windows 2000 Pro IE6

の環境でテストしたところ、正常に動作していなかったので自分で差し戻しておきます。

[fukamachi]

r11292で修正しました。

[fukamachi]

カーソル位置への挿入に、expand()は使わなくてもいいみたいです

修正すべき内容

-     selection.expand("character"); 
-     if (selection.text === "") { 
-         selection.text = str + selection.text + "\n"; 
-     } else { 
-         selection.text = str + selection.text; 
-     }
+     selection.text = str + selection.text;

時間があるときに修正します。

[fukamachi]

r11317で修正しました。

[kudo]

FireFox3.0では修正を確認できましたが、IE6で確認したところ修正されていません。

• 確認ブラウザ:IE6
• 確認バージョン：trunk

日記の作成で適当な文字列を本文に入力し任意の位置にカーソルを移動後

アルバムの挿入を行うと、本文の一番上に挿入されてしまいます。

[fukamachi]

r11292時点ではWin2000, IE6で動作確認したのですが、
r11317の変更後に動かなくなったのかもしれません。
申し訳ないです。時間があるとき(明日)に修正します

[fukamachi]

r11418時点で確認しましたが、再現しませんでした。

• 確認OS : Windows2000
• 確認ブラウザ : IE6
• 確認バージョン : trunk(r11418)

[fukamachi]

開発版へのマージ忘れが原因でした。

r11515でマージしましたので、再確認お願いします。

[nagasawa]

firebugでjsのエラーが出ています、firefox3では動作上問題ありませんでしたが
IETesterでIE6をテストしたところエラーが出てしまい正常に挿入されませんでした。

[nagasawa]

エラー内容について追記します。

• エラー内容
  • t.editor is null
    
• ファイル
  • js/tiny_mce/tiny_mce_popup.js?r7140　23行目

[fukamachi]

上の内容とは関係ないですが、JSLintで警告が出ていたのでr11518で修正しました。

• 値を必ずしも返さない関数がある
• セミコロン忘れ
• 変数の再定義
• 条件文での == の使用(===を使用して厳密化)

[fukamachi]

アルバム小窓を開いた際に42のエラーが出る問題は、今チケットの修正範囲外であり、安定版にも同様の不具合が見つかったため、別チケット(#3893)で対応します。

[kudo]

#3897:一部のブラウザでアルバム小窓がカーソルの位置に挿入されない不具合修正のコードチェック

上のチケットでコードチェックを行います。

IETesterでの動作テストで出たエラーは#3893で修正してください。firefox3、IE6では動作上問題ありませんでしたので、今回はこれで閉じます。