Skip to content

Conversation

@ViniciusCorbelli
Copy link
Collaborator

Primeira versão do chat, ja foi feito utilizando banco de dados para evitar retrabalho, além disso utilizando apenas javascript não seria possível fazer questões de segurança dos usuarios so mandarem mensagem para a ontologia que possui permissão.
O chat ficou em cima do console, não encontrei um lugar melhor para posicionar, então aceito sugestão.

@Piazzi Piazzi added the enhancement New feature or request label Jan 21, 2022
@Piazzi Piazzi added this to the Onto4ALL 4.2 milestone Jan 21, 2022
@Piazzi
Copy link
Owner

Piazzi commented Jan 21, 2022

Oi Vinicius, fez bem em fazer as tabelas do BD de uma vez, economizou tempo. Sobre a questão da interface, pode deixar que eu faço isso depois do merge.

Algumas considerações que gostaria que você resolvesse antes de eu dar o merge:

  • O chat só deve ficar habilitado quando a ontologia atual estiver no banco de dados, ou seja, quando ele entra no editor, o chat nao pode estar habilitado, só quando ele carregar uma ontologia, ou entao salvar. Recomendo que você deixa em cinza o chat quando ele estiver desabilitado, e coloque um ícone "?" no header do chat, isso servirá de tooltip para o usuário. Nessa tooltip, explique a funcionalidade do chat pf, tanto em inglês quanto português.

  • Quando você mandar mensagem pros usuários, lembre de sempre fazer a mensagem em português e em inglês. Você pode pegar a linguagem atual do usuário utilizando o getLanguage() em JS ou pegar da própria URL no php

  • O chat deve começar colapsado assim como o console de avisos

@Piazzi Piazzi merged commit 0fe4511 into dev Jan 22, 2022
@Piazzi
Copy link
Owner

Piazzi commented Jan 22, 2022

Muito bom @ViniciusCorbelli

@Piazzi
Copy link
Owner

Piazzi commented Jan 22, 2022

@ViniciusCorbelli , uma dica, evite usar position: fixed / absolute . Geralmente quando você faz isso tem chances grandes de quebrar a responsividade ou o layout da página, quando tiver duvida em como posicionar os elementos pode me chamar que te ajudo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Desativar socket.io localmente Chat em tempo real entre os colaboradores de uma ontologia

3 participants