Frontend Mentor | Christina Hicks's profile

@OstatidieSubmitted 5 months ago

Recipe page

@PTN6389Posted 4 months ago

Good job. I like how you added the color variables and named them appropriately. A couple suggestions: 1) You can use the <hr> tag to add the divider between the sections; and 2) On the nutrients section, you can use the same class for each line item to style since it's the same style, or in the css file, you can separate the classes by comma to use the same style block.

Marked as helpful

0

Kraiimen•40

@KraiimenSubmitted 5 months ago

What are you most proud of, and what would you do differently next time?

I finally understood how to use flexbox. (I think)

What specific areas of your project would you like help with?

Everything about flexbox, responsive design, and the code itself. I'm open to suggestions on how to improve my code.

Social links profile card

1

Christina Hicks•80

@PTN6389Posted 4 months ago

Good job, I like the comments in the css file, which helps with readability. A couple suggestions:

You have box-sizing: border-box; within many selectors, but you only need to do in one * { box-sizing: border-box;}
The links within the socials-container can have one class since there isn't a difference in the look per social link. The style can be added based on the one class (i.e. class="socials-container__item").

0

Christina Hicks•80

@PTN6389Submitted 5 months ago

What are you most proud of, and what would you do differently next time?

It was a lot easier to do this one, and I felt more confident.

Social Link Profile

2

Christina Hicks•80

@PTN6389Posted 5 months ago

Thank you for the input and suggestion on the opacity. I knew the text was different, but couldn't determine the difference.

0

ange14032005•40

@ange14032005Submitted 5 months ago

box shadow, display

1

Christina Hicks•80

@PTN6389Posted 5 months ago

I like how you organized your css, and used the variables for the color and radius. Also, I like how you used the min-content value for the max-width property on the 'Learning' button class.

There does seem to be a lot of spacing between the elements within the card. I used the display: flex, justify-content: center, and align-items: center on the body element instead of the card (.article) class, which helped with the spacing.

Marked as helpful

0