Project

General

Profile

Actions
Copy link

提案 #1525

closed

closeupメソッドの利用をやめる

Added by toshi_a 初音 over 3 years ago. Updated over 3 years ago.

Status:
終了
Priority:
通常
Assignee:
toshi_a 初音
Target version:
5.0
プラグイン名:
ブランチ:
topic/1525-avoid-using-closeup

Description

#1515 の続きで、標準でcloseupメソッドを使っているのをやめる。
互換性のため、closeupメソッド自体はdeprecateとして残す。ただ、多く使われているサードパーティプラグインが大量に警告を出す場合は、標準プラグインの利用をやめるがdeprecateにしないといった対応も考えられる。

Files

Download all files
0001-fix-deprecated-messages.patch (2.08 KB) 0001-fix-deprecated-messages.patch Izumi Tsutsui, 2021-11-27 02:24
0002-replace-closeup-mui.patch (5.72 KB) 0002-replace-closeup-mui.patch Izumi Tsutsui, 2021-11-27 02:24
0003-repace-deprecated-mastodon_account_viewer.patch (3.85 KB) 0003-repace-deprecated-mastodon_account_viewer.patch Izumi Tsutsui, 2021-11-27 02:24
0004-modelviewer-deprecated.patch (1.24 KB) 0004-modelviewer-deprecated.patch Izumi Tsutsui, 2021-11-28 11:11

Related issues 2 (0 open — 2 closed)

Related to バグ #1515: gtk3: mui 関連の deprecated 警告終了Izumi TsutsuiActions
Related to バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated終了Izumi TsutsuiActions
Actions
Copy link
 #1

Updated by Izumi Tsutsui over 3 years ago

81a86502 だとほかにも以下が deprecated にされています。
  • deprecate :top, :valign=, 2018, 9
  • deprecate :center, :halign=, 2018, 9
  • deprecate :left, :halign=, 2018, 9
  • deprecate :right, :halign=, 2018, 9
  • deprecate :tooltip, :tooltip_text=, 2018, 9

こちらも機械的置き換えなのでいいのですが、 deprecated のメッセージがもう少し親切にならないかなあ、とは思いました。

twitter の user_detail_view だと

  relation_container.closeup(::Gtk::WebIcon.new(user.icon, icon_size).tooltip("#{user.idname}(#{user[:name]})"))

みたいなのがあって、 :tooltip_text= と言われても、みたいな。（素人ですいません）
これは set_tooltip_text にしましたが。
https://github.com/tsutsui/user_detail_view/commit/55d5c8f872b75f71ea1aa5d06920a54b0a57b2ca
Actions
Copy link Download all files
 #2

Updated by Izumi Tsutsui over 3 years ago

ガッツが足りない日々の中で作業した気分になるために機械的にやってみました。
思ったよりは多くなくて source:core/mui@f6025344 と source:plugin/mastodon_account_viewer@f6025344 だけっぽいです。

:left :center :right :top はほかにもあるかもしれません（未確認）。
……と、差分見てたらパッチの中の source:core/mui/gtk_mtk.rb@f6025344 にも .left と .right ありますね。
Mtk 自体 deprecated という気もするのでこれらはそのうちで。

Actions
Copy link
 #3

Updated by Izumi Tsutsui over 3 years ago

  • Related to バグ #1515: gtk3: mui 関連の deprecated 警告 added
Actions
Copy link
 #4

Updated by Izumi Tsutsui over 3 years ago

modelviewer にも .tooltip あって deprecated 出ていたのでついででパッチ添付しておきます。
mastodon の「ユーザーについて」で参照されてます。

Actions
Copy link
 #5

Updated by toshi_a 初音 over 3 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui
  • ブランチ set to topic/1525-avoid-using-closeup

すべて取り込みます。ついでにちょうどpostbox気になっていたので混ぜておきました

Actions
Copy link
 #6

Updated by Izumi Tsutsui over 3 years ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音

チケット添付のパッチはこちらで使っているものそのままなのでOKと思います。

9729806e の gtk_postbox 分の2つ目の get_iter_at_offset が
#1547 と重複してしまっています。（あと #1547 では end の改行も入れている）

これはどうしましょうか。（いったん まだダメ にすべき？）

Actions
Copy link
 #7

Updated by Izumi Tsutsui over 3 years ago

  • Related to バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated added
Actions
Copy link
 #8

Updated by Izumi Tsutsui over 3 years ago

9729806e の gtk_postbox 分の2つ目の get_iter_at_offset が
#1547 と重複してしまっています。（あと #1547 では end の改行も入れている）

#1547-4 にも書きましたが、 #1547 を先にマージしてからこちらをマージして
conflict部分を手編集で処置しまえばよさそうです。

これはどうしましょうか。（いったん まだダメ にすべき？）

「提案」チケットだと「レビュー待ち」のあとの「マージ待ち」がないということに気づきました……

Actions
Copy link
 #9

Updated by toshi_a 初音 over 3 years ago

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

merged

「提案」チケットだと「レビュー待ち」のあとの「マージ待ち」がないということに気づきました……

ミスだったので修正しました。

Actions
Copy link

Also available in: Atom PDF