What challenges did you encounter, and how did you overcome them?
I got one problem which I dont solved, dont know why I cant change color of svg graphic with 'fill' property.
@oshudev
What challenges did you encounter, and how did you overcome them?
I got one problem which I dont solved, dont know why I cant change color of svg graphic with 'fill' property.
@oshudev
Posted
Hello there π. Good job on completing the challenge!
About your question, you can't use fill property if the svg
file is inside the img
tag. What you can do is copy and paste the svg code inside your html. Alternatively you can use filter
property to change the color. I found an article explaining how to do it, Swapping Fill Color on Image Tag SVGs (Using CSS Filters!).
I hope you find it useful! π
Happy coding!
Marked as helpful
@halex-panama
Submitted
What specific areas of your project would you like help with?
Is there any way to make the heading and paragraph text looks exactly like the design?
@oshudev
Posted
Hello there π. Good job on completing the challenge!
About your question, there is a way to make the heading and paragraph look exactly like the design. Try add a container to h1
and p
tags. It looks something like this:
<div class="text-container">
<h1>Improve your frontend skills by building projects</h1>
<p>
Scan the QR code to visit Frontend Mentor and take your coding skills
to the next level.
</p>
</div>
Then try adding inline padding to .text-container
to push the text inward. Adjust it to your liking to achieve similar results.
I hope you find it useful! π
Happy coding!
Marked as helpful
@gmatt20
Submitted
What are you most proud of, and what would you do differently next time?
This is my personal record! I have successfully made this simple project in under 2 hours. I am extremely proud of myself for this accomplishment. I used barely any help and references. I was able to be in a constant flow state which had greatly enriched my coding experience.
A major thing I learned in CSS is target very specific elements by using the > key. What I would do differently next time is plan the layout and deeply analyze before actually writing code. For example, I can draw sketches.
What challenges did you encounter, and how did you overcome them?
The only challenges were jogging my memory on pseudo-classes. Other than that, it went smoothly.
What specific areas of your project would you like help with?
I need to learn better responsive web design. For example, I need to understand the pixel size differences between a computer, phone, and other technologies. I need my project designs to be consistent with every platform. I also want to delve deeper into how I can make my websites accessible. Generally, I shall improve my craft of HTML and CSS before moving onto JS.
@oshudev
Posted
Hello there π. Good job on completing the challenge!
span
for name, location, and description, you could've use h1
or p
tags. You can learn more about HTML semantics here Semantics - MDN Web Docs Glossary: Definitions of Web-related terms | MDN.flex
to center text content, try using text-align: center
.I hope you find it useful! π
Happy coding!
Marked as helpful
@oshudev
Posted
Hello there π. Good job on completing the challenge !
<body>
<main>
<div class="container"></div>
</main>
<footer></footer>
</body>
Overall great job!π
Happy coding!
@gmatt20
Submitted
What are you most proud of, and what would you do differently next time?
I am proud of making general CSS style code for main, footer, and image. By declaring for 3 big elements in CSS, everything has been able to ideally be aligned (for the most part) and have a consistent font.
What challenges did you encounter, and how did you overcome them?
I had difficulties with CSS and styling as usual. My main issues were having everything aligned perfectly. I have used the div element multiple times which has helped. I am particularly concerned about CSS layout and aligning code. I have coded so that it is perfectly precise. This form of code is very particular, and I want to ideally make them general rather than specific.
What specific areas of your project would you like help with?
I want to continue my layouts and aligning in CSS. Again, I want to improve my skills in CSS so I can make awesome looking websites! The looks is where I am difficulties at the moment.
@oshudev
Posted
Hello there π. Good job on completing the challenge !
Here are my basic suggestion for you to improve on:
* {
margin: 0;
padding: 0;
box-sizing: border-box;
}
This snippet allows you to reset browser default for margin and padding. If you notice there is box-sizing
that is included. The CSS box-sizing property allows us to include the padding and border in an element's total width and height. You can read more here if you are interested.
This section is css specific suggestions:
font-family
in the body
tag. It also should have a fallback font just in case your primary font don't work, e.g. font-family: 'Figtree', sans-serif
.margin
or padding
to only left and right, you can use margin-inline
or padding-inline
. Similarly if you want to define margin
or padding
to only up and down, you can use margin-block
or padding-block
.margin
or padding
like this padding: 8px 15px 8px 15px;
, you can rewrite it to padding: 8px 15px;
.You can re-generate your screenshot to reflect the latest changes.
I hope you find it useful! π
Happy coding!
Marked as helpful
@QuyetTran1211
Submitted
First it's hard to imagine what to build but i try my best. Thank frontendmentor for intersting project. I love fem
@oshudev
Posted
Hi Tran Minh Quyet π. Nice work! The only thing I have a problem with is when the user clicks the rating button, it doesn't stay selected. You should provide feedback to the users that the rating button is selected. Besides that, everything is fine.
Marked as helpful
@Achilles43
Submitted
I am open to everyone's feedback;))
@oshudev
Posted
Hi Achilles43 π, great job! When writing HTML, you should always use proper landmarks for Accessibility. For Example:
<body>
<main>
<div class="all"></div>
</main>
</body>
Overall great job!
@Vorczu
Submitted
Hello, I'm pretty new here, I've done coding before, but never learn the basics. I am open to anyone who can point out my mistakes and point better solution :) I'm unsure to all of my scss, paddings, margins and all. I don't really know when I should use this or that property.
@oshudev
Posted
Hi Adam π. You've done well on your first challenge. To have an effective centred flex-box
, you should consider having height: 100vh;
. In this way, no matter what is the view height, it will constantly position itself in the middle. Also, check the report for other issues as well.
@varunsharmablog
Submitted
This is my first solution and I'm very excited to see what kind of feedback I would get from everyone. Here are a few things I would really like to know:
@oshudev
Posted
Hi Varun Sharma, great job!
Add the main
tag and wrap the card for Accessibility. It should be like this:
<body>
<main>
<div class="card"></div>
</main>
</body>
Overall you did well! Hope it helps.
Marked as helpful
@ejionaut
Submitted
There were quite a few things I struggled with and one of them is the responsiveness of the website. I am not sure if I did the sizing right.
I would love some feedback on the work, as I am a beginner :)
@oshudev
Posted
Hi EJ π. You've done well on your first challenge. About responsiveness, there is a trick that I always use. Do max-width: (whatever the size you want)
then width: 100%
. In this way, when the view width is less than the set max-width
, it will naturally scale to whatever the size of the parent element is. Then consider using the main
tag and wrapping the card for accessibility. It goes like this:
<body>
<main>
<section class="Main-Display"></section>
<div class="attribution"></div>
</main>
</body>
Another thing is that you can't use z-index
if there is no position
property. And also, align-self
and justify-self
can't be used if you're not using display: grid
. You should use align-items
and justify-content
on flex-box
. Also, check the report for other issues as well.
Marked as helpful
@SebastianLaka
Submitted
Hi it's my third project and it learned me so much. If you know how make it better tell me please. Thank you :)
@oshudev
Posted
Hi Sebastian, great job!
Add the main
tag and wrap the card for Accessibility.
It should be like this:
<body>
<main>
<div class="card"></div>
</main>
</body>
Also when styling a button always put font-family: inherit;
so that it matches the overall font family.
Overall you did well! Hope it helps.
Marked as helpful
@sofiamatrella
Submitted
This is the first time I make something with html & css after learning it for 2 days, so it's not perfect hahaha. I had trouble with aligning vertically the white box that contains the image and the text (tried using flexbox and other methods, neither of them worked without a specific height so I had to specify it and because of that it's not responsive, not sure of how to do it correctly :( ) All feedback is welcome, thank you!
@oshudev
Posted
Hi SofΓa Matrella, great job!
Add the main
tag and wrap the card for Accessibility. On aligning it vertically, you can do height: 100vh
so that it can be responsive on any device.
Overall you did well! Hope this helps.
Marked as helpful