flackr / dungeon

A D&D based multiplayer role playing game.
13 stars 4 forks source link

Sidebar character selection does not match UI selection. #2

Closed flackr closed 12 years ago

flackr commented 12 years ago

Summary.

The character which is shown in the sidebar is not necessarily selected. We should have a stronger guarantee that when the selection changes the sidebar will also update.

Implementation.

I recommend implementing this by dispatching selectionChange events in dungeon-client.js and listening for those from combat-tracker.js. It should be fairly straight-forward.

kellis commented 12 years ago

Added highlighting on the map to show which character is selected. Dispatch a 'power-selected' event from combat-tracker.js that dungeon-client.js listens to in order to re-select the character. The character that is visible in the tracker should remain visible even when not selected in dungeon-client.js provided a different character is not selected. Selecting another character on the map updates the sidebar accordingly. Furthermore, it now updates the character details page as well with full character information.