Project

General

Profile

Actions

提案 #1525

closed

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

Added by toshi_a 初音 3 months ago. Updated about 1 month ago.

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

Description

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


Files


Related issues

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

Updated by Izumi Tsutsui 3 months 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 #2

Updated by Izumi Tsutsui about 2 months ago

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

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

Actions #3

Updated by Izumi Tsutsui about 2 months ago

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

Updated by Izumi Tsutsui about 2 months ago

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

Actions #5

Updated by toshi_a 初音 about 2 months ago

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

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

Actions #6

Updated by Izumi Tsutsui about 2 months ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音

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

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

これはどうしましょうか。(いったん まだダメ にすべき?)

Actions #7

Updated by Izumi Tsutsui about 2 months ago

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

Updated by Izumi Tsutsui about 2 months ago

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

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

これはどうしましょうか。(いったん まだダメ にすべき?)

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

Actions #9

Updated by toshi_a 初音 about 1 month ago

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

merged

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

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

Actions

Also available in: Atom PDF