Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

nft-preview-card using html and css with flex layout

@grgrocky

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


The most challenging is to fit the avatar with white circle border. But some how I managed embedding it with the div container.

How did you embed .svg icon file? Did you specify width and height of it? or how did you resized it?

Any suggestions will be appreciated. Thank you.

Community feedback

Kelvin 770

@githukelvin

Posted

@grgrocky Hi

I wanna help you with the image styling i mean the svg. So instead of adding the div of the avatar svg to style it you can just use . To style the svg you could have added a class or use this i mean for you you used id will work the same #creator-name svg {here you write your styling and for the border you can you use this border: 1px solid hsl(0,0%,100%;} this will work too . Try this will help you remove the html issue replace section with divs

But the best idea i will give you is try stop using id and instead use classes they work well. And trying using divs in a div not section in a divit should vice versa divs will do well in styling

Marked as helpful

0

@grgrocky

Posted

@githukelvin Sounds good. Thanks mate for giving me different view. I will surely apply it. Cheers.

0
Kelvin 770

@githukelvin

Posted

@grgrocky my pleasure happy coding buddy

1
Danilo Blas 6,300

@Sdann26

Posted

Hi Rocky!

I just reviewed your project and your HTML layout and structure is fine.

Personally I don't recommend you to use section, when making these components. Most newbie level projects are components that can be reused in a page so they will be added in some section of this page. Therefore section is usually used for sections of a page not for components, they usually use div to be able to group the blocks of the component.

That's why in the report it says that it expects your other section to have an h2 inside, although as a general rule it is not necessary to add it, it still detects this error, from my opinion I would recommend you not to use section unless you have to group different themes of your page (sections).

On the other hand, although your layout is fine, you are missing the active states. If you check the design folder you will find an image with the active states that refers to when you pass the courses or click on something it activates or changes color for example. Add them to finish the challenge completely.

Good Luck!

Marked as helpful

0

@grgrocky

Posted

@Sdann26 Exactly, I also had doubt about section. Thanks mate for pointing it out. And I didn't have any idea what to do with this active states. Thank you for clarifying it.

Cheers

1
Danilo Blas 6,300

@Sdann26

Posted

@grgrocky Don't worry, just add them with the hover pseudoclass. By the way, if you remove all the errors from your report you can generate a new one to have 0 errors.

Good Coding!

Marked as helpful

1

@grgrocky

Posted

@Sdann26 Thanks mate. I will. Cheers

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord