Feature/edit community contacts #740
Conversation
✅ Deploy Preview for cambiatus-elm-book ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
working good, I think the label is too small for certain screens, not really readable, but as far as functionality goes, it is getting awesome! |
|
I adjusted the size of the form inputs. I'd say the idea is that the labels should be short and descriptive, and I think the small form input encourages that. On the other hand, it might be frustratingly small 😬 |
|
Maybe @juramos-2020 can help us out here ☝️ |
There was a problem hiding this comment.
Oh, forgot to approve the PR. @juramos-2020 I think we could have a minimum font size to make readability possible.
@lucca65 @NeoVier de qual fonte vocês estão falando? em qual componente? |
|
@NeoVier quanto de largura vc está usando nesse board branco? Podemos usar 810px conforme layout |
|
@NeoVier ⚠Teria como termos aqueles espaçamentos automáticos quando a gente preenche um campo de telefone? ⚠Uma outra coisa que aconteceu foi que, ao terminar de digitar o e-mail eu cliquei no "ENTER", como de costume, e apareceu esse alerta de erro.
|
|
⚠E agora estou tentando salvar o formulário com os campos preenchidos corretamente mas não está salvando. Está dando erro.
|
|
⚠ Quando eu adiciono uma opção nova de conta ela já abre com um alerta de erro:
|
Ele está alinhado com o resto do conteúdo do app. Eu deixei ele maior para termos mais espaço para os campos de label, pois com 810px não tem espaço para colocar labels maiores (inclusive também aumentei o tamanho dos campos). Podemos alterar se quiser, mas acredito que seria bom termos mais espaço para a label
Pode ser que a comunidade tinha contatos, mas decidiu apagá-los. Nesse caso, tem que ter a opção de salvar os contatos como uma lista vazia
Sei sim, isso seria a máscara 😅. O problema é que, no outro form de contatos (que usamos no perfil), o usuário escolhe o país, e com isso conseguimos saber qual máscara aplicar (padrão brasileiro, Costa Rica, etc.). Nesse form, temos apenas um campo de texto, sem o seletor de país. Como cada país tem seu padrão de número, temos que saber o país para poder aplicar a máscara. Podemos tentar usar o código de país (+55, +1, etc.) pra mudar a máscara enquanto o usuário digita, mas isso pode causar alguns problemas com o cursor de texto. Ou então podemos aplicar a máscara somente quando o usuário termina de digitar, e vai para outro campo. Vou investigar os outros erros 🏃♂️ |
|
OI @NeoVier
Qual seria o tamanho máximo que vc acha que podemos ter para uma label? Podemos falar melhor sobre isso aqui no figma? acho que fica melhor para eu entender e poder sugerir algo melhor. |
|
@NeoVier segue a versão desk atualizada. Sobre a fonte dos rótulos dos icones sugiro seguirmos com essas especificações: Acabei de fazer o teste na versão mobile e funcionou tudo como o esperado 🎉 |
What issue does this PR close
Closes #713
Changes Proposed ( a list of new changes introduced by this PR)
To make things better with our current architecture, I created a new
Contactmodule to organize the contact form and related types, functions and views. That means we now haveContactandProfile.Contact, whereContactis used for community contacts, andProfile.Contactis used for profile contacts. In order to not clutter this PR too much, I will leave the work of removingProfile.Contactin favor ofContactfor another issue/PRHow to test ( a list of instructions on how to test this PR)