Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

London class 8/teniola ogunjinrin #270

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

Conversation

teniolao
Copy link

@teniolao teniolao commented Dec 3, 2021

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name: Teniola Ogunjinrin
  • Your City: London
  • Your Slack Name: teniola

Homework Details

  • Module: CSS Grid
  • Week: 3

Notes

  • What did you find easy? Nothing! lol it took me 3 days to eventually get my thoughts together. Thanks to Maten, I was able to manipulate a grid layout to fit the project template.

  • What did you find hard? I would say css grid in general is very confusing, I won't say hard. getting the basics helped me to work my way around it. For a start I think my code for this current project is not the best. I think I could have done better as well as using shorter code methods. to made my code look shorter and readable. Text and button layout

  • What do you still not understand? I have not been able to figure out how to use the grid-template-area, it only worked why I tried it during the tutorial. If I am able to figure it out it will save me writing a lot of code. combining Flexbox with grid was kind of confusing. Might research more on that. I am unable to move the text and button around in a container. Getting rid of unnecessary gaps (I’m thinking this is due to my code layout, I’m certain I need to research more to improve my code.

  • Any other notes?

  • I would really appreciate if we are given more time to learn these topics. I know there's a lot to cover.

@LucyMac LucyMac added the LDN-8 London class #8 (start Nov 2021) label Dec 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
LDN-8 London class #8 (start Nov 2021)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants