Conversation
Assets/SmartAddresser/Editor/Core/Tools/Addresser/LayoutRuleEditor/LayoutRuleEditorPresenter.cs
Show resolved
Hide resolved
Haruma-K
left a comment
There was a problem hiding this comment.
- Layout Viewerの方も対応が必要そうかも?
- 優先度は低いですが、Layout Rule Editor を開いた状態でソート順を変更した時に、その変更を Layout Rule Editor に反映させる対応はできそうでしょうか
- Hackyな感じになるならやらなくてもいいかなと思いつつ
これはすでに出来てる・・と思ったけど、初回だけ反映されないケースがあるみたいでした。 ただLayoutRuleEditorは更新のたびにのOnModificationコールバックを登録しなおしているので、2回目以降は登録順が後ろになってうまくいくようです。 回避するとしたら・・・ModificationEvent.GroupMovedイベントだったらEditorApplication.delayCallで遅延更新するとかでしょうか? |
Haruma-K
left a comment
There was a problem hiding this comment.
一点コメントしましたが、それ以外は問題ないのでApproveします
| return; | ||
| } | ||
|
|
||
| var addressableAssetSettings = AddressableAssetSettingsDefaultObject.Settings; |
There was a problem hiding this comment.
こちらも
#if AAS_SORT_ORDER
が必要そうでしょうか?
There was a problem hiding this comment.
失礼しました!#ifを追加して、sortOrderのないバージョンでエラーにならないことを再確認しました
こちらでマージしますね
AAS側で保存されているAssetGroup並び順をLayoutRuleEditorにも反映します。
動作確認
AddressableAssetGroupSortSettingsが追加される境界バージョンで動作確認しました