Darwin Junior
@djdeveloper22All comments
- @Kvngjakes1Submitted 5 months ago@djdeveloper22Posted 5 months ago
Good job friend, but I feel that you complicated your existence, using the position property is a solution, although you can omit it and only with grid you achieve the desired design. I share with you 2 repositories where you can guide yourself and study how to improve the structure that you propose: https https://www.frontendmentor.io/solutions/four-card-feature-with-grid-SimWakGw99, and this is my solution: https://www.frontendmentor.io/solutions/web-site-using-css-4ughGXhL2y ....... :)
0 - @sergrosuSubmitted 5 months ago@djdeveloper22Posted 5 months ago
Visually it looks good, but you can improve it much more, an image is broken on the second card and it is not aligned either,
Marked as helpful0 - @Stephanie0905Submitted 5 months ago@djdeveloper22Posted 5 months ago
Great!, I loved what you did, I like the way you gave a solution with the grid-template-areas property, in my challenge I did not apply it, it would have been easier for me :)
Marked as helpful1 - @yas-avocadSubmitted 5 months agoWhat challenges did you encounter, and how did you overcome them?
I'm having problems with the smooth transition between screen sizes. With the absolute image, the text on the side runs together with the hero.
What specific areas of your project would you like help with?How to make the main text and image hero not run into each other when changing the browser size.
@djdeveloper22Posted 5 months agoExcellent work, I liked what you did, in mobile view I would add the object-fit: contain property to the logo so that the image is not distorted
Marked as helpful0 - @khalagaiSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
Getting the responsive layout using different images either side of the breakpoint.
What challenges did you encounter, and how did you overcome them?Using different images in the different layouts. I included both images in the HTML code and used the display: none property in the different screen sizes.
What specific areas of your project would you like help with?Improvements I could make as well as see how other members tackled the project.
@djdeveloper22Posted 5 months agoGreat, I like what you did, it can be improved much more, greetings
1 - @Dev-FarhanSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
.
What challenges did you encounter, and how did you overcome them?.
What specific areas of your project would you like help with?.
- @antonbest444Submitted 5 months ago@djdeveloper22Posted 5 months ago
Visually it is pleasant, things to check is the container that has a height not similar to that of the original design but good job
0 - @AllanCKSubmitted 5 months ago@djdeveloper22Posted 5 months ago
At the body level, in general I would add the margin and padding properties all to 0 through the property or universal selector * to format the styles and remove the white, if where you have the user image with the name you apply flex you can separate them with the gap property, but with margin left or rigth, for my part I would give it a smaller width so that the card looks more subtle. The rest seems great to me. excuse my bad english
Marked as helpful1 - @Mahrukh-AdeelSubmitted 5 months ago@djdeveloper22Posted 5 months ago
First of all, sorry for my bad English, I will try to help myself with technology to give a good message. Excellent! I like your solution, visually it is very similar to the current design, I like the shading of the card, the title was a little far from the card but overall perfect.
0 - @KeylaLinsSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
Fiquei feliz com a velocidade de ter feito, muito mais rápido que os últimos exercícios.
What challenges did you encounter, and how did you overcome them?Tive um pouco de problema na imagem, pois na hora de baixar os arquivos, a imagem não veio junto.
What specific areas of your project would you like help with?Uma revisão no código por completo.