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

Feature/fields redesign #865

Open
wants to merge 39 commits into
base: develop
Choose a base branch
from
Open

Feature/fields redesign #865

wants to merge 39 commits into from

Conversation

gcarpi
Copy link
Contributor

@gcarpi gcarpi commented Aug 8, 2023

BREAKING CHANGES

  • Tecnicamente nenhuma alteração realizada nos campos de formulário é uma breaking change, porém, devido as mudanças de Design System realizadas com a nova estilização dos campos e com a adição do QasBox no QasFormGenerator poderá haver quebras visuais. É importante após a atualização dessa versão verificar todos os formulários da aplicação e caso haja alguma quebra, verificar com o time de design como proceder.

Adicionado

  • QasBox: adicionado propriedades paddingHorizontal, paddingVertical e usePadding para controlar o espaçamento interno do componente.
  • QasBox: adicionado provider isBox com o valor true para ser utilizado por componentes filhos para saber se estão dentro de um QasBox.
  • QasInput: adicionado propriedade dense com o valor default true.
  • QasInput: adicionado suporte ao atributo counter do QInput. Por padrão, campos do tipo textarea e com maxlength maior que 0 irão exibir o contador de caracteres.
  • QasInput: adicionado atributo autogrow do QInput para campos do tipo textarea.
  • [QasField, QasInput, QasNumericInput, QasDateTimeInput]: adicionado atributo placeholder com o valor default definido pelo tipo do campo.
  • QasFormGenerator: adicionado propriedade useBox para controlar se os fieldsets do formulário serão renderizados dentro de um QasBox.
  • QasSelect: adicionado por padrão o atributo useChips com o valor true para select múltiplos.
  • QasSelect: adicionado por padrão o atributo dense com o valor true.
  • QasDialog: adicionado provider isDialog com o valor true para ser utilizado por componentes filhos para saber se estão dentro de um QasDialog.
  • QasNestedFields: adicionado provider isNestedFields com o valor true para ser utilizado por componentes filhos para saber se estão dentro de um QasNestedFields.
  • [checkbox.scss, editor.scss, field.scss, radio.scss, toggle.scss]: adicionado novos estilos de acordo com o Design System.
  • typography.scss: adicionado nova tipografia $label para ser utilizada nas labels de componentes de input.
  • QasTreeForm: adicionado propriedade use-box com o valor false para não renderizar o formulário dentro de um QasBox.

Modificado

  • QasBox: modificado para composition API.
  • QasBox: modificado estilo de um QasBox dentro de outro QasBox, o box interno se comportará como um QasBox do tipo outlined.
  • QasInput: modificado valor da propriedade outlined para false por padrão.
  • [QasCheckboxGroup, QasOptionGroup]: modificado espaçamentos entre opções.
  • QasListItems: modificado espaçamentos entre os itens e do box.
  • QasSelect: modificado dropdownIcon e clearIcon para utilizar ícones do Material Symbols Rounded.
  • QasSelect: modificado estilos do componente de acordo com o Design System.
  • QasTableGenerator: modificado espaçamentos do componente QasBox.
  • QasNestedFields: modificado espaçamentos do componente.
  • set-typography.scss: modificado mixin para permitir o atributo color.

Versão do asteroid

  • v2 -> a partir da branch v2.
  • v3-beta.x -> a partir da branch develop.
  • v3-stable -> a partir da branch main.

Tipo de alteração

  • Adicionado | Added (novos componentes e/ou funcionalidades);
  • Modificado | Changed (alterações que podem ou não conter breaking changes);
  • Corrigido | Fixed (correção de bugs, typos, etc);
  • Removido | removed (remoção de algum componente e/ou funcionalidade).

O que foi alterado/adicionado

  • CSS
  • Componentes
  • Composables (v3)
  • Diretivas
  • Documentação
  • Helpers
  • Mixins
  • Paginas
  • Plugins
  • Testes
  • Outros

Este pull request introduz algum breaking change?

  • Sim
  • Não

Checklist

  • Foi discutida anteriormente com os times de Frontend e Design;
  • Foi testado manualmente no ambiente de desenvolvimento (/docs se v3 ou ui/dev se v2);
  • Foi constatado que esta modificação não gerou erros ou alertas no Console;
  • Foi verificado se o código segue os padrões de escrita e validado com o ESLint;
  • Foi escrito teste automatizado;
  • Foi atualizada e testada a documentação;
  • Foi atualizado o changelog seguindo o padrão "Keep a Changelog";
  • Fiz meu próprio code review antes de abrir este pull request.

@vercel
Copy link

vercel bot commented Aug 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
asteroid ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2023 1:16pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
asteroid-v2 ⬜️ Ignored (Inspect) Visit Preview Oct 6, 2023 1:16pm

Comment on lines +187 to +190
pre {
white-space: normal;
word-wrap: break-word;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Qual motivo dessa alteração?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screenshot 2023-09-22 at 15 31 47

Conteúdos dentro do pre não quebram, eu coloquei um break-word para começar a quebrar.

ui/src/components/box/QasBox.vue Outdated Show resolved Hide resolved
Comment on lines +20 to +30
paddingHorizontal: {
default: Spacing.Md,
type: String,
validator: value => Object.values(Spacing).includes(value)
},

paddingVertical: {
default: Spacing.Md,
type: String,
validator: value => Object.values(Spacing).includes(value)
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Será que a nomenclatura spacing não seria mais adequada que padding? O que acha?

Copy link
Contributor Author

@gcarpi gcarpi Sep 22, 2023

Choose a reason for hiding this comment

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

Hmm, eu não sei, eu também fiquei pensando nisso...

No componente QasLabel a prop de espaçamento utilizada é margin, bem específica exatamente como pensei em fazer aqui também.

Pensando se futuramente o QasBox também ter uma prop de margem, ficaria um pouco confuso um spacing controlando o padding e um margin controlando a margem. O que acha?

ui/src/components/box/QasBox.vue Outdated Show resolved Hide resolved
ui/src/components/box/QasBox.vue Outdated Show resolved Hide resolved
line-height: 140%,
letter-spacing: 0,
weight: 400,
color: $grey-8
Copy link
Collaborator

Choose a reason for hiding this comment

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

Label são "titulos" de uma seção, não seria grey-9?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ui/src/css/variables/typography.scss Outdated Show resolved Hide resolved
ui/src/css/variables/typography.scss Outdated Show resolved Hide resolved
@@ -0,0 +1,32 @@
export default function (type, defaultPlaceholder = 'Digite aqui...') {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Eu acho que o nome da função faz mais sentido se for getPlaceholderByMask

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Será? Pois pode ser por mask ou por type...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screenshot 2023-09-25 at 11 36 14

ui/src/helpers/get-placeholder.js Outdated Show resolved Hide resolved
@gcarpi
Copy link
Contributor Author

gcarpi commented Sep 4, 2023

Reunião time de design 04/09/2023

  • Tag do multiselect no QasFilters não está sendo removida
  • Alterar font-weight das opções do QasSelect para 400
  • Alterar cor do texto de observações para $grey-8

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