Project

General

Profile

Actions

バグ #1537

closed

gtk3: Plugin::GUI::Timelineのdestroyが呼び出されていない (gui_destroyイベントで飛んでこない)

Added by Shibafu Midorino 2 months ago. Updated 28 days ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
ブランチ:
topic/1537-timeline-destroy
クラッシュする:
No

Description

Timelineを含む一時タブを削除した時にPlugin::GUI::Timelineのgui_destroyイベントが飛んできていません。どうも実態が削除された時にimaginaryのdestroyをきちんと呼び出せていないような感じがします。

このイベントが呼ばれないことによって、具体的には user_detail_view プラグインが動作不良を起こすことを確認しています。


再現手順

1. twitter userのプロフィールタブを開く
2. 1で開いたタブを閉じる
3. mikutterを終了する
4. mikutterを再起動すると2で閉じたはずのタブが生き返っている

Actions #1

Updated by Shibafu Midorino 28 days ago

  • Status changed from 分類待ち to 実装待ち
Actions #2

Updated by Shibafu Midorino 28 days ago

  • Status changed from 実装待ち to レビュー待ち
  • 再現手順 updated (diff)
  • ブランチ set to topic/1537-timeline-destroy

修正しました。
GTK2版ではInnerTLのdestroyシグナルハンドラでimaginaryのdestroyを呼んでいたのが、GTK3版ではPlugin::Gtk3::Timeline内のListBoxのdestroyシグナルハンドラになっていたのですが、このハンドラは何故か一生実行されないので親であるPlugin::Gtk3::Timeline自体にハンドラをconnectしておきました。

再現手順にはとりあえずこの問題を発見した時の操作手順を書いておきましたが、これだと見れる人が限られるような…

Actions #3

Updated by Izumi Tsutsui 28 days ago

  • Status changed from レビュー待ち to マージ待ち

develop相当で git merge topic/1537-timeline-destroy して再現手順を試して
閉じたタブが蘇らないことを確認しました。

Actions #4

Updated by toshi_a 初音 28 days ago

  • Status changed from マージ待ち to 終了

merged.

Actions

Also available in: Atom PDF