バグ #1531
closedgtk3: 設定→表示 のフォント選択 のウインドウ構成が gtk2 時の意匠と異なる?
Description
各種 gtk3 チケットが解決してきて mikutter_gtk3 がだいたい実用上問題なく動くようになってきていますが
いざリリースすると「gtk2 時代と違う」と言われそうな挙動をメモるチケットその5 というのは微妙ですが、
百聞は一見にしかず、ということでこの外観って意図通りなんでしょうか。なんとなく暫定っぽいのですが。
Files
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 の最初のコミットからこうなので、
暫定でこうなっているのか意図してこのデザインなのかが判別できませんね。
Updated by toshi_a 初音 almost 3 years ago
- Status changed from 分類待ち to 実装待ち
これはもとに戻したいと思っているので実装待ちに変更しておきます。
Updated by Izumi Tsutsui almost 3 years ago
- File 1531-settings-3.png 1531-settings-3.png added
- File 1531-settings-2.png 1531-settings-2.png added
- File 1531-settings-1.png 1531-settings-1.png added
とりあえず単純に 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 配置がおかしい?
枠を重ねると横にはみ出す問題が出るから変えてみたということなのかも。
Updated by Izumi Tsutsui almost 3 years ago
- File 0001-revival-settings.patch 0001-revival-settings.patch added
- File 1531-settings-fix-1.png 1531-settings-fix-1.png added
- File 1531-settings-fix-2.png 1531-settings-fix-2.png added
- File 1531-settings-fix-3.png 1531-settings-fix-3.png added
- Status changed from 実装待ち to パッチ適用待ち
mikutter も GTK もなんもわからん、とつぶやきながらいじってそれっぽくなった感じです。
デフォルトのウインドウ幅がどこで決まっているのかがよくわかりませんが、
アクティビティのように項目の文字列が横に長い場合に折り返す仕様とか考えるべき?
Updated by Izumi Tsutsui almost 3 years ago
としぁさんが力尽きた感がありますが 0b52816e をgit merge topic/1531-revive-setting-frame
して
パッチそのままなので設定表示OKです。
Updated by toshi_a 初音 almost 3 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- ブランチ set to topic/1531-revive-setting-frame
力尽きていたのか(驚愕)