Implementar EmptyView #46
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Adicionando componentes da EmptyView e adicionando live preview para essa view.
Adicionando parâmetros de inicialização para EmptyView de forma dinâmica.
Adicionando teste de Snapshot para emptyView e stub da emptyView para criar um padrão de testes.
Contributor
FernandoVGMonteiro left a comment
FernandoVGMonteiro left a comment There was a problem hiding this comment.
Muito bom mano!
Código bem limpo e gostei que você deixou o texto flexível pra conseguir usar em outras telas!
Uma coisa só que eu ia comentar é pra trocar as constraints que estão na label. Acho que faz mais sentido deixar a constraint na própria UIStackView e sem constraints no conteúdo dela!
Eu expliquei sobre isso na sessão de ontem e usei esse código também como exemplo pra falar um pouco também de responsividade! Assim que sair o vídeo da sessão, dá uma olhada lá e vê se faz sentido alterar alguma coisa, aí me dá um toque por aqui :)
Ajuste nas constraints da stackView da EmptyView.
Contributor Author
| Boaa muito obrigado, fiz os ajustes nas constraints da stackView e ja subi as alterações. |
FernandoVGMonteiro approved these changes Dec 9, 2022
Contributor
FernandoVGMonteiro left a comment
FernandoVGMonteiro left a comment There was a problem hiding this comment.
Boa! É isso aí :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Descrição e Solução
Checklist:
Evidências: