Skip to content

Create template, description has a character limit of 128, but doesn't show error immediately and shows no counter #11012

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

Closed
mafredri opened this issue Dec 4, 2023 · 1 comment · Fixed by #11558
Assignees
Labels
s3 Bugs that confuse, annoy, or are purely cosmetic site Area: frontend dashboard

Comments

@mafredri
Copy link
Member

mafredri commented Dec 4, 2023

Currently when creating a template, the description has a 128 character limit, but there are two issues.

  1. When you focus the "Description" input for the first time, you can type >128 characters and the input box will not turn red until you de-focus the input box
  2. Since I can't see how many total characters I have, it's hard to plan how to fix it

Before de-focus (typing out 927 chars):

image

After de-focus:

image

User question: When did I exceed it?

After initial focus/defocus, the error will show up in real-time, but that also requires looking at the screen to figure out when you exceed it.

Suggested fixes:

  1. Trigger errors even on initial focus
  2. Show counter (64/128, 142/128) all the time so that the user can more easily resolve the issue
@mafredri mafredri added site Area: frontend dashboard bug labels Dec 4, 2023
@matifali matifali added the s3 Bugs that confuse, annoy, or are purely cosmetic label Dec 4, 2023
@MrPeacockNLB
Copy link
Contributor

as a side note: Why is this value so restricted? Maybe a customer want to write a more detailed description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s3 Bugs that confuse, annoy, or are purely cosmetic site Area: frontend dashboard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants