aroberge / reeborg

Enhanced Karel-the-robot clone
http://reeborg.ca/reeborg.html
Other
47 stars 36 forks source link

3 bugs in development version #322

Closed aroberge closed 7 years ago

aroberge commented 7 years ago

These bugs may turn out to be related:

In import_world, edit_robot_menu.toggle() appears to be undefined. The clear_trace button does not do anything (See other comment below.) When saving a world in the browser, the dialog does not go away when clicking ok

aroberge commented 7 years ago

4th bug: when loading a world that has an empty goal, running a simple program (say a single move() instruction), nothing is given as feedback in the text. This may be due to a recent change in check_goal().

aroberge commented 7 years ago

When a selector for frames was added, I had to change the way that the robot trace was drawn so that we could go back in history and ensure that the correct trace (up to that point) was drawn correctly. Before, the trace was added to denote what changed in a given frame, and it was possible to erase everything up to that point. With the new trace history, it does not make sense to attempt to have a button that erases the trace, and this should be removed.