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

Contador de caracteres e mensagem de erro ao ultrapassar limite dos campos #40

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Contador de caracteres e mensagem de erro ao ultrapassar limite dos campos #40

wants to merge 4 commits into from

Conversation

Andre-Sacilotti
Copy link
Contributor

Tentei fazer uma coisinha aqui, espero q seja util

O banco de dados limita o tamanho do nome e descrição da tarefa, ou seja, quando a descrição tinha mais que 300 caracteres estava retornando 500, então eu fiz um contador de caracteres e uma mensagem de erro, impedindo a criação da tarefa.

Quando o nome da tarefa ultrapassa os 80 caracteres:
Erro no nome
Descrição da imagem: A imagem representa o formulario para criação da tarefa, onde é exibido uma mensagem de erro avisando que o campo nome tem mais que 80 caracteres

Quando a descrição da tarefa ultrapassa os 300 caracteres:
Erro na descrição
Descrição da imagem: A imagem representa o formulario para criação da tarefa, onde é exibido uma mensagem de erro avisando que o campo descrição tem mais que 300 caracteres

Copy link
Owner

@dunossauro dunossauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O comprotamento descrito no BDD não foi implementado, os steps não foram implementados.

Sobre o texto nas mensagens, achei que trouxe bastante complexidade no código, if elif elif.

A minha ideia era usar um toast para fazer essas notificações

https://www.w3schools.com/bootstrap4/bootstrap_toast.asp.

Assim não ficaria tão complicada a leitura. Pois fica confuso de ler no quadradinho pequeno de erro.

@dunossauro
Copy link
Owner

Sobre o toast temos a issue #29 criada

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