Skip to content

Made and Implemented Voice Changing Options!#42

Open
jmoldyvan wants to merge 4 commits intoXollabOS:mainfrom
jmoldyvan:mybranch
Open

Made and Implemented Voice Changing Options!#42
jmoldyvan wants to merge 4 commits intoXollabOS:mainfrom
jmoldyvan:mybranch

Conversation

@jmoldyvan
Copy link
Copy Markdown

Hello Everyone!
I went ahead and altered the code to include an option for voice/accent changing. It could use a little CSS sparkle, but it works with all the listed voices. As well as functioning with the boxes and the typed sentences. I'll list the changes I made in each file below. I also included comments in the JS file. Thanks!

main.JS:

  • globally defined variables voice/voices/select
  • created function to define and populate the "select" tag with getVoices() list
  • add event listener to change the voices/lang based on which option is chosen in the select dropdown
  • called populate function for the drop down list.
  • within the MakeNewUtterance, set utterance to global voice variable
  • within run(), set yellText to global voice variable

index.html:

  • created a select tag to host the dropdown options of different voices

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2022

Deploy Preview for talktometechnology ready!

Name Link
🔨 Latest commit d6f7152
🔍 Latest deploy log https://app.netlify.com/sites/talktometechnology/deploys/625b9af6e4278f00085d3130
😎 Deploy Preview https://deploy-preview-42--talktometechnology.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jmoldyvan
Copy link
Copy Markdown
Author

jmoldyvan commented Apr 17, 2022

The two most recent commits are the most up to date files.
I can resend the pull request to only contain the 2 proper files.

@laurelayounis
Copy link
Copy Markdown
Member

thank you so much for your commits, there's some conflicts within the code that need to be changed before we're able to merge the updates, @codebymatthew is going to reach out to you to discuss. We really appreciate all of your help.

@jmoldyvan
Copy link
Copy Markdown
Author

any update on this?

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.

2 participants