Project

General

Profile

Actions

バグ #1524

closed

gtk3: UserList の Scrollbar が効かず Window が拡大し続ける問題

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

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

Description

Twitter 関連プラグインを入れているときのユーザープロフィールタブのフォロー・フォロワータブの表示において、
機械的な GTK3 対応の変更を入れるだけでは以下の動作になってしまいます。

  • 表示するユーザーの数だけ mikutter のウインドウの縦幅が拡大していく
    つまり VScrollbar が(表示されているものの)機能していない
  • 表示するユーザーの bio の長さの最大値×ペイン数だけウインドウの横幅が拡大していく
    つまりはみ出した領域が隠れるのではなく拡大されていく

実際に拡大されるかどうかは WM によるかもしれません。(こちらでは jwm です)

GTK3 での仕様変更要因なのか GTK2 でたまたま動いてただけなのかよくわかりませんが
source:core/mui/gtk_userlist.rb@ab0bcc0e#L28
Box(:horizontal) + VScrollbar をやめて ScrolledWindow を使うと一応それっぽく動くようになるようです。

Twitterプロフィールタブのフォロー・フォロワー以外で UserList を使っているものがあるのかどうかわからないので
再現手順が書きづらいのですが、とりあえずパッチは追って投げます。


Files

Actions #1

Updated by Izumi Tsutsui over 3 years ago

とりあえず ScrolledWindow に置き換えるパッチ

Actions #2

Updated by Izumi Tsutsui over 3 years ago

#1515closeup の置き換えが入ったのでパッチ更新しておきます。

Actions #3

Updated by Izumi Tsutsui over 3 years ago

  • Status changed from 分類待ち to パッチ適用待ち

手元では動いていてステータス的に区別したいので先走って変更してしまいます

Actions #4

Updated by Shibafu Midorino over 3 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1524-userlist-scroll

#1500 と同じかも。このパターンで実装されてるものを今後見つけたら全部同様に対処しちゃって良さそうですね。

Actions #5

Updated by Izumi Tsutsui over 3 years ago

  • Status changed from レビュー待ち to マージ待ち

topic/887-gtk3topic/1514-fix-gtk-userlisttopic/1524-userlist-scroll とマージして
Twitter user_detail_view でのフォロー・フォロワー表示でもともともパッチ通り動いていて問題ありません。

Actions #6

Updated by Izumi Tsutsui over 3 years ago

メモ

GTK3 での仕様変更要因なのか GTK2 でたまたま動いてただけなのかよくわかりませんが
source:core/mui/gtk_userlist.rb@ab0bcc0e#L28
Box(:horizontal) + VScrollbar をやめて ScrolledWindow を使うと一応それっぽく動くようになるようです。

しばふ氏によればこれは #1500 と同様の問題ではないかと思われる、
つまり、他に同様の実装があれば ScrolledWindow に置き換える、ではよいのでは、とのこと。

Actions #7

Updated by Izumi Tsutsui over 3 years ago

slack と mastodon TL を混同して同じことをチケットに書いてしまうおじいちゃんになっていた(死

Actions #8

Updated by toshi_a 初音 over 3 years ago

  • Status changed from マージ待ち to 終了

merged.

Actions

Also available in: Atom PDF