-
-
Notifications
You must be signed in to change notification settings - Fork 71
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: Create Toggle for Selection #1400
Conversation
Setup Toggle と Create Toggleの名前で違いがわからないのであまりよろしくないですね・・・
|
objToggle.Objects.Add(new ToggledObject | ||
{ | ||
Object = new AvatarObjectReference(){ referencePath = path }, | ||
Active = !selected.activeSelf | ||
}); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
メニューの状態とオブジェクトの状態を合わせたほうがわかりやすいかも
つまり MenuItemのDefault = activeSelf, Inverted = activeSelf, Active = !activeSelf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
アニメーションは再生されているものの、Expression Parameterが意図せずオフで固定されるというような状況は、恐らく今も存在すると思うのですが(相手だけ先にアバターが読み込めている時とかFBT Callibration時とか)、その場合もbd_さんの方式(オブジェクトのオンオフにメニューのオンオフを合わせる)の方が良いのでしょうか…?
(意図せず脱げる等が発生しそう)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あー確かに。ただ、それを対策するならMenuItem側で対策したいかな?(同期パラメーターとメニュー用パラメーターを別にするとか)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ふーむ(負荷面であんまりParameter Driver使う箇所を増やさない方が良さそうな感じもしつつ…?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
とりあえず fa4e714 でメニューの状態とオブジェクトの状態を合わせるようにしました。Revertは可能です。
同期された方が基本的に分かりやすいのはあるので悩ましいところではありますが…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うーん…MenuItem側で対策できるのはしばらく先なので、やっぱり元の仕様(OFF=基本状態)にいったんしておいて、MenuItem側の対策ができたら変更しましょう。
|
This reverts commit fa4e714. This is temporarily reverted until we have support in MenuItem to ensure that `0` is always the default selection, even for toggles. This should be reverted when bdunderscore#1433 is done (which will resolve bdunderscore#1434)
closes #1399
不要な場合は閉じてもらって大丈夫です