Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Project News Site #532

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

Anna2024WebDev
Copy link

This is a News Site about baking. It is made with Flexbox and Grid to make it responsive. This page was built during the second week of the Technigo Web Development bootcamp.

@Anna2024WebDev
Copy link
Author

Here is the link to my news site. https://thesweetscoop.netlify.app/

Also added in the readme file previously.

@JennieDalgren JennieDalgren self-assigned this Aug 29, 2024
Copy link
Collaborator

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job with this project!

Use of Flexbox and Grid:

You've done a great job applying these techniques to structure your layout. This is foundational for responsive design and will serve you well in more complex projects.

CSS Structure and Classes:

It looks like your CSS could benefit from more structure and the use of reusable classes. By organizing your CSS with more defined classes, you can avoid repetition and make future maintenance easier. Consider grouping related styles and using more descriptive class names for easier readability and reusability.

Well done, keep up the good work! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants