search

thechiselgroup / biomixer

BioMixer
http://bio-mixer.appspot.com/
16 stars 13 forks source link

Undo Redo Crumb Name Should Reflect Number of Nodes Added #465

Closed everbeek closed 9 years ago

everbeek commented 9 years ago

When updating the crumb names, add how many nodes were expanded. Helps distinguish the same expansion made a few times in a row.

everbeek commented 9 years ago

Have it working, and will commit once I clean it up a bit.

It seemed complicated to do this for the composite expansion sets in particular. This is a sore point, so I should look at refactoring after I commit the working code.

everbeek commented 9 years ago

Managed to clean up multiple ways of updating breadcrumb text, and am only using the listener system now. If there is need for the old approach for a future feature, it will be easy to recreate or pull from the history.