Skip to content

Conversation

@JoshuaGrams
Copy link

I don't know if this is a good way to do this (especially putting the file in lib/ui/content/transcript.js instead of putting it in lib/engine.js...

  • created a lib/ui/content/transcript.js inspired by lib/ui/content/text.js
  • updated lib/engine.js to use it (and turned enableTranscript on)
  • added a window.saveTranscript() function to lib/ui/browser.js
  • added a link to the header and footer (currently commented out?) of lib/templates/default/+index.html

Checklist

  • The new code additions passed the tests (npm test).

Hrm. I don't seem to be able to run the tests? Sorry, probably wonky Windows dev environment.

* created a lib/ui/content/transcript.js inspired by lib/ui/content/text.js
* updated lib/engine.js to use it (and turned enableTranscript on)
* added a window.saveTranscript() function to lib/ui/browser.js
* added a link to the header and footer (currently commented out?) of
  lib/templates/default/+index.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant