プロジェクト

全般

プロフィール

機能 #1250

残り文字数カウントの処理を remain_charcount Spell に分離する

Shibafu Midorino7ヶ月前に追加. 6ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2018-05-20
期日:
進捗率:

0%

プラグイン名:
topic/1250-remain-charcount

説明

mikutter Nagoya 2018.5 の成果物となります、表題の件のパッチです。
(資料: https://speakerdeck.com/shibafu528/mikutter-nagoya-2018-dot-5)

従来 Gtk::PostBox.remain_charcount で行われていた、Twitterを前提とした残り文字数の計算処理を remain_charcount というSpell経由で実行するように変更し、計算処理自体はTwitter Pluginに移動しました。

Twitter依存部をプラグインに逃がすこと、残り文字数のヒント情報をTwitter以外のWorldからも提供できるようにすることを目的としています。

ただし、この実装ではTwitter DirectMessageのように同一のWorldで別の文字数計算基準を持つものを上手く処理できません。
この点が課題となっています。

ご検討よろしくお願いします。

0001-remain_charcount_spell.patch (6.11 KB) 0001-remain_charcount_spell.patch Shibafu Midorino, 2018-05-20 23:40
0002-fix_stack_overflow.patch (2.07 KB) 0002-fix_stack_overflow.patch Shibafu Midorino, 2018-05-21 23:27
0003-change_remain_charcount_options.patch (2.97 KB) 0003-change_remain_charcount_options.patch Shibafu Midorino, 2018-05-21 23:27

関連するチケット

関連している 提案 #1153: Twitter postbox 残り文字数表示 280字対応終了

関係しているリビジョン

リビジョン 08839822 (差分)
Shibafu Midorino7ヶ月前に追加

remain_charcount Spell refs #1250

リビジョン 551e4f95 (差分)
Shibafu Midorino7ヶ月前に追加

URLを含む文字列を入力した際にスタックオーバーフローを起こしてしまう refs #1250

リビジョン 9dda8736 (差分)
Shibafu Midorino7ヶ月前に追加

remain_charcountのオプションを、compose spellと統一 refs #1250

履歴

#1 toshi_a 初音7ヶ月前に更新

  • ステータス新規 から 実装待ち に変更
  • 担当者Shibafu Midorino にセット
  • プラグイン名topic/1250-remain-charcount にセット

土曜日隣で直接レビューしていたのでcommitしました。

DMなど、本文以外の投稿オプションで字数制限が変化するケース

土曜日、口頭で

PostBoxがDiva::Modelなら、remain_charcount[world, postbox] のような構文で残り文字数を取得できると良さそう

と言いましたが、よく考えると投稿する手段はPostBoxだけじゃないし、World系プラグインがguiに依存することになるのでよくありませんね。

compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。
残り文字数を知りたい場所では、compose spellを呼ぶはずですから、そのようなルールならわかりやすいと思います。

#2 Shibafu Midorino7ヶ月前に更新

本題の前に、patch 0001にてURLを含む文字列を入力した際にスタックオーバーフローを起こしてしまう問題がありましたので、patch 0002にて対応しました。
メソッドの移動を行った際に、名前がかぶっていることに気づいていませんでした...

compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。

とても良いと思います。そのような形で対応しましたので、patch 0003をご確認ください。

対応にあたってPostBox内のcompose呼び出しを探していたところ、remain_charcountの呼び出し側でフッタを差し込むかのフラグの考慮が漏れていることが分かりました。
そのため、双方のオプションを統一できるような形にしています。

#3 toshi_a 初音7ヶ月前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Shibafu Midorino に変更

topic/1250-remain-charcount ブランチにcommitしました。こっちで試してるぶんには問題ないかなと思います。そのまま当てただけですが、もう一度確認お願いします。

#4 Shibafu Midorino6ヶ月前に更新

  • 担当者Shibafu Midorino から toshi_a 初音 に変更

確認しました。問題ありません。

機能トラッカーには レビュー待ち → マージ待ち の状態遷移がないようですので、ステータスはそのままにしておきます。

#5 toshi_a 初音6ヶ月前に更新

  • ステータスレビュー待ち から 終了 に変更

mergeしました

#6 Izumi Tsutsui6ヶ月前に更新

  • 関連している 提案 #1153: Twitter postbox 残り文字数表示 280字対応 を追加

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