LouiseCW
@costelloeward
All comments
- LouiseCW• 235
@costelloeward
Posted
Hi - this looks pretty good overall. I'd like to second James' comment about semantic html and share this cheat sheet I have found useful: https://learn-the-web.algonquindesign.ca/topics/html-semantics-cheat-sheet/
1 - LouiseCW• 235
@costelloeward
Submitted
Hi everyone, this is my first challenge for a while! Any feedback appreciated. I moved the position of the quote mark as I found it made the text harder to read.
LouiseCW• 235@costelloeward
Posted
There is no level 1 header as I've treated this as a potential section of something larger.
0 - Juliette• 100
@juliettesimonot
Submitted
Hey everyone :)
I am learning coding, this is my first project. Please tell me if you see mistakes or if you have somes solutions to improve my code. I am focusing only on html ans css for now, before learning a new language :)
Thanks a lot ! Juliette
LouiseCW• 235@costelloeward
Posted
Hi Juliette, This is great for a first project - well done! I'd suggest looking at the padding for the Annual Plan button - otherwise you are quite close to the design layout.
Welcome!
Marked as helpful
1 - Alessandro• 215
@alessandro-giuzio
Submitted
Hello there, please i need help with this challenge, i am hitting the wall. Not sure how to change the opacity.. to start
Thank you
LouiseCW• 235@costelloeward
Posted
Hi Alessandro, Welcome to Frontend Mentor. Well done for trying this one!
I hit a wall too, until someone told me to look up mix-blend-mode.
I used this along with
- object-fit: cover;
- put the image inside a div
- set the color as the background for the div rather than styling it on the image.
Hope this helps!
0 - sabrina• 65
@itssSabrina
Submitted
what do you think?
LouiseCW• 235@costelloeward
Posted
Hi Sabrina, Well done on getting this far! Your html is referring to an image folder called "images", but your code repository has stored the images without a folder.
So you can either take the /images out of the html, or make a folder for the images in your repository.
Hope this helps!
Marked as helpful
0 I used <div> tags for all the containers. Is there a better way (semantically speaking) in this case?
LouiseCW• 235@costelloeward
Posted
Hi Arthur,
Good job! You can use main, section and picture tags instead of divs -though it might lead to coding the responsive image changes in the html.
Marked as helpful
0- authenticCoder• 75
@authenticoder
Submitted
I was not able to add border-radius to the top of the image alone. The border-radius affected all sides of the image. I would like to have a border-radius on the top section alone. How can I do that? Thanks.
LouiseCW• 235@costelloeward
Posted
Hi, great job! Border-radius can take multiple values in the same way as margin and padding. So you can use something like this: border-radius: 0.5rem 0.5rem 0 0; I use rem, but this works with other unit types as well.
Marked as helpful
0 - Lesley Kimutai• 565
@Leskim
Submitted
I'm hopeless in my margin and padding know-how in solving these problems....Any help or resource location to help me manage my margins and padding responsiveness better will be much appreciated
Thanks in advance👊🏾
LouiseCW• 235@costelloeward
Posted
Hi, you're doing pretty good so far! I found this article helpful for getting my head around when to use padding and when to use margins. https://medium.com/frontendshortcut/margin-vs-padding-c1fc8ea8bfaf
0 - Beshoy S.• 460
@BeshoyS
Submitted
Hi guys, i have a problem with responsive design which i don't know which units to use to not make a big messy code.
Can anyone help me to understand it?
LouiseCW• 235@costelloeward
Posted
Hi Beshoy,
Looking good so far!
When I did this I was advised to go mobile first. So I wrote the CSS for the smaller screen first, then had one min-width query at the end. This meant all the styling for the larger screen was in one place.
0 - John Ayanyemi• 20
@johnayanyemi
Submitted
Hi guys, my second challenge on here!
The problem I faced was getting the background circles to fit for mobile. Anyone one would like to help out?
LouiseCW• 235@costelloeward
Posted
Hi John, I'm new and have just done this challenge as well. I used background-image in the CSS rather than adding the images in the html. I looked at the documentation here https://www.w3schools.com/cssref/pr_background-image.asp
Your solution looks fairly close overall though, so well done!
0 - Priyanka B M• 20
@Priyankabm564
Submitted
This is my first NEWBIE challenge. Please review my work and provide feedbacks.
LouiseCW• 235@costelloeward
Posted
Hi, I'm also a newbie to Frontend Mentor and I picked this as my first challenge too. It looks good, the positioning of the circles is slightly off, but not by much. There is also a missing line above the stats area of the card. Otherwise, I like it.
0