Project

General

Profile

Actions

提案 #1255

open

Worldの選択を保持する

Added by Yuto Tokunaga over 6 years ago. Updated about 6 years ago.

Status:
実装待ち
Priority:
通常
Target version:
プラグイン名:
current_world
ブランチ:

Description

起動時に前回の終了時に選択していたWorldを復元する機能の提案です.


Files

1255-1.patch (1.67 KB) 1255-1.patch Yuto Tokunaga, 2018-05-27 19:51
1255-2.patch (1.18 KB) 1255-2.patch master (4cf82920)に対するパッチ Yuto Tokunaga, 2018-06-02 02:11
Actions #1

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 新規 to 実装待ち
  • Target version changed from 3.7 to 3.9

良い提案だと思います。ただし以下の修正をしないと取り込めないと思っています。

mergeする条件

current_worldプラグインにこの機能をもたせる

「現在のWorld」という考え方はcurrent_worldプラグインによって提供されていて、gtkはそれに依存しているという形です。レジューム機能を実装するなら、UIを提供しているgtkより、current_worldまたは新たなプラグインでやるのが良いと思います。個人的には、current_worldプラグインに機能追加するのが良いと思います。

Worldを保存する時、world_change_currentにのみをlistenする

これは細かな話ですが、カレントworldが切り替わる時には、どのような理由であってもworld_change_currentイベントが発生します。当然Worldが追加・削除されることによって変わった場合でも同じです。また、一つのプラグインが同じイベントに複数のリスナを登録できます。

この2つの性質を利用すれば、カレントworldを保存する処理をメソッドに抽出する必要がなくなるので、もう少しシンプルになると思います。

mergeにあたって必須ではないこと

これは個人的にはぜひやりたいので、条件が達成されればmergeする前に私が実装しようと思います。

レジューム機能の無効化設定

個人的には今の、一番頭のWorldが選択されている状態で起動するのを気に入っていますが、一般的にはこのチケットで達成しようとしているような振る舞いが受け入れられると思います。

このパッチを取り込むと現在のような挙動を実現するのが難しくなるので、設定できるようにしたいところです。

Actions #2

Updated by Yuto Tokunaga over 6 years ago

修正しました. UserConfig[:restore_last_world] で無効化できるようにしましたが,設定UIは実装してません.

Actions #3

Updated by Izumi Tsutsui over 6 years ago

  • Tracker changed from 機能 to 提案
  • Start date deleted (2018-05-27)

これも「提案」に変えてみます。

「実装待ち」と「パッチ適用待ち」の中間の「仕様・実装についてレビュー中」みたいな状態の場合に
ステータスをどうするか悩ましいですが、とりあえず運用としてどちらかに含めてしまうべき?
担当者としてレビュアーをアサインしてその人がアクションすることにする?

Actions #4

Updated by Izumi Tsutsui about 6 years ago

  • Assignee set to toshi_a 初音

新しいステータス「toshi_aの判断待ち」とするのが適切なチケット内容ですが、
「実装待ち」からの遷移ができないのでとりあえず toshi_a さん担当に変えておきます。

としぁさん自身で追加実装したいと考えている部分があるのであれば「機能」に戻すべきかも。

Actions

Also available in: Atom PDF