Suzunatsu

Latest comments

• 

  Synne Storm•190

  @synnestormSubmitted 3 days ago

  What specific areas of your project would you like help with?

  Feedback is very appreciated!

  

  Testimonials Grid Section using HTML and CSS

  1

  20
  

  Suzunatsu•100

  @st0272Posted 3 days ago

  Your code is generally good. However, using the root-relative size unit (rem) is more desirable.

  0
• 

  Jose Beltran•100

  @jose-beltranSubmitted 20 days ago

  

  Responsive landing page using css flex

  1

  10
  

  Suzunatsu•100

  @st0272Posted 19 days ago

  Your code is generally well-structured, but several issues and areas for improvement in terms of semantics, accessibility, and organization. Below is my opinion for improvement.

  Use of <p> for titles

  The <p> tag is used for headings like "Reliable, efficient delivery" and "Powered by Technology", which should be in <h1> and <h2> respectively for better semantics.

  Missing alt attributes in images

  The <img> tags have an empty alt="", which is not good for accessibility.

  Improper nesting of divs

  The content-cards-two div is nested inside the .content-cards but is structured inconsistently. It appears that all .card elements should be direct children of .content-cards for better alignment and flexibility. Using grid to style this four-card layout is efficient.

  I hope my advice will be helpful for your coding. Keep up the good work!

  0
• 

  Erkant•730

  @erntTt94Submitted 4 months ago

  What specific areas of your project would you like help with?

  Any feedback appricated...

  

  Article Preview Component

  1

  40
  

  Suzunatsu•100

  @st0272Posted 4 months ago

  Your JavaScript(React) code is well-written, but there are several areas for improvement in the HTML and CSS:

  1. The design is larger than specified in the design file.
  2. It's preferable to use class instead of id for styling and structure.
  3. The desktop design is incomplete. Wrap the image and content items in <div class="card"> and use display: flex or display: grid to align the items properly.

  I hope my advice will be helpful for your coding. Keep up the good work!

  1
• 

  andreshinostroza•30

  @andreshinostrozaSubmitted 5 months ago

  

  Responsive social links with css flex

  2

  20
  

  Suzunatsu•100

  @st0272Posted 5 months ago

  Your code is well-structured, but there are several points for improvement:

  1. Accuracy of Sizes

  • Font sizes: Each font size is 1 or 2 pixels larger than the design file.
  • Margin: The margin size for the text "London, United Kingdom" is slightly wider than expected.

  2. Unnecessary CSS Styles

  • h1 and h2 class bg-c: The principal class already applies the same background-color: var(--Gray-800), making this redundant.
  • enlace class: The cursor: pointer style is unnecessary since it is the default behavior for anchor (<a>) tags.

  I think it could be even better with a little more attention to detail. Keep up the good work!

  0
• 

  Ololade•110

  @zololadeSubmitted 6 months ago

  

  Responsive landing page using CSS Grid and FlexBox

  2

  00
  

  Suzunatsu•100

  @st0272Posted 6 months ago

  Your code is well-structured, but there are several important points for improvement:

  • The desktop design is slightly smaller compared to the original design. Please make sure to measure the padding, image sizes, and other elements correctly.
  • The mobile version of the design is the same as the desktop version. Please create a mobile version using appropriate media queries.
  • Ensure that the h4 tag with the .header__caption class follows the appropriate heading structure. It's better not to use an h4 tag before an h1. It might be better to use a p tag instead of an h4.
  • The h2 and h3 tags inside the section tag with .main__pricetag class might confuse the document hierarchy. You could consider using <span> with appropriate class names and using <div> tag or <p> tag instead of <section> tag.

  I think your coding skills could be even better. and Keep up the good work!

  0
• 

  Mehedi Hasan•60

  @sudo-mehediSubmitted 6 months ago

  

  Blog preview

  1

  10
  

  Suzunatsu•100

  @st0272Posted 6 months ago

  The HTML and CSS code you provided is well-structured but could benefit from a few improvements for better accessibility, maintainability, and design.

  • Ensure that the <h4> tag follows the appropriate heading structure. It's better not to use an <h4> tag before an <h1>. It might be better to use a <p> tag instead of an <h4>.

  • Inline styles are not good for maintainability, so it's better to use external CSS files.

  • The .card doesn't have a border.

  • The content size on the desktop version is slightly smaller compared to the design file. Please make sure to measure the padding, image sizes, and other elements correctly.

  I think it could be even better with a little more attention to detail. Keep up the good work!

  0