操作
機能 #1205
完了
返信時にGtk::PostBoxがIntelligentTextviewを付加するための要件を緩和してほしい
プラグイン名:
topic/1205-intelligent-text-view-for-everything
説明
現在は
m.is_a?(Message)
で対象を絞っていますが、以下のようになっているとカスタムModelで対応しやすいです。
m.is_a?(Diva::Model::MessageMixin) && m.respond_to?(:user) && m.user.is_a?(Diva::Model::UserMixin)
試作したパッチを添付します。
worldonプラグインではこの対応のためにGtk::PostBox#generate_boxをモンキーパッチしているんですが、将来的な本体側の変更を考えると、できればモンキーパッチではなく、本体側の要件緩和で対応されてほしいです。
ファイル
関連するチケット
toshi_a 初音 さんが6年以上前に更新
- ステータス を 新規 から 実装待ち に変更
- 対象バージョン を 3.6 から 3.7 に変更
これはダックタイピングを阻害していて良くないですね。
ただ、このパッチでも不十分だと思っていて、最低限icon(model.user.iconではなくmodel.icon)とdescriptionが取得できたら表示できるはずなので、そのようにしたいと思います。
（本当はアイコンはoptionalで良いんだけど、それは今回はどっちでも良いかな）
toshi_a 初音 さんが6年以上前に更新
- ステータス を 実装待ち から レビュー待ち に変更
- 担当者 を cob odo にセット
- プラグイン名 を topic/1205-intelligent-text-view-for-everything にセット
修正しました。これでほかのModelも表示されることが確認できました。
ただ、Worldonで検証した場合は、Message Modelのiconメソッドが定義されていないためアイコンを表示しません。全てのMessage ModelにUserに相当するものが存在するとも限らないのでMessage#iconを使っています。
これで問題なければ、developにmergeしておいてください。
操作