Closed afomi closed 4 years ago
Thanks for taking time on this. I need to think about the changes. Things I am not sure about:
map.js just has the logic of the map. Height and width are presentation details. Best to keep them somewhere else.
renderSwardley has been written to be used in the browser, imported into server-side javascript or called from a framework like React. The draw function is implementation specific and probably belongs somewhere else.
We could keep map.js and renderSwardley.js and do everything else in a script in index.html. What do you think?
Makes sense. 👍 Thanks for the review!
Closing. See comments for resolution