プロジェクト

全般

プロフィール

Bug(バグ) #2000

完了

プレビューモードの絵文字パレットの右上にスペースがある

Shinichi Urabe さんが13年以上前に追加. 約9年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-04-09
期日:
進捗率:

100%

予定工数:
3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

現象

プレビューモード時の絵文字パレットの右上(「テニスの絵文字」の右)にスペースがある(添付画像参照)。テキストモード時には絵文字パレットはインラインで表示されるためこの問題はない。

原因

  • web/js/tiny_mce/plugins/openpne/editor_plugin.js.src
    web/js/tiny_mce/plugins/openpne/editor_plugin.js.src
    55-            for (var num in s.emoji) {
    56-                var emoji = s.emoji[num];
    57-                for (var i = emoji.start; i <= emoji.end; i++) {
    58:                    if (i == emoji.start || i % 25 == 0) {
    59-                        tr = DOM.add(tb, 'tr');
    60-                    }
    

この 58 行目では (i == emoji.start || i % 25 == 0) としているが、(i <= emoji.end と等号を含んでいることからも分かるように) i は [0,len) ではなく [1,len] の値を取ることが想定されている。

58 行目が (i == emoji.start || i % 25 == 0) である場合、

  • i=1 のとき、 emoji.start なので新たな行を開始( [1, 24] がこの行に含まれる)
  • i=25 のとき、 i%25 == 0 なので新たな行を開始( [25, 49] がこの行に含まれる)
  • i=50 のとき、 i%25 == 0 なので新たな行を開始( [50, 74] がこの行に含まれる)
  • i=(25*n) のとき、...

[1, 24] では 1 から 24 までの 24 個しか含まれない。このせいで 1 行目には 25 個目の絵文字が表示されていない。

修正内容

web/js/tiny_mce/plugins/openpne/editor_plugin.js.src 58行目の if 文

if (i == emoji.start || i % 25 == 0) {

を次のように書き換える。

if ((i - 1) % 25 == 0) {

i が emoji.start と一致しているかを評価する必要はない。

  • 追記
    • for 文では i = emoji.start とループ変数を初期化しており、 i が 1 から始まることは必要条件としていない。
      for (var i = emoji.start; i <= emoji.end; i++) {
          if ((i - 1) % 25 == 0) {
              tr = DOM.add(tb, 'tr');
          }
      
    • このため、 if 文では (i - 1) ではなく (i - emoji.start) と書き換えることにする。

また、これはJSファイルであり、 js.src を修正すると同時に js ファイルを更新する必要がある。

補足

js.src から js ファイル(minimize js)を作成するのに JSMIN を用いているらしい。 jsmin.c をコンパイルし、実行時にリダイレクトでファイルを渡せば minimize js が出力できる。

$ ./jsmin.out < editor_plugin.js.src > editor_plugin.js

ファイル

emoji_palette.png (35.2 KB) emoji_palette.png Minoru Takai, 2011-07-06 22:25

関連するチケット 3 (1件未完了2件完了)

関連している OpenPNE 3 - Backport(バックポート) #2263: プレビューモードの絵文字パレットの右上にスペースがあるFixed(完了)Minoru Takai2011-04-09

操作
関連している OpenPNE 3 - Backport(バックポート) #2859: プレビューモードの絵文字パレットの右上にスペースがあるFixed(完了)Minoru Takai2011-04-09

操作
関連している OpenPNE 3 - Task(タスク) #4071: OpenPNEに同梱されている TinyMCE 3.4.4 が IE11 に対応していないため、バージョンアップについて調査するNew(新規)Youichi Kimura2016-12-20

操作

Minoru Takai さんが13年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Minoru Takai にセット
  • 3.6 で発生するかYes にセット

Minoru Takai さんが13年以上前に更新

  • ファイル を削除 (Screen_shot_2011-04-09_at_13.53.28のコピー.png)

Minoru Takai さんが13年以上前に更新

Minoru Takai さんが13年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 1deb834eb65ba32c6645ef26d7fe0325baf72ccc で適用されました。

Naoya Tozuka さんが13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

修正箇所確認しました。

ここで (i - 1) % 25 の 1 は emoji.start の値であり、emoji.start が別の値だった場合にはその値に差し替えることになりますので、マジックナンバー的にソースに埋め込むのは好ましくありません。(25という値もマジックナンバーだと思いますがここでは触れません)

    for (var i = emoji.start; i <= emoji.end; i++) {
      if ((i - emoji.start) % 25) {
        tr = DOM.add(tb, 'tr');
      }
      ...
のようにするのが良いと思われます。

Minoru Takai さんが13年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

note-5 を受けて修正します。確かに emoji.start が 1 であることを前提とした修正になっていました。 emoji の id に関わらず行当たり25個の絵文字が表示されるように if ((i - emoji.start) % 25) という条件式に書き換えることにします。

Minoru Takai さんが13年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

更新履歴 ccaf4813ac629002d13c20f541105e486d7b08d8 で適用されました。

Naoya Tozuka さんが13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

修正確認しました。OKです。

Shingo Yamada さんが13年以上前に更新

  • 360対象RC1 にセット

Yuma Sakata さんが13年以上前に更新

  • 360対象 を削除 (RC1)

Yuma Sakata さんが13年以上前に更新

  • 360対象RC1 にセット

Yuma Sakata さんが約13年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

kaoru n さんが約9年前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

kaoru n さんが約8年前に更新

  • 関連している Task(タスク) #4071: OpenPNEに同梱されている TinyMCE 3.4.4 が IE11 に対応していないため、バージョンアップについて調査する を追加

他の形式にエクスポート: Atom PDF