search

C-Nedelcu / talk-to-chatgpt

Talk to ChatGPT AI using your voice and listen to its answers through a voice
GNU Affero General Public License v3.0
1.96k stars 335 forks source link

Code enhancement #202

Open justincontributor opened 8 months ago

justincontributor commented 8 months ago

Hello, Thanks for this great extension.

function CN_SendMessage

rather than the hazardous : jQuery("#prompt-textarea").closest("div").find("button:last")

I propose : jQuery('[data-testid="send-button"]')

C-Nedelcu commented 7 months ago

Thanks - I will look into it. Is there any particular reason you are suggesting this?

justincontributor commented 7 months ago

Hi, Yes, the reason is that there are changes with the buttons from time to time. Getting the button by ":last" could fail one day if another button comes up with a new chatpt option. With that, you would be more sure that you enable or click the good one.

It's not (yet) a bug. No hurry.