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

Altera campo gênero para texto livre. #284

Closed
wants to merge 4 commits into from

Conversation

yrachid
Copy link

@yrachid yrachid commented Aug 13, 2018

Requisitos

Descrição da Mudança

Esta PR introduz as seguintes modificações:

Execução de módulos executáveis via npx

Algumas tarefas do projeto dentro do package.json eram executadas acessando node_modules através do seu caminho relativo (../node_modules). O uso do caminho relativo não garante uma execução que funcione entre diferentes ambientes (as tarefas falharam no meu ambiente local, por exemplo).

As versões mais recentes do npm (incluindo a que se utiliza no projeto) oferecem algumas alternativas para resolver este problema:

  • npm bin: retorna o caminho absoluto de onde dos módulos executáveis do projeto estão instalados
  • npx: executa um módulo executável

Optei pelo npx por deixar as tarefas mais legíveis dentro do package.json. Usando o npm bin teríamos algo como:

{
  "scripts": {
    "test": "$(npm bin)/jest"
  }
}

Outro ponto que pode ser um problema é o fato de o projeto usar o yarn em vez de npm diretamente. Aparentemente, o Yarn ainda não oferece uma solução parecida com npx. Acredito que se o CI passar com o uso do npx, não devem haver maiores problemas quanto ao uso do npx.

Inversão da ordem entre labels e campos em alguns componentes

Por alguma razão, em algumas situações, os navegadores estavam renderizando o label abaixo do campo (em vez de renderizá-lo em cima, como nos demais), o que pode ser visto nas capturas de tela abaixo:

Chrome:
enegrecer-label-chrome

Firefox:
enegrecer-label-firefox

Ao inverter a ordem do label dentro do componente, o problema foi resolvido.

O campo gênero passa a ser um campo de texto livre

O campo gênero deixou de ser um campo de seleção de opções fixas (ComboBox) para tornar-se um campo de texto livre, permitindo que a pessoa se identifique da maneira em que sentir-se mais confortável (ou deixe o campo em branco, caso não queira identificar seu gênero).

Benefícios

Uma pessoa usuária terá maior liberdade de escolher como descrever seu gênero, tendo até mesmo a opção de não descrevê-lo caso não se sinta confortável.

Possíveis Desvantagens

Não consegui verificar se a mudança teria algum impacto no backend da aplicação. Acredito que o CI possa ajudar a prover algum feedback em relação à isso.

Issues relacionados

Não se aplica.

Abordagens Alternativas

Se houver sentido, poderia existir um componente que seja uma caixa de seleção de opções fixas (como o componente original) mas que permita, caso a pessoa deseje, introduzir texto livre para descrever o gênero.

@yrachid yrachid changed the title Development Alter campo gênero para texto livre. Aug 13, 2018
@yrachid yrachid changed the title Alter campo gênero para texto livre. Altera campo gênero para texto livre. Aug 13, 2018
Copy link
Collaborator

@nayara nayara left a comment

Choose a reason for hiding this comment

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

Precisamos conversar mais sobre deixar o campo de seleção de gênero livre.

@nayara nayara closed this Oct 17, 2018
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