mikutter

1. 一般的には

Rubyらしさという観点では、value= ですね。

Ruby Style Guideでもそう定義されています。 https://rubystyle.guide/#accessor_mutator_method_names

ゲッターの get_value は多くのルールで禁止されていて、 value とすべきです。一方、 set_value はメソッドチェーン的に書けたほうが嬉しいこともあるからか、禁止しないルールも見覚えがあります。

1. mikutterでは

mikutterのコーディング規約は source:.rubocop.yml にあり、rubocopで検出できるコーディング規約違反が新規に含まないようにしていますが、今確認したら Naming/AccessorMethodName が無効になってますね。これは理由を調査したほうが良さそう（先のset_を許容するためか？）。
ただし、以上はメソッド定義に関するルールで、呼び出しに関するルールではない認識です。同様に適用して良いと思います。

漏れがありそうですが↓を2つとも満たせば set_ を使っても良い、かも

• メソッドチェーンを使えばウィジェットを変数に代入することを避けられる場合
• 代入するプロパティが1-2個など、少ない場合

toshi_a 初音 さんは #note-3 で書きました:

漏れがありそうですが↓を2つとも満たせば set_ を使っても良い、かも

• メソッドチェーンを使えばウィジェットを変数に代入することを避けられる場合
• 代入するプロパティが1-2個など、少ない場合

解説ありがとうございます。だいたい自分がコードを読んだ感覚的にはそんな感じかなというところです。

実装して書いている側の立場だと、メソッドチェーンでまとめて書いたほうが全体脳内設計を吐き出しやすい一方で、
ドキュメント無しでコードから設計を読み解く（あるいはコード≒設計書になるようにしてサボる）立場から見ると
個別要素が1行ずつ書いてあったほうが把握しやすい、ということかなあ、
みたいな印象です。

添付のパッチ 0001-no-overlay-settings.patch については
set_policy で hscroll_policy と vscroll_policy の2つを代入する、
というのが微妙ですが、GTK側の設計としてオブジェクト指向的定義に失敗している、という感じですかね。

TLは今までも常時表示するように意図的に指定していたので酸性だったのですが、設定にもやりだすと、あらゆるところに設定して回る必要があるのできりがないんですが、Gtk2のときも環境変数で指定していて全てのスクロールバーが表示されていたのと、スクロールバー出ているほうが嬉しいと思ってしまうので、TLと設定くらいはスクロールバー出しておきましょうか。

toshi_a 初音 さんは #note-6 で書きました:

Gtk2のときも環境変数で指定していて全てのスクロールバーが表示されていたのと、

Gtk::Settings.default.gtk_overlay_scrolling = false
を指定すると全部のウインドウに適用されて全部スクロールバーオーバーレイ無効になるっぽいです。
これをどこに書くべきなのかがピンときませんが、plugin/gtk3/gtk3.rb の最初に書いておけば一応効いてるっぽいです。

このままmergeして、他に同様の起票があった時に考えます。

toshi_a 初音 さんは #note-9 で書きました:

このままmergeして、他に同様の起票があった時に考えます。

了解です。
たちまち気づくのは UserList くらいですね（フォロー・フォロワー・ふぁぼられ・RT一覧）

同様の提案が入ったら修正方針として #note-8 を検討したいということです。俺は気になってないです。

メモ書きですが

Gtk2のときも環境変数で指定していて

いまさらですが
mikutter.rb の
ENV['LIBOVERLAY_SCROLLBAR'] = '0'
のことだとすると、GTK3 的にはこれを
ENV['GTK_OVERLAY_SCROLLING'] = '0'
にするだけですね……。

mikutter.rb で設定するのが妥当なのか、という観点はありますが。