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

Modal of active card #24

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Modal of active card #24

wants to merge 2 commits into from

Conversation

devmarcosanjos
Copy link
Collaborator

No description provided.

Copy link
Member

@rafaelcn rafaelcn left a comment

Choose a reason for hiding this comment

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

Atente-se as mudanças pedidas e aguarde o meu outro feedback pois vou testar todas essas modificações aqui.

@@ -1,47 +1,282 @@
{% extends "default_internal.twig" %}

{% block content %}
<script src="https://cdn.plot.ly/plotly-latest.min.js"></script>
Copy link
Member

Choose a reason for hiding this comment

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

Não faça inline include dos scripts. Ao invés disso, inclua-os no modelo da página.

@@ -1,47 +1,282 @@
{% extends "default_internal.twig" %}

{% block content %}
<script src="https://cdn.plot.ly/plotly-latest.min.js"></script>

<div class="container-fluid">
<div class="row align-items-center">
<div class="col-sm">
<div id="myDiv"><!-- Plotly chart will be drawn inside this DIV --></div>
Copy link
Member

Choose a reason for hiding this comment

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

Escolha um ID condizente com o conteúdo do gráfico. Não simplesmente copie e cole isso dentro do div.

@rafaelcn
Copy link
Member

rafaelcn commented Jul 9, 2019

@marcosanjos o que você acha que está errado nestas seguintes imagens?

image

image

image

Acredito que você não entenda do mercado e por isso está utilizando uma terminologia incorreta, além da disposição de alguns elementos de maneira incorreta.

O valor do ativo é um valor fixo no nosso caso e deve ser somente colocado num campo em modo "disabled", cujo valor não será enviado ao servidor no momento da compra. Outra coisa é que somente números poderão ser inseridos nestes campos e para facilitar nossa vida simplesmente utilize o tipo number.

Sobre a disposição dos elementos, a primeira coisa que tenho a citar é a indentação do código incorreta. Atente-se a esse parâmetro pois é importante para a aceitação do código. Os botões da primeira imagem estão equivocadamente dispostos, para o cliente uma disposição clara e de comportamentos separados é ideal.

O botão fechar deve ser colocado de maneira sutil (btn-light) e à direita de todos os botões e portanto deve ser separado como a figura ASCII abaixo:

}---Comprar--Vender---------------------------------------------------Fechar---{

Essa disposição deve ser repetida em outros modais criados na página da carteira do usuário. Há um erro também no título do modal de venda de ativo, onde pode ser visto o título Compra de ativo, é um erro crasso cara e fácil de ser visto. Você deve ser mais atencioso.

O texto de detalhes da empresa deve ser posto de modo justificado e o símbolo deve ser colocado em destaque sem nenhum texto que o precede. No campo de venda deve ser também ser colocado em modo desabilitado a quantidade de dinheiro aplicada pois não se pode alterar esse campo.

Além disso, você imaginou como será realizado o retorno de dados do servidor ao cliente? Vários modais ou vários dados recebidos em javascript e onde cada clique irá ativar uma função para colocar os dados no modal criado?

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