@erlanggaadptr
Submitted
Looking to hire developers?
@nerometa
@erlanggaadptr
Submitted
@nerometa
Posted
Wow! Great job of you to make this looks very close to the design. A few recommendations, though:
<img>
tag inside <p>
tag, which mainly uses for texts. I think you can improve upon it by grouping these elements together with <div>
, then use <img>
for icon and maybe <span>
for text so it begins on the same line as your icon. Something like this:<div>
<img src="ethereum.svg" alt="Ethereum">
<span>0.041 ETH</span>
</div>
<a>
tag over any text element to make it resembles a link. I kinda get it. Personally, I would put an <a>
tag inside text element like:<span>I would like to introduce you to this <a href="#">hyperlink</a>.</span>
That's my two cents. You're doing great!
Marked as helpful
@correlucas
Submitted
👾 Hello, Frontend Mentor coding community. This is my solution for the 3 Columns Card Component.
Feel free to leave any feedback and help me improve my solution or make the code clean!
I'll be happy to hear any feedback and advice!
@nerometa
Posted
This is truly amazing, Lucas! Love the hover effect and your take on color that's entirely different to the design.
After seeing this, I HAVE to follow you man. You're that amazing😀
@julianv97
Submitted
@nerometa
Posted
Hmm, looks like your colors are a bit off. But don't worry, it's pretty easy for you to do and I know you can do it.
in your tailwind.config.js
file, you can add custom colors from the style guide that's been provided with the challenge like this:
theme: { extend: { colors: { "custom-color": "#FF0000" } } }
and to use it as a utility class, you can then do bg-custom-color
or text-custom-color
since you've put your color in your config file, Tailwind will know.
Also, good work! I wanna see you improve this!
@MURRAY122
Submitted
I did have an img tag within the HTML page, but I removed it due to not being able to change the src of the image as the media screen changed from mobile to desktop. I do wonder however if it would be possible to have changed this without the need for JavaScript and strictly CSS.
I also used CSS variables but had to resort to manually inputting the hsl for the button hover. I did attempt to use hsla with the variable within in it and then just change the alpha but didn't have the desired effect. I wondered if I missed something or if there is a better approach to darkening the background without effecting the text color of the button.
Edit - Updated CSS Layout
@nerometa
Posted
You're doing great so far! Here's one of the solution I use for your image issue:
You make 2 <img> tags with different images together, maybe assign a class to work with CSS later. Suppose you got one <img> with mobile image and another <img> for desktop image. Then, for it to change, you put all of that <img> into @media queries and set property display: none
when you want to hide it and set display: block
for an image you want to show.
Suppose you got your images and you decided that 720px will be your breakpoint. Set that @media query to (max-width: 720px), which means anything that's below 720px will show up (smaller devices). You might put your desktop image to display: none and mobile image to display: block.
You don't have to do another @media (min-width: 720px) for desktop since ANYTHING that's not in media query works similar to that. So, out of @media query, you set your desktop image to display: block and then your mobile image to display: none
Other than that, great job! Keep coding✌⌨
@ChanonNon
Submitted
@nerometa
Posted
ทุกอย่างโอเคเลยฮะ responsive ก็ทำได้ดีเลย ทีนี้ผมอยากให้ลองอะไรหน่อย
@EmmaDwebber
Submitted
I appreciate every feedback. How do i make my image show when using netlify. After deploying the image refuses to come up It was difficult styling the page at first i guess i was able to figure it out. Thank you!
@nerometa
Posted
Like others say, it seems like you didn't include images folder into your project. Make sure you push it into Github so your image will show. Add an 'alt' for more accessibility.
I'm not certain about filepath. maybe "./images/image.png" works, maybe "images/image.png" works depends on where you deploy your app, I think.
Marked as helpful
@KarnThapagon
Submitted
@nerometa
Posted
ค่อนข้างตรงดีไซน์เลยฮะ ถือว่าทำได้ดีมากสำหรับโปรเจคท์แรก ผมมีข้อแนะนำอยู่ประมาณนึง คิดว่าคุณต้องแก้ไขได้แน่ๆ เลยฮะ
โดยรวมถือว่าทำดีมากๆ ฮะ พัฒนาต่อไป💪
Marked as helpful
@JairoAtoche
Submitted
First time using Sass. I´d like to know your feedback ^^
@nerometa
Posted
Dude, this is so well done! Congratulations🎉 I legit have nothing to say since it's so good, even HTML and accessibility issues aren't present.
Any feedback will be appreciated. Thanks
@nerometa
Posted
You're doing great, dude! No major problems. Only minor problems that I think you can fix it.
Headings. It should be a bit bigger like 2rem or 36px. You also need to make sure that it's in uppercase. Can be done by text-transform: uppercase if I remember correctly
Buttons. Text can be bolder and font size a bit bigger.
Responsive: I understand that the challenge requires you to make desktop and mobile responsive, not in between. If you can make tablet responsive that's a plus.
You can check out my solution on this challenge. I wanna hear your feedback as much as you do.
Marked as helpful
@SamuelOsewa
Submitted
The issue I had most was with CSS in terms of sizing Sizing: Are you supposed to hardcode the width and height of the elements? and if not? What is considered the best practice for that?
@nerometa
Posted
You could do that. What I'm gonna say here might not be the best solution though:
Hardcode width and height of your card and use @media query to hardcode yet another width and height properties when your screen got smaller or bigger.
You could also use min(), max() and clamp() to do sizin. It might work a bit better.
Marked as helpful
@hobbycoder7
Submitted
I'm not very "artistic". I find it hard to visualize things. I was able to use a Chrome extension to overlay the design image and fiddled with CSS in developer tools until I was able to create my solution. How do you go from Idea or Visual to code?
@nerometa
Posted
First of all, that was great. Your design is pretty similar to the challenge's design. I might have to checkout PixelPerfect that you using though. It's interesting.
I think when it comes to transferring idea to code, you might have to do some UX/UI design beforehand like these challenges provide you. You can literally see how your code will look like on it.
Maybe you cannot 'see' how the page are structured. I think learning more about different HTML tags and what and how they're use is crucial here.
Maybe you don't know some of CSS properties. That explains you fiddling in dev tools, which I suggest you go check out CSS guides here: https://css-tricks.com/guides/ or you might have to learn the basics more.
Marked as helpful
What could i have done better? i'm open to learning from you
@nerometa
Posted
I guess you're new to coding. This is good enough based on your experiences. Now, several things I want you to fix and I know you can do it: