プロジェクト

全般

プロフィール

バグ #1531

完了

gtk3: 設定→表示 のフォント選択 のウインドウ構成が gtk2 時の意匠と異なる?

Izumi Tsutsui さんが2年以上前に追加. 2年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
-
対象バージョン:
プラグイン名:
クラッシュする:
いいえ

説明

各種 gtk3 チケットが解決してきて mikutter_gtk3 がだいたい実用上問題なく動くようになってきていますが
いざリリースすると「gtk2 時代と違う」と言われそうな挙動をメモるチケットその5 というのは微妙ですが、

百聞は一見にしかず、ということでこの外観って意図通りなんでしょうか。なんとなく暫定っぽいのですが。


ファイル

mikutter_gtk3-settings-appearance.png (75.5 KB) mikutter_gtk3-settings-appearance.png Izumi Tsutsui, 2021-11-01 23:14
mikutter_gtk2-settings-appearance.png (64.4 KB) mikutter_gtk2-settings-appearance.png Izumi Tsutsui, 2021-11-01 23:14
1531-settings-3.png (72.7 KB) 1531-settings-3.png Izumi Tsutsui, 2021-12-15 20:21
1531-settings-2.png (64.7 KB) 1531-settings-2.png Izumi Tsutsui, 2021-12-15 20:21
1531-settings-1.png (59.7 KB) 1531-settings-1.png Izumi Tsutsui, 2021-12-15 20:21
0001-revival-settings.patch (1.65 KB) 0001-revival-settings.patch Izumi Tsutsui, 2021-12-16 01:12
1531-settings-fix-1.png (61.9 KB) 1531-settings-fix-1.png Izumi Tsutsui, 2021-12-16 01:13
1531-settings-fix-2.png (70.2 KB) 1531-settings-fix-2.png Izumi Tsutsui, 2021-12-16 01:13
1531-settings-fix-3.png (96.3 KB) 1531-settings-fix-3.png Izumi Tsutsui, 2021-12-16 01:13

Izumi Tsutsui さんが2年以上前に更新

  • プラグイン名 を削除 (set_view)

当該実装箇所は
source:core/mui/gtk_form_dsl.rb@1f9a609b#L459
settings なので set_view ではなくて mui ですね。

ログだと c3ad6bc1 の GTK3-ish の最初のコミットからこうなので、
暫定でこうなっているのか意図してこのデザインなのかが判別できませんね。

toshi_a 初音 さんが2年以上前に更新

  • ステータス分類待ち から 実装待ち に変更

これはもとに戻したいと思っているので実装待ちに変更しておきます。

Izumi Tsutsui さんが2年以上前に更新

とりあえず単純に revert するだけでもある程度それっぽい表示にはなるようです。

diff --git a/core/mui/gtk_form_dsl.rb b/core/mui/gtk_form_dsl.rb
index c498afa1..fc9b01c2 100644
--- a/core/mui/gtk_form_dsl.rb
+++ b/core/mui/gtk_form_dsl.rb
@@ -461,18 +461,16 @@ module Gtk::FormDSL
   # [title] ラベル
   # [&block] ブロック
   def settings(title, &block)
-    @headings ||= []
-    @headings << title
-
-    label = Gtk::Label.new @headings.map { |s| "<b>#{s}</b>" }.join ' > '
-    label.use_markup = true
-    label.halign = :start
-    attach_next_to label, nil, :bottom, 2, 1
-
-    instance_eval(&block)
-    @headings.pop
-
-    Chainable.new label
+    group = Gtk::Frame.new.set_border_width(8)
+    if(title.is_a?(Gtk::Widget))
+      group.set_label_widget(title)
+    else
+      group.set_label(title)
+    end
+    box = create_inner_setting.set_border_width(4)
+    box.instance_eval(&block)
+    add group.add(box)
+    group
   end

   # 〜についてダイアログを出すためのボタン。押すとダイアログが出てくる

ただ、「表示」の設定についてはなんか右にはみ出している感じです。
特に「タブの位置」「投稿ボックス」「アカウント切り替え」のプルダウンの Grid 配置がおかしい?
枠を重ねると横にはみ出す問題が出るから変えてみたということなのかも。

Izumi Tsutsui さんが2年以上前に更新

mikutter も GTK もなんもわからん、とつぶやきながらいじってそれっぽくなった感じです。

添付したパッチと同じ差分

デフォルトのウインドウ幅がどこで決まっているのかがよくわかりませんが、
アクティビティのように項目の文字列が横に長い場合に折り返す仕様とか考えるべき?

Izumi Tsutsui さんが2年以上前に更新

としぁさんが力尽きた感がありますが 0b52816e
git merge topic/1531-revive-setting-frame して
パッチそのままなので設定表示OKです。

toshi_a 初音 さんが2年以上前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • ブランチtopic/1531-revive-setting-frame にセット

力尽きていたのか(驚愕)

toshi_a 初音 さんが2年以上前に更新

  • ステータスレビュー待ち から 終了 に変更

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