プロジェクト

全般

プロフィール

提案 #1153

Twitter postbox 残り文字数表示 280字対応

Izumi Tsutsui8ヶ月前に追加. 2ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
ブランチ:
topic/1153-twitter-280

説明

雑に twitter-text 2.x 的パッチを書いてみたらなんとなく動いているっぽいので投棄しときます
(rubyなんもわからん)
gtk_postbox.rb.diff

リプライの @ユーザー名 についても文字数カウントに入ってしまうようなので、これだけでは足りてないかも。

#1130#note-4 にあるように
mui/gtk_postbox.rb が twitter-text 依存しているのがおかしいので twitter plugin に移動させるべき、
というほうが先かもしれません。

gtk_postbox.rb.diff (838 Bytes) gtk_postbox.rb.diff Izumi Tsutsui, 2018-01-07 22:34
tweet-too-long-response.png (20.5 KB) tweet-too-long-response.png Izumi Tsutsui, 2018-05-22 00:11

関連するチケット

関連している 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する終了2018-05-20

関係しているリビジョン

リビジョン bfbb09fc (差分)
toshi_a 初音2ヶ月前に追加

twitter: 280文字対応 refs #1153

履歴

#1 Izumi Tsutsui4ヶ月前に更新

mikutter Nagoya 2018.5 後のメモ

mastodon 上でいろいろ議論のような toot が飛び交っていたので、
参照用に適当にチケット上に拾っておきます。


文字数カウントをどのプラグインが行うべきか

mui/gtk_postbox.rb が twitter-text 依存しているのがおかしいので twitter plugin に移動させるべき、

https://social.mikutter.hachune.net/@tsutsuii/100061694973944377

Izumi Tsutsui
@tsutsuii

残りカウントを「どこでやるか」という話と「どうやるか」という話は独立なので、しばふ氏のはとりあえずチケット入れるという話ですかねえ。280文字対応のチケットは一応ある(パッチが適当すぎる)ので議論はそこでもいいし
2018年5月20日 21:44 · mikutter Worldon

というわけで しばふ氏による #1250 で解決されると思われます。


twitter-text の URL込み文字数カウント

https://social.mikutter.hachune.net/@tsutsuii/100061709578785640

Izumi Tsutsui
@tsutsuii

twitter-text は一応残り文字数の URLも見ているっぽくて、 teokurebsd.or まで入れた時と teokurebsd.org まで入れたときでは最後の g を入れた瞬間に一気に -10されたりする。
一方で libhoge.so の文字列が twitter-text の文字数カウントでは URL扱いされないにもかかわらず投稿する(もしくは公式 web の入力フォームの表示)だとURL扱いになる。
これは twitter-text もしくはその他のデータベースが追いついていないということなのか
2018年5月20日 21:48 · mikutter Worldon

このチケットのパッチはこれを見て元のコードのURL部分の実装を削除しています。


残り文字数 vs 現在文字数

https://mstdn.kanagu.info/@cobodo/100060113405100881

金具✅
cob odo

https://github.com/cobodo/mikutter_justsize こういうの勝手に書いたりしてました #toshiakaigo2018

cobodo/mikutter_justsize
mikutter_justsize - postboxに残り文字数ではなく現在の文字数を表示するmikutterプラグイン
github.com
2018年5月20日 15:02 · ◆ Tootdon ◆

https://mstdn.kanagu.info/@cobodo/100061580169633732

金具✅
cob odo

残り文字数の件、もう少し考えたけど、各Worldに残り文字数spellを要求するんじゃなくて、現在文字数spellと最大文字数spellを要求すべきのような気がしてきた。最大文字数が提供されない場合は現在文字数を普通に出す。現在文字数も提供されない場合、普通に.sizeで出す。
2018年5月20日 21:15 · mikutter Worldon

https://social.mikutter.hachune.net/@ahiru/100061600936730806

炎のバーナー
@ahiru

この場合、数値の表記について慎重にキメないとまずそう
そこに表示されている数字は現在の文字数なのか残り文字数なのか
2018年5月20日 21:21

https://social.mikutter.hachune.net/@ahiru/100061604065166432

炎のバーナー
@ahiru

ぶっちゃけ残り文字数は投稿後のアラートとかでいいじゃん派ではある
2018年5月20日 21:21

https://social.mikutter.hachune.net/@shibafu528/100061608131333828

芝生
Shibafu Midorino

現在文字数を出すメリットそんなにあるかなぁって思ってます。
2018年5月20日 21:22 · SubwayTooter

https://mstdn.kanagu.info/@cobodo/100061613793538087

金具✅
cob odo

文字数で蹴られたのか、単にAPIが死んでて失敗したのかが多少わかる、くらいのメリットはあるかなぁと(逆に言うとそれくらいしかないと思っている)
2018年5月20日 21:24 · mikutter Worldon

https://social.mikutter.hachune.net/@tsutsuii/100061613330424930

Izumi Tsutsui
@tsutsuii

残り文字数が必要と思うのは 一メッセージに詰め込みたい時にどこまで削るか みたいな状況以外にあるかというと、どうだろう
2018年5月20日 21:24 · mikutter Worldon

ここから先は感想戦なので省略引用

https://mstdn.kanagu.info/@cobodo/100061617045310664

思ってるけどUIの要素を安易に変えると変更点を追ってないユーザは混乱すると思うしあまり望ましくないかなぁと

https://mstdn.kanagu.info/@cobodo/100061618380674789

だから現在文字数を出すようにする機能はサードパーティプラグインとして書いた

https://social.mikutter.hachune.net/@shibafu528/100061620615842846

これは身も蓋もないが、mikutterユーザそんなに残カウンタ見てないから情報がないときはウィジェットごと不可視にしてもバレないくらい影は薄い

https://social.mikutter.hachune.net/@ahiru/100061621428699809

文字数の表記、限られたUIのスペースを食うだけの役を果たしているのか
というのが、自分の考えの基準になっている

https://mstdn.maud.io/@zgock999/100061620676384465

文字数が128文字ジャストとか64文字ジャストとか256ジャストとか気持ちいいじゃない(ぇ

https://mstdn.kanagu.info/@cobodo/100061624830031766

mikutter全画面表示派なので言うほど食ってないしあってもなくてもまぁ……という感じ

https://social.mikutter.hachune.net/@ahiru/100061630163377824

これ結構個人の好みに依存してそうだし、こうやってサードパーティー製で出してるのが正解っぽい


文字数オーバー時のレスポンス

https://social.mikutter.hachune.net/@tsutsuii/100061619662581879

Izumi Tsutsui
@tsutsuii

今日の会場でもちらっと会話したけれど、文字数が多い時は「ちょっと長すぎるんじゃ」みたいなレスポンスが返ってくるので、それを表示すれば事足りる(何文字はみ出しているのかの数字は通知されないのであまり意味がない説)
2018年5月20日 21:25 · mikutter Worldon

https://social.mikutter.hachune.net/@tsutsuii/100061640040816031

Izumi Tsutsui
@tsutsuii

Tweetが長すぎる時は status code 186 "Tweet needs to be a bit shorter." が返ってくる(というログ)

https://mstdn.kanagu.info/@cobodo/100061643607777969

金具✅
cob odo

この辺は、投稿時にエラーが発生したときにエラーメッセージを表示する機能をpostboxプラグインが持ったほうがいいのではないか的な話になるのかな……
2018年5月20日 21:31

https://mstdn.kanagu.info/@cobodo/100061649598981505

金具✅
cob odo

postboxは単独のプラグインってわけじゃないか
2018年5月20日 21:33 · mikutter Worldon

https://social.mikutter.hachune.net/@ahiru/100061648985376612

炎のバーナー
@ahiru

それはありそう
大抵のサービスはほとんどの場合、一度送ってしまってそれに対するレスポンスとして何故そのリクエストが却下されたかを返すといった形になってるし
2018年5月20日 21:33

https://social.mikutter.hachune.net/@shibafu528/100061652280911210

芝生
Shibafu Midorino

Postboxはmuiなのでね、難しいですね(この土日でどうするかやってた感想として)
2018年5月20日 21:34 · SubwayTooter

https://social.mikutter.hachune.net/@shibafu528/100061674441810782

芝生
Shibafu Midorino

Postboxがエラー持つの多分仕組みが必要になるやつで、compose spellは発行元コンテキスト持ってない気がするし適切な例外を発行するみたいな規約があるとPostboxでトラップできそうだけど、それはmikutter wayか…?ってなるな
2018年5月20日 21:39 · SubwayTooter


twitter 以外の残り文字数

https://mstdn.kanagu.info/@cobodo/100060131685933621

金具✅
cob odo

mastodonもURLに関しては文字数制限に緩和が入るらしいので調べて対応しないといけないなー(対応するとは言っていない) #toshiakaigo2018
2018年5月20日 15:07 · ◆ Tootdon ◆

https://social.mikutter.hachune.net/@ahiru/100060165832939123

炎のバーナー
@ahiru

公式はまだ見つけてないけど、Slackはmessageあたり4000字っぽいな
2018年5月20日 15:16

https://social.mikutter.hachune.net/@ahiru/100060189214694451

炎のバーナー
@ahiru

SLackの文字数確認の為に*を4000字投稿したら理解されなかった
2018年5月20日 15:22

#2 Izumi Tsutsui3ヶ月前に更新

  • 関連している 機能 #1250: 残り文字数カウントの処理を remain_charcount Spell に分離する を追加

#3 Izumi Tsutsui3ヶ月前に更新

  • トラッカー機能 から 提案 に変更
  • ステータス新規 から 分類待ち に変更
  • 開始日 を削除 (2018-01-07)

履歴#1 の議論のあと、3.7.2 で入った
#1250 「残り文字数カウントの処理を remain_charcount Spell に分離する」
の変更のため添付の gtk_postbox.rb.diff はそのままでは当たらなくなっています。

従来から手元ではモンキーパッチで過ごしていてこのチケット優先度は下がっていたりするのですが、
3.7.2 後は「spell をプラグインでモンキーパッチすることは無理」というコメントを金具さんからいただいていて、
たしかに暫定処置はしづらくなっています。
https://gist.github.com/tsutsui/5c7ae61a3d28084632270f2b2d18273d

全体の方針として twitter-text の文字カウントだけを分離して mikutter に取り込もう、
みたいな話も挙がっていますが、自分で入れたチケットなのでとりあえず保留で
トラッカーのみ変えておきます。

#4 toshi_a 初音2ヶ月前に更新

  • ステータス分類待ち から 実装待ち に変更
  • 担当者toshi_a 初音 にセット

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

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Izumi Tsutsui に変更
  • 対象バージョン3.6 から 3.7 に変更
  • ブランチtopic/1153-twitter-280 にセット

対応しました。確認してみてください。

#6 Izumi Tsutsui2ヶ月前に更新

  • ステータスレビュー待ち から マージ待ち に変更

確認しました。問題ありません。
ありがとうございます。

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

  • ステータスマージ待ち から 終了 に変更

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