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

Eliminate auto capitalization of thread names #1174

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gm3197
Copy link

@gm3197 gm3197 commented Jul 29, 2024

Removing auto-capitalization of thread names because it is not necessary and forces this design decision on developers.

@dokterbob
Copy link
Collaborator

Hi @gm3197, thanks for your patch!

As it stands, applying it would unexpectedly change the UX for pre-existing users, while we do not (yet) have a clear-cut/well-documented way of directly controlling thread names. If there were, developers can just implement capitalisation on their own and we could document this as a breaking change, giving an example of how to do this.

Is there any chance you'd be able to have a look at this ticket? #1097

With a clear and sensible API to be able to control chat titles (in a way where it's not glitching, e.g. changing erratically), merging your patch would be a no-brainer.

Curious to hear what @willydouhard images here.

@dokterbob dokterbob added the frontend Pertains to the frontend. label Aug 22, 2024
@dokterbob dokterbob added debate Under open debate. enhancement New feature or request labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debate Under open debate. enhancement New feature or request frontend Pertains to the frontend.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants