-
Notifications
You must be signed in to change notification settings - Fork 45
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
Size Selector #79
Size Selector #79
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O nome da pasta não está bom (está em português). A pasta de imagem não está no padrão das outras.
O componente está bem limitado e engessado, não consigo adicionar ou remover itens. Os nomes dos métodos não estão bons e não consigo reagir a alteração dos estados.
Não consigo obter o valor atual programaticamente.
Eu estou deixando o CSS com o @Alecell mas tem muita coisa errada, como os breakpoints não padronizados e a estilização por tag
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preciso que preencha corretamente o template da PR, aquela primeira etapa que ta com uns N/A, essas coisas tem que estar preenchidas de acordo!
Faltou criar o storybook do componente para documentação, da uma olhada nos arquivos .story
que temos no projeto, pra visualiza-los usa pnpm storybook
!
O que exatamente não tá no padrão na pasta de imagem? o nome das img ? |
Estamos padronizando como "images" e não "img" |
}); | ||
} | ||
|
||
export default function SizeSelector({ slide }) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export default function SizeSelector({ slide }) { | |
export default function SizeSelector({ slide } = {}) { |
}); | ||
} | ||
|
||
export default function SizeSelector({ slide }) { | ||
Component.call(this, { html, events }); | ||
|
||
const slideItems = this.selected.get(slide); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Essa seleção pelo data-select está muito estranha pra mim, o data-select não foi pensando pra isso
@Alecell
} | ||
}, | ||
}, | ||
{}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{}, |
</div> | ||
</body> | ||
</html> | ||
</html> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não entendi por que esse arquivo foi alterado
E aqui está sem o EOF
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E ae man, seria interessante um stories do componente ein, pra testar e etc
Task vai ser resolvida no PR #265 |
Closes #23
Feature
Criação do componente Seletor de Porte, esse componente tem três opções para o usuário descrever o tamanho do pet dele
Bugfix
Description
N/A
Cause
N/A
Solution
N/A
Changelog
N/AVisual evidences 🖼️
document_4936182222253196287.mp4
Checklist
Additional info
N/A