Skip to content
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

feat: add parameter default value options #72

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

nekobako
Copy link
Contributor

Resolves #64 です。
なくてもおそらく困りはしない設定なので Inspector の一番下に詳細設定の項目を追加してそこに入れました。

Expression Parameters がデフォルト値の設定に関わらず False や 0 になるかもしれない件については私の方では確認できていませんが、絶対にないとは言えないので注意文言を表示するようにしています。

@nekobako
Copy link
Contributor Author

nekobako commented May 18, 2024

ItemToggler の処理が CostumeChanger と比べて特殊に感じるかもしれないのでその補足を書いておきます。

まず前提として、ItemToggler では

  1. アバターアップロード前の状態
  2. アバターリセット後の状態
  3. セーフティーでアニメーションが無効になっている状態

が一致することを保証したいというのがあります。
これは現在の LI でもそうで、CostumeChanger にはメニューの状態数 (= コスチュームの数) だけアバターの状態を指定できるのに対し、ItemToggler のアニメーション設定に指定できるのが 1 状態だけなのはこの思想からと認識しています。

この前提なしに「デフォルト値」の仕様を考えると、

  • A
    • ItemToggler のアニメーション設定に指定されている状態は、メニューをオンにしたときの状態を表す
    • 「デフォルト値」を False にした場合、アバターリセット時にメニューがオフになり、アバターはアップロード前と同じ状態になる
    • 「デフォルト値」を True にした場合、アバターリセット時にメニューがオンになり、アバターは ItemToggler のアニメーション設定に指定されている状態になる
    • 要するに「デフォルト値」は ItemToggler のアニメーション設定に指定されている状態をアバターの初期状態として扱うかどうかを表す
  • B
    • ItemToggler のアニメーション設定に指定されている状態は、メニューを初期状態から変更したときの状態を表す
    • 「デフォルト値」を False にした場合、アバターリセット時にメニューがオフになり、アバターはアップロード前と同じ状態になる
    • 「デフォルト値」を True にした場合、アバターリセット時にメニューがオンになり、アバターはアップロード前と同じ状態になる
    • 要するに「デフォルト値」はアバターの初期状態と紐づくパラメータの値を表す

の 2 パターンが考えられますが、A だと前述の前提を満たさなくなってしまう上に、本 PR で実現したかった #64 の「初期状態がアクティブなオブジェクトのアクティブ/非アクティブをメニューのオフ/オンではなくオン/オフに対応させたい」を実現できるのも B なので、B の仕様を採用しています。

補足は以上ですが、CostumeChanger も前述の前提に従った方がよいのではないかという issue が #68 になります。
本 PR がマージされた場合、#68 で「0 番目の衣装」と表記しているところは「デフォルト番目の衣装」と読み替えることになります。

@nekobako nekobako mentioned this pull request May 18, 2024
@nekobako nekobako force-pushed the parameter-default-value branch from 7c18463 to eb7ee59 Compare May 24, 2024 07:49
@lilxyzw
Copy link
Owner

lilxyzw commented Jun 5, 2024

pull requestありがとうございます
LIの思想的に安全寄りにしているので初期値は設定できないようにしていましたが、警告が表示される仕様ならありかなと思ったので取り込もうと思います!

@lilxyzw lilxyzw merged commit 73be7a0 into lilxyzw:dev Jun 5, 2024
@nekobako
Copy link
Contributor Author

nekobako commented Jun 5, 2024

ありがとうございます!

@nekobako nekobako deleted the parameter-default-value branch June 8, 2024 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants