操作
提案 #1525
完了
closeupメソッドの利用をやめる
プラグイン名:
説明
#1515 の続きで、標準でcloseupメソッドを使っているのをやめる。
互換性のため、closeupメソッド自体はdeprecateとして残す。ただ、多く使われているサードパーティプラグインが大量に警告を出す場合は、標準プラグインの利用をやめるがdeprecateにしないといった対応も考えられる。
ファイル
Izumi Tsutsui さんが3年以上前に更新
81a86502 だとほかにも以下が deprecated にされています。
みたいなのがあって、
これは
https://github.com/tsutsui/user_detail_view/commit/55d5c8f872b75f71ea1aa5d06920a54b0a57b2ca
deprecate :top, :valign=, 2018, 9deprecate :center, :halign=, 2018, 9deprecate :left, :halign=, 2018, 9deprecate :right, :halign=, 2018, 9deprecate :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
Izumi Tsutsui さんが3年以上前に更新
- ファイル 0001-fix-deprecated-messages.patch 0001-fix-deprecated-messages.patch を追加
- ファイル 0002-replace-closeup-mui.patch 0002-replace-closeup-mui.patch を追加
- ファイル 0003-repace-deprecated-mastodon_account_viewer.patch 0003-repace-deprecated-mastodon_account_viewer.patch を追加
- ステータス を 分類待ち から パッチ適用待ち に変更
ガッツが足りない日々の中で作業した気分になるために機械的にやってみました。
思ったよりは多くなくて source:core/mui@f6025344 と source:plugin/mastodon_account_viewer@f6025344 だけっぽいです。
思ったよりは多くなくて 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 という気もするのでこれらはそのうちで。
Izumi Tsutsui さんが3年以上前に更新
modelviewer にも .tooltip あって deprecated 出ていたのでついででパッチ添付しておきます。
mastodon の「ユーザーについて」で参照されてます。
toshi_a 初音 さんが3年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Izumi Tsutsui に変更
- ブランチ を topic/1525-avoid-using-closeup にセット
すべて取り込みます。ついでにちょうどpostbox気になっていたので混ぜておきました
Izumi Tsutsui さんが3年以上前に更新
- 関連している バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated を追加
toshi_a 初音 さんが3年以上前に更新
- ステータス を レビュー待ち から 終了 に変更
merged
「提案」チケットだと「レビュー待ち」のあとの「マージ待ち」がないということに気づきました……
ミスだったので修正しました。
操作