You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We understand that GroupSortInteraction does not currently support proper cue behavior for individually disabled balls. This is an enhancement that should eventually be added to GroupSort.
Assigning to @zepumph to begin with in case he has anything to add.
The text was updated successfully, but these errors were encountered:
Yeah. This is a challenge since the cue visibility logic exists in the model, but the general notion of item enabled is Node.enabled. This meant that over in the above issue, we decided to add individual enabled over in the view in 3690a61.
Some challenges with supporting this in the model:
ItemModel as a type would want to support some sort of enabledProperty.
Ideally we combine enabled strategies, and don't just keep the view option from the above commit (isGroupItemEnabled).
A good default for this logic is Node.enabled, but I'm unsure how that would be able to live in the model type.
This issue recognizes that the model/view pattern is a bit strange here, since all model Properties are really about the controller/view interaction, and thus create problems with wanting Node Properties in the GroupSortInteractionModel (which still seems "view-like" per say).
I'm not planning to work on this, but it seems valuable to track for next time we work on group sort.
zepumph
changed the title
Enhance GroupSortInteraction Cue Logic to handle individually disabled balls
[GroupSortInteraction] Cue Logic to handle individually disabled items
Aug 19, 2024
This issue comes from phetsims/mean-share-and-balance#337 (comment)
We understand that GroupSortInteraction does not currently support proper cue behavior for individually disabled balls. This is an enhancement that should eventually be added to GroupSort.
Assigning to @zepumph to begin with in case he has anything to add.
The text was updated successfully, but these errors were encountered: