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: Create Toggle for Selection #1400

Merged
merged 7 commits into from
Feb 3, 2025
Merged

Conversation

Tliks
Copy link
Contributor

@Tliks Tliks commented Dec 20, 2024

closes #1399
不要な場合は閉じてもらって大丈夫です

@bdunderscore
Copy link
Owner

Setup Toggle と Create Toggleの名前で違いがわからないのであまりよろしくないですね・・・

Create Toggles for Selectionとかのほうがいいかも?

Comment on lines +59 to +64
objToggle.Objects.Add(new ToggledObject
{
Object = new AvatarObjectReference(){ referencePath = path },
Active = !selected.activeSelf
});
}
Copy link
Owner

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

Copy link
Contributor

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_さんの方式(オブジェクトのオンオフにメニューのオンオフを合わせる)の方が良いのでしょうか…?
(意図せず脱げる等が発生しそう)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あー確かに。ただ、それを対策するならMenuItem側で対策したいかな?(同期パラメーターとメニュー用パラメーターを別にするとか)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ふーむ(負荷面であんまりParameter Driver使う箇所を増やさない方が良さそうな感じもしつつ…?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

とりあえず fa4e714 でメニューの状態とオブジェクトの状態を合わせるようにしました。Revertは可能です。
同期された方が基本的に分かりやすいのはあるので悩ましいところではありますが…

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

うーん…MenuItem側で対策できるのはしばらく先なので、やっぱり元の仕様(OFF=基本状態)にいったんしておいて、MenuItem側の対策ができたら変更しましょう。

@Tliks
Copy link
Contributor Author

Tliks commented Feb 2, 2025

Create Toggle for Selectionに名称変更しました

@Tliks Tliks changed the title feat: Setup Toggle feat: Create Toggle for Selection Feb 2, 2025
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)
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.

Feature request: Setup Object Toggle
3 participants