Project

General

Profile

Actions

バグ #1531

closed

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

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

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

Description

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

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


Files

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
Actions #1

Updated by Izumi Tsutsui almost 3 years ago

  • プラグイン名 deleted (set_view)

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

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

Actions #2

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from 分類待ち to 実装待ち

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

Actions #3

Updated by Izumi Tsutsui almost 3 years ago

とりあえず単純に 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 配置がおかしい?
枠を重ねると横にはみ出す問題が出るから変えてみたということなのかも。

Actions #4

Updated by Izumi Tsutsui almost 3 years ago

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

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

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

Actions #5

Updated by Izumi Tsutsui almost 3 years ago

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

Actions #6

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1531-revive-setting-frame

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

Actions #7

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from レビュー待ち to 終了
Actions

Also available in: Atom PDF