szamā¢ 800
@k-stopczynska
Submitted
Any feedback much appreciated
Jade
@jadetrue
All comments
- Jadeā¢ 105
@jadetrue
Posted
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!
Marked as helpful
0 -
- Jadeā¢ 105
@jadetrue
Submitted
I could improve a lot on this - but I would love some constructive feedback overall on my code please.
The overlay is not quite the purple I had hoped for either - any tips would be amazing.
Jadeā¢ 105@jadetrue
Posted
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
