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
When adding new canvas strategies it is very easy to forget to use the setElementsToRerenderCommand when required (which is most of the time). The CustomStrategyState already has a field elementsToRerender. It might make more sense to remove the command, and to bring elementsToRerender up into the StrategyApplicationResult to enforce a conscious decision around whether elements need to be re-rendered as a result of applying this strategy.
The text was updated successfully, but these errors were encountered:
When adding new canvas strategies it is very easy to forget to use the
setElementsToRerenderCommand
when required (which is most of the time). TheCustomStrategyState
already has a fieldelementsToRerender
. It might make more sense to remove the command, and to bringelementsToRerender
up into theStrategyApplicationResult
to enforce a conscious decision around whether elements need to be re-rendered as a result of applying this strategy.The text was updated successfully, but these errors were encountered: