NFT Component by Wahyu Fatur Rizki

Community feedback

• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted almost 3 years ago

  Hello @wahyufaturrizky ,

  I have some suggestions regarding your solution:

  <Footer> should not be in the <main >.

  • You are misusing the <section> tag . section is not meant to be used anytime you feel tempted to use a div . section is for a bigger chunk of content often titled by <h2> Read more aout usage notes.

    • You can add a <h1> with class="sr-only" (Hidden visually, but present for assistive tech). And use<h2>instead of <h3>

  • Anything with a hover style in design means it's interactive. you need to add an interactive element<a> around the image Equilibrium #3429 and Jules Wyvern .

  • <svg> 's do not add important information to a document should be considered decorative. You can use aria-hidden="true" to hide the SVG from screen readers. focusable="false" is also used to ensure Internet Explorer won’t allow the Tab key to navigate into the SVG in( icon-ethereum, icon-clock ).

  • The avatar's alt shouldn't be avatar-author it's meaningless , you can use Jules Wyvern instead .

  • the link should be wrapping the main image and either have Sr-only text, an aria-label or alt text that says where that link takes you.

  • Read more how to write an alt text

  • Give the main image display : block; , that would remove the little gap I see under the image.

  • you can use the flexbox properties and min-height: 100vh To center the card on the middle of the page .

  .container-layout {
      background-color: #0d192b;
      /* height: 100vh; */
      padding-top: 20px;
      display: flex;
      align-items: center;
      justify-content: center;
      flex-direction: column;
      min-height: 100vh;
  }
  

  And you can remove the margin from .container-card

  • width: 347pxan explicit width is not a good way . Remove the width from the main component and change it to max width instead. That will let it shrink a little when it needs to.

  • You should use em and rem units .Both em and rem are flexible, Using px won't allow the user to control the font size based on their needs.

  • You can use <ul> instead of the section . And in each <li> , there would be<svg>and <p> .

  • For the avatar's part , you can use <figcaption > and <figure> .

  • General point : remember a css reset on every project. That will do things like set the images to display block and make all browsers display elements the same

  Overall, your solution is good, Hopefully this feedback helps.

  0