Project

General

Profile

バグ #839

sub_parts_clientを入れていてSubpartsReplyViewerの設定で枠を表示しないようにしていると、ツイートの高さが必要以上に確保される

Added by Izumi Tsutsui about 3 years ago. Updated about 3 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

題名のとおりですが、
mikutter.3.4.0-alpha2 で sub_parts_client.rb
https://github.com/toshia/mikutter-sub-parts-client
を使った場合、返信ツイートのリプライ元表示を描画するところで
「リプライ描画中にエラーが発生しました」が発生するようです。

起動時の表示画面を見ていると
  1. ツイート本文を表示する
  2. client 名を表示する
  3. リプライ元メッセージを表示する

という順番で、client 名表示までは正常で、リプライ表示描画をしようとしたところで
クライアント名の1行下に空白行が出てくるような感じです

例によって NetBSD + pkgsrc + ローカルパッチ というオレオレ環境ですが、
alpha に対してとりあえず報告ということで。


Files

mikutter340-reply-error.png (91.4 KB) mikutter340-reply-error.png Izumi Tsutsui, 2016-05-25 01:14
error-reply-drawing.png (45.8 KB) error-reply-drawing.png Izumi Tsutsui, 2016-05-25 20:19
mikutter-liveimage.png (138 KB) mikutter-liveimage.png 枠なしリプライ表示 Izumi Tsutsui, 2016-05-28 12:54
mikutter-liveimage-with-waku.png (142 KB) mikutter-liveimage-with-waku.png 枠ありリプライ表示 Izumi Tsutsui, 2016-05-28 12:56

Related issues

Related to 機能 #828: 引用や返信元ツイートが鍵垢で表示できないときにそれとわかる表示をする機能新規2016-05-08

Actions

Associated revisions

Revision 54352def (diff)
Added by toshi_a 初音 about 3 years ago

枠のサイズを高さ計測前に準備。高さが変わったときには再計算を適切にリクエストする refs #839

History

#1

Updated by toshi_a 初音 about 3 years ago

  • Assignee set to Izumi Tsutsui

設定の「枠線を表示する」のチェックを外した状態でのみ、添付画像のような隙間が発生しました。

「リプライ描画中にエラーが発生しました」が発生するようです。

これについてもう少し詳しく教えてもらっていいですか

#2

Updated by Izumi Tsutsui about 3 years ago

toshi_a 初音 さんは書きました:

「リプライ描画中にエラーが発生しました」が発生するようです。

これについてもう少し詳しく教えてもらっていいですか

Activity に ↓のようなエラーが出ます。

あと、現状以下のプラグインが入っています。
display_requirements.rb
event_watch/
gray_protected_users.rb
load_more/
mikutter-uwm-hommage/
most_faved/
open_favstar/
pulseaudio/
show_tweet/
sub_parts_client/

プラグイン無しではどうなるか、設定ファイルなしの場合ではどうなるか、
コンソールメッセージでどこに対応するのか、等々は Live Image で確認してみます。

#3

Updated by Izumi Tsutsui about 3 years ago

設定ファイルなし、
プラグインは sub-parts-client + display_requirements.rb のみだと
「リプライ描画中にエラーが発生しました」は出ないようです。
もう少し条件確認してみます。

#4

Updated by Izumi Tsutsui about 3 years ago

直接関係ありませんが、こういうツイートもありました

https://twitter.com/dai_lxr/status/735268566593740800

プラグインを1つ1つ外してみたところ、 mikutter-datasource-search を外したら「リプライ描画中にエラーが発生しました ()」が出なくなり動くようになった。でもこれないと不便。 https://t.co/ziVRnqINXd

#5

Updated by Izumi Tsutsui about 3 years ago

いまいち確証がありませんが、なんとなくわかりました。

フォローしていない鍵アカウントに対するリプライのツイートの表示で
リプライ先を表示しようとして
「リプライ描画中にエラーが発生しました」
が出ているようです。リプライ表示の枠なし時に1行開く現象とは関係なさそうです。

1行空くのが意図通りであればチケットとしては却下でもOKです。

#6

Updated by Izumi Tsutsui about 3 years ago

またまた微妙に関係ありませんが
リプライのツイートがふぁぼられて ageられると、
最後の1行の空行は無くなる(こともある)ようです。

TLと自分のツイートタブでは空行なくなったんですが、
most_faved のタブでは同じツイートなのに空行残っている、
というよくわからない状態です。

#7

Updated by toshi_a 初音 about 3 years ago

  • Subject changed from sub_parts_client.rb 使用時「リプライ描画中にエラーが発生しました」が出る to sub_parts_clientを入れていてSubpartsReplyViewerの設定で枠を表示しないようにしていると、ツイートの高さが必要以上に確保される
  • Status changed from 新規 to 実装待ち
  • Assignee changed from Izumi Tsutsui to toshi_a 初音

Activityだったんですね。

protectedなアカウントへのreplyを表示する時にActivityにerrorとして「リプライ描画中にエラーが発生しました()」と表示されます。ツイートを取得することができないからです。
これを正しく表示する手段は無いので、

  • #828 を実装する
  • エラーメッセージを変更する

が現実的な解決策です。が、場合によっては表示されていないタブ上のツイートのリプライを勝手に取得しに行った結果失敗しているうえ、どのタブでそれが引き起こされたかを報告することはできないので、わかりやすいエラーメッセージをここに表示することはできないかもしれません。その意味でもこの問題は #828 に引き継いだほうが良いでしょう。

最初の説明を読むと、ここで本来指摘されているのはツイート表示領域の高さだと思います。これは全く無関係な問題で、

  • サードパーティプラグイン sub_parts_client を入れている
  • SubpartsReplyViewerの設定で枠を表示しない設定にしている

の両方を満たした環境で再現しています。恐らく高さ再計算がかかると、正しい高さになるのでしょう。

#8

Updated by toshi_a 初音 about 3 years ago

  • Related to 機能 #828: 引用や返信元ツイートが鍵垢で表示できないときにそれとわかる表示をする機能 added
#9

Updated by toshi_a 初音 about 3 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui

release/3.4ブランチです。これで大丈夫だと思います。

#10

Updated by Izumi Tsutsui about 3 years ago

とりあえず適当に 54352def の差分を 3.4.0-alpha2 に当ててみたのですが
まだすき間は表示されているような気がします。
が、いまいち自信がないので明日 Live Image でまた試してみます。

描画としては
(1)ツイート本文表示
(2)クライアント名表示
(3)リプライ描画
で、 (2) まではクライアント名の下にすき間がなく、(3)の描画の瞬間にすき間ができる感じです。

枠のないときも枠の高さが含まれている、とかいうことはないですよね……。

#11

Updated by Izumi Tsutsui about 3 years ago

54352def を当てる前ですが、 Live Image でちょっと試してみました。
枠なしの時の時も枠ありの時と同じ高さになっているような気がします。
どうでしょう?

枠なしリプライ表示 枠ありリプライ表示

#12

Updated by Izumi Tsutsui about 3 years ago

  • Status changed from レビュー待ち to 解決

いまさらながら 3.4.0-alpha3 を試してみたら、
枠なし設定でも余白(?)なく表示されました。なので解決でOKです。

#839-10 のはなにかミスってたみたいです。失礼しました。

#13

Updated by toshi_a 初音 about 3 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF