@devEduSousa
Submitted
@ArryBlack
@devEduSousa
Submitted
@ArryBlack
Posted
Hi, I like the approach you have taken of using a single HTML page, I was thinking of a similar approach, but ended up opting for a separate success page. The site looks great, great job on the challenge!
Here are a few things I would like to point out that can improve your solution:
Once again, a great job on the challenge! If you get the time, please check and provide some feedback on my solution for the same challenge. Happy coding!
@delroscol98
Submitted
What are you most proud of, and what would you do differently next time?
I'm super proud of the CSS, I got it as close as I could to the design. As stated in the README, it's really important that as project become more complex to assign appropriate classes to elements based on their purpose. Also to identify the pattern of certain components in their styling. This would produce a personal CSS library which would be pretty cool.
What challenges did you encounter, and how did you overcome them?
Definitely the construction of the tooltip. This was my first time making one. I was tempted to use a CSS library like Bootstrap, but I wanted to build it from scratch first to understand the design and logic.
What specific areas of your project would you like help with?
Readability of the HTML file and the assignment of class names to elements. As the class names link directly to the CSS file, I want to make sure that my assignment of class names and the organisation of my CSS code is the best it can be.
@ArryBlack
Posted
@delroscol98 Great job man! Really concise code. I solved this challenge just now and your solution gave me an idea of how to improve it. Btw, if I can provide any feedback, on smaller screens from around 500px to 767px, the share button is overlapping with the social media links. You can maybe try adding a margin.
@mkhantk
Submitted
What are you most proud of, and what would you do differently next time?
I'm proud of doing most of the work done by myself. I'll try to improve to handle this kind of work single-handedly.
What challenges did you encounter, and how did you overcome them?
I did encounter my html body only taking half of the height and I got to google how to fix it. Only then, I was able to start with the flexbox.
What specific areas of your project would you like help with?
I would like to ask seniors for feedbacks. My code is a mess as it seems to be. So... How can I make it better overall? I appreciate it, thanks.
@ArryBlack
Posted
Hi @mkhantk, great job! The code seems good. You can try making the wrapper div into a main tag for more semantic markup.
Marked as helpful
@LorenzoSerra1
Submitted
What are you most proud of, and what would you do differently next time?
I realize that I'm getting better at running and writing code in terms of speed
What challenges did you encounter, and how did you overcome them?
The biggest challenge was finishing the project by perfecting the details
What specific areas of your project would you like help with?
I would like to improve on the latest revisions of the project, to understand if my work is valid or not
@ArryBlack
Posted
Layout looks great.
Btw, I solved the same challenge recently, any feedback on it will be appreciated, thank you!
Marked as helpful
@Mtrxxo
Submitted
What are you most proud of, and what would you do differently next time?
I would really appreciate it if you all could help me view the page and give me feedbacks.
What challenges did you encounter, and how did you overcome them?
I encountered a challenge with the sizing of the section.
@ArryBlack
Posted
Hi Ibrahim, great work on the layout. I've a few suggestions.
Marked as helpful
@evsta24
Submitted
What are you most proud of, and what would you do differently next time?
I am proud, that I used element box-shadow and object-fit in css file.
@ArryBlack
Posted
Looks great, you can try adding a transition property for the box-shadow. Maybe 300ms.