提案 #1525: closeupメソッドの利用をやめる - mikutter - やること

Custom queries

2021除夜のチケット棚卸し
注目のバグ

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 Tsutsui				Actions
	Related to バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated	終了	Izumi Tsutsui				Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

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

Updated by Izumi Tsutsui over 3 years ago

File 0001-fix-deprecated-messages.patch 0001-fix-deprecated-messages.patch added
File 0002-replace-closeup-mui.patch 0002-replace-closeup-mui.patch added
File 0003-repace-deprecated-mastodon_account_viewer.patch 0003-repace-deprecated-mastodon_account_viewer.patch added
Status changed from 分類待ち to パッチ適用待ち

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

0001-fix-deprecated-messages.patch
deprecated メッセージを実作業の置き換え内容に修正
0002-replace-closeup-mui.patch
core/mui 以下の closeup() を置き換え
0003-repace-deprecated-mastodon_account_viewer.patch
mastodon_account_viewer の closeup(), .tooltip, .right を置き換え

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