Design comparison
Solution retrospective
O que você achou difícil ao construir o projeto?
- Eu achei mais difícil a parte dos efeitos de sombra e o gradiente de cores
Quais áreas do seu código você não tem certeza?
- Na parte do responsividade do site
Você tem alguma dúvida sobre as melhores práticas?
- Se tiver melhorias em meu site, poderia me ajudar ?
Community feedback
- @jarleisassiPosted over 1 year ago
Cara o efeito de sombras e gradiente tu consegues atraves dessa ferramenta online : https://www.cssmatic.com/box-shadow Fica mole mole chegar no resultado que queres. Valeu :D
Marked as helpful2 - @rafaeldgeoPosted over 1 year ago
Parabéns Jarlei, ficou ótimo o projeto!!
Algumas sugestões para o projeto: Você poderia usar a propriedade border-radius: 30px que substitui: border-top-right-radius: 30px; border-bottom-right-radius: 30px; border-top-left-radius: 30px; border-bottom-left-radius: 30px;
Quando as bordas forem diferente você pode usar border-radius: 30px 20px 10px 40px (seguindo a ordem top-left, top-right, bottom-right e bottom-left) ou border-radius: 30px 30px (border-top e border-bottom).
Mais informações consulte o W3. Espero que tenha te ajudado.
Marked as helpful1@jarleisassiPosted over 1 year ago@rafaeldgeo Acho que voce marcou a pessoa errada. Nao sou eu o dono do projeto haha
0@rafaeldgeoPosted over 1 year ago@jarleisassi
Desculpe. Errei... estou aprendendo ainda rs
0 - @0xabdulkhaliqPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
BODY MEASUREMENTS 📐:
- The
width: 100vw
property forbody
element is not necessary. because it's a block level element which will take the full width of the page by default.
- Use
min-height: 100vh
forbody
instead ofheight: 100vh
. Setting theheight: 100vh
may result in the component being cut off on smaller screens, such as mobile devices in landscape orientation.
- For example; if we set
height: 100vh
then thebody
will have100vh
height no matter what. Even if the content spans more than100vh
of viewport.
- But if we set
min-height: 100vh
then thebody
will start at100vh
, if the content pushes thebody
beyond100vh
it will continue growing. However if you have content that takes less than100vh
it will still take100vh
in space.
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1 - @NehalSahu8055Posted over 1 year ago
Olá Coder 👋.
Parabéns por concluir o desafio com sucesso! 🎉
Poucas sugestões em relação ao design.
- Substitua
height
pormin-height:100vh
no corpo para centralizar adequadamente o cartão e o plano de fundo. - use
.btn:hover
background: linear-gradient(#6542FE,#342CE2);
➨ Use largura máxima em vez de largura para tornar seu cartão responsivo.
➨ Remover
width:100vw
.- Definir a largura de um elemento
width 100vw
o torna tão largo quanto a viewport atual. Mas fazer isso é uma prática ruim. Primeiro, quase sempre é desnecessário. Um elemento de bloco, por padrão, já ocupa toda a largura disponível. Definir a largura manualmente não é necessário nessa situação.
➨ Você esqueceu de adicionar o
cursor: pointer
; no botão. Será mais fácil de usar para adicioná-lo.- Você pode usar
recursos de acessibilidade
comoaria, sr-only e title
que são acessíveis para leitores de tela.
Eu espero que você ache isto útil.
Feliz codificação😄
Marked as helpful1 - Substitua
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord