プロジェクト

全般

プロフィール

提案 #1525

完了

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

toshi_a 初音 さんが2年以上前に追加. 2年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:

説明

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


ファイル


関連するチケット

関連している バグ #1515: gtk3: mui 関連の deprecated 警告終了Izumi Tsutsui操作
関連している バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated終了Izumi Tsutsui操作

Izumi Tsutsui さんが2年以上前に更新

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

Izumi Tsutsui さんが2年以上前に更新

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

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

Izumi Tsutsui さんが2年以上前に更新

  • 関連している バグ #1515: gtk3: mui 関連の deprecated 警告 を追加

Izumi Tsutsui さんが2年以上前に更新

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

toshi_a 初音 さんが2年以上前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から Izumi Tsutsui に変更
  • ブランチtopic/1525-avoid-using-closeup にセット

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

Izumi Tsutsui さんが2年以上前に更新

  • 担当者Izumi Tsutsui から toshi_a 初音 に変更

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

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

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

Izumi Tsutsui さんが2年以上前に更新

  • 関連している バグ #1547: gtk3: TextBuffer#get_iter_at_offset の deprecated を追加

Izumi Tsutsui さんが2年以上前に更新

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

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

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

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

toshi_a 初音 さんが2年以上前に更新

  • ステータスレビュー待ち から 終了 に変更

merged

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

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

他の形式にエクスポート: Atom PDF