Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Basic Improvements UI/ UX wanted #208

Open
Chinyeins opened this issue Nov 15, 2023 · 1 comment
Open

Basic Improvements UI/ UX wanted #208

Chinyeins opened this issue Nov 15, 2023 · 1 comment

Comments

@Chinyeins
Copy link

Chinyeins commented Nov 15, 2023

Hey, I love your extension, it is so usefull. Id like to share some improvements that might be interesting to other people using the extension. Unfortunately people that require changes, probably just fork the project and impl. the changes themselves, but I think these changes could benefit greatly in general for this project:

  1. I find the settings page is not very readable and i think the font and structure makes it hard to navigate
    image
  • one single set of save, cancel
  • better readability for text (change font and or make bigger)
  • seperate info from actual settings, since its confusing
  • removed background tansparency, since it reduces readability
  1. The API Key for the ElevenLabs should be hidden by default i think, since it´s a secret, I think it´s better practice to auto hide it when openeing the settings page.
    image

  2. depending on your setup the volume of tts can be quite loud, so a slider to change tts volume would be great
    image

  3. A way of replaying messages via TTS feature would be very useful, since currently after asking something, TTS will kick in and read the message generated, but cant be replayed.

  4. Technical improvements, im curious why the markup and code are mixed and most of the code is in one big god class instead of trying to split features to extra classes. Theres a lot of room for improvements on the code side by extracting the markup to seperate file and referecing in code and extracting features to dedicated files / classes. But that sprobably a big one

  5. Deactivate initialization sound after pressin start button. There should be an option to deactivate the sound in settings page, that also gets saved into settings cookie.

  6. refactoring settings cookie to make us of chromes storage api. The reason for why this could be usefull is, you could store / sync settings for this extension accross browsers. API Key should be excluded from that I think.

For some of my concerns i already impl. at least a suggestions of how it can look like: you can find it unter this PR
Pull Request, suggested changes: #205

If you like to test the changes quickly, heres my fork for the changes i made so far:
https://github.com/Chinyeins/talk-to-chatgpt-brainstuck

Love your work

@C-Nedelcu
Copy link
Owner

C-Nedelcu commented Nov 16, 2023 via email

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

No branches or pull requests

2 participants