-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
「保存」ボタンでProsetsリストが更新されるようにしました #43
Conversation
This fork is based on v1.15. The base side was updated to v1.16 two hours before posting the pull request, but fortunately v1.15 and v1.16 seem to be the same, so I think that it is possible to merge as it is. (Is #42-v1.16 resolved by a rebuild?) このフォークは v1.15 をベースにしています。pull requestを投稿する2時間前に ベース側が v1.16 にアップデートされましたが、幸いなことに v1.15とv1.16は同一のようですのでこのままmerge可能だと思います。(#42-v1.16はリビルドで解決ですか?) |
ありがとうございます。1.16は1行変更しただけですので問題ないかと思います。 |
下記URLからベータ版をダウンロードできるようにしました。 突然のPULLリクエストに答えていただきありがとうございます。 |
ありがとうございます。手元でのビルドも試してみました。プリセット更新後にちゃんと反映されるのを確認しました。実際動かしてざっと読ませていただいた限り、おそらく大丈夫そうなのですが、けっこう大きめの変更なのでリリース前にもういっぺんcoreのコードも合わせて読むことにしようと思います。
そうえば「保存したらリストが更新されるようにしたいけれど実際やろうとするとかなり大掛かりになりそうだからやめとこう」と思ってあきらめていたのを思い出しました。 |
Modified so that the edited result in the PresetEditorDialog is reflected to the ManagePresetsDialog and the GroupPresetMenu without restarting JOSM.
PresetEditorDialogでの編集結果がJOSMを再起動しなくてもManagePresetsDialogとGroupPresetMenuに反映されるように修正しました。