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

Fix: correção de componente Text, Input e Button #37

Merged
merged 4 commits into from
Mar 9, 2023

Conversation

YaGRRusso
Copy link
Collaborator

@YaGRRusso YaGRRusso commented Mar 9, 2023

Componentes afetados

Text

Input.Icon
Input.Root

Button.Root

Descrição

Algumas pequenas correções e adições de algumas props em componentes.

Input.Icon - Adicionado o type='button' para evitar o submit de formulários ao ciclar
Input.Root - Adicionado à variant status a opção 'default'

Button.Root - Adicionado à variant full a opção false, tornando o width 'fit-content' evitando que o botão ocupe toda a tela dentro de um display flex-col

Text - Adicionar o estilo font-size 'inherit' para seguir o tamanho de fonte de acordo com o container e evitar o padrão css de 1rem independente do container
Text - Adicionada a prop 'inline' que permite colocar ícones alinhados ao lado de textos dentro da tag Text (beta)

@github-actions github-actions bot added the bugfix Something isn't working label Mar 9, 2023
@leandroppf
Copy link
Contributor

Tem que mudar o nome do PR, segue o padrão Fix: (nome descritivo)
Na descrição descreve as correções, mexendo com lib a gente vai deixar tudo claro para o changelog ficar certinho

Copy link
Contributor

@leandroppf leandroppf left a comment

Choose a reason for hiding this comment

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

Realizando as modificações do comentário anterior o resto ta ok

@YaGRRusso YaGRRusso changed the title Fix small fixes Fix: correção de componente Text, Input e Button Mar 9, 2023
@YaGRRusso YaGRRusso merged commit aee1e11 into main Mar 9, 2023
@YaGRRusso YaGRRusso deleted the fix-small-fixes branch March 9, 2023 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants