Jose
@josenegrete123All comments
- @Hlm19966Submitted 8 days ago@josenegrete123Posted 8 days ago
Great job on this challenge. A tip I could offer, is to try and write classes for multiple
HTML
elements so you won't have to rewrite code over and over again, instead you can detail a card for all 5 of the objects. Then 1 by 1 change any elements that need changing, instead of rewriting multipleCSS
codes over and over again.For example, you have the
card
class, so instead of rewriting thebox-shadow
,border-radius
,padding
, etc. you can just write all of that into the.card
class inCSS
.Marked as helpful0 - @tobaojoSubmitted 9 days agoWhat are you most proud of, and what would you do differently next time?
Using CSS grid
@josenegrete123Posted 9 days agoGreat job, the designs are near identical.
The only thing I would personally change would be the
font-size
around the paragraph in the header. It's a really easy change though so no stress there. Amazing work!0 - @rafirachmawanSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
learn more about responsive and improve my knowledge in the frontend field
What challenges did you encounter, and how did you overcome them?the challenges I face are a little more difficult on the layout of text images and responsiveness
What specific areas of your project would you like help with?in terms of distance and layout because I didn't get the figma design
@josenegrete123Posted about 1 month agoGreat work! For your
@media
section of the CSS, you don't need to change thefont-size
around because per the figma documents, they are all the samefont-size
, from mobile-version to desktop. Also theh1
and the firstspan
should be in a different font. But that was given in the style-guide. Theline-height
in theh1
could also be change to0
.Marked as helpful0 - @praistylesSubmitted 3 months ago@josenegrete123Posted 3 months ago
You could have added the span html element to the instructions list to create that bold lettering on the first few words. For example:
<span>Beat the eggs</span>
to create Beat the eggs. Besides that, the rest looks good. Maybe just increase the size to better match the design.Marked as helpful0 - @ShubhamDRajSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Adding hover and active state animations to the buttons.
@josenegrete123Posted 3 months agoGreat job with your work! Instead of using a grid to display the items, I would personally use a flexbox. The reason being that grids are mainly used to display data. But besides that everything here looks great!
0 - @Guitar8634Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am most proud of learning how to use the Figma File to create a more accurate representation of the design.
What challenges did you encounter, and how did you overcome them?Some challenges included figuring out how to justify and align all the elements within the card correctly. I solved these using
display: grid; justify-content: center; align-items: center;
What specific areas of your project would you like help with?position: relative;
I am curios how other people solved the indenting of the title and paragraph in the hover state.
@josenegrete123Posted 3 months agoFor the indenting in the hover state, use the :hover pseudo class to mess with the hover state of the object.
Here is some code example using your title class: .title:hover { color: yellow; cursor: pointer; }
This way when you hover over the title, only the color changes and it doesn't indent itself.
0 - @samritbasnetSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am happy to finish this project although it was simple project learnt some styling syntax and flexbox and lot more.
What challenges did you encounter, and how did you overcome them?I had trouble with styling the class component in center thanks to help of community and online resource i overcame it.
@josenegrete123Posted 3 months agoOh sorry my comment got cut off. I would migrate the style html tag into a separate CSS file. So that the html won't be as bloated and would just be pure html. That way you can reduce confusion when changing around the style of the project.
Marked as helpful0 - @samritbasnetSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am happy to finish this project although it was simple project learnt some styling syntax and flexbox and lot more.
What challenges did you encounter, and how did you overcome them?I had trouble with styling the class component in center thanks to help of community and online resource i overcame it.