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
actions will need a new method to return a compact string for the tree
might be beneficial to change how broken actions are handled, so the action JSON knows its broken and it can be marked as such in the tree visualization
actions on various branch depth are "indented" to show they are branches
undo actions spins off a branch; undo and redo actions are hidden from the tree visualization
action tree page will have a selection toolbar
when one action is selected, show "review from here" and "undo to here" buttons, like in history view
when in new mode, actions still validated by server (will need 'branch' param passed in so server knows which branch to validate, or maybe just param of last action to process) and added to actions list, but users will only see their own such branches
from normal mode, can select some premoved actions to "graft" them on to the current mainline branch and apply them now
option to hide grafted actions from the tree view
branch tracking?
each action has a branch field now? hopefully something more lightweight can be found
each action's parent and children can (should?) be computable
messages will need special handling
Benefits
will help with implementation of premoves
make undone actions discoverable/reviewable again--useful for debugging, and seeing paths explored by players
if an action is broken, but a lot of actions have been taken after it, can go back in time to that point, fix the broken action, and then graft on all the actions that followed
makes going back to change one small thing a turn more feasible--don't need to manually repeat all the actions since then, just graft them
The text was updated successfully, but these errors were encountered:
undo
actions spins off a branch;undo
andredo
actions are hidden from the tree visualizationBenefits
The text was updated successfully, but these errors were encountered: