Jade

Latest comments

• 

  szam•800

  @k-stopczynska

  Submitted almost 3 years ago

  

  faq accordion

  2

  50
  

  Jade•105

  @jadetrue

  Posted almost 3 years ago

  Hey @k-stopczynska,

  I've had a go at this particular challenge myself and it's a really good one to test yourself, so well done for getting this up! 👏

  I don't do a lot of reviewing so I will just give you some visual feedback.

  • I love the little bit of animation you've added on the arrows.

  • The highlighting of the accordions is also a nice touch.

  • The background gradient is not quite the same as the design, it should be lighter on top.

  • Font for the accordions - the Header text could be larger.

  • I don't know if you've tested this on different size screens, but you may want to look at the cube that's meant to sit within the image area - I'm using a MacBook Pro 16" to view your link: https://paste.pics/7e5a38e17954136efd6438f52af32dea

  Hope this is useful!

  0

  Marked as helpful
• 

  Jade•105

  @jadetrue

  Submitted about 4 years ago

  

  Stats Preview Card - using Flexbox and Grid

  2

  20
  

  Jade•105

  @jadetrue

  Posted about 4 years ago

  Hey Bernardanxiety! Thanks for the comment and feedback I really appreciate it.

  I am aware of that in this particular piece and if I were to implement this into a project/full page website, it would definitely be h3's (IMO).

  Just another tip from beginner to beginner, with Heading Tags, you will want to bear in mind what you would like search engines to pick up - I found a useful article on this too: https://cultofweb.com/blog/multiple-h1-heading-tags/.

  I'm not sure if you clicked on the Live link but my stats aren't squished up against each other and I am using Grid to display these with plenty of space between similarly the widths look better than in the screenshot too.

  0