Skip to content
OverviewSolutions
3
Comments
2
Adina
@adinabbb

All comments

  • Adina•50
    @adinabbbPosted almost 3 years ago

    Hey Jrgwz! Great job, it's really close to the original design. I see you have a ease-in transition - I suggest maybe leaving only "ease" to capture the hover in and out of the element in order to look more seamless.

    Regards,

    0
  • hamidreza•20
    @hamidrezataklooSubmitted almost 3 years ago

    hi guys i will happt if u see my code and get your feedback

    flex box in css

    1
    Adina•50
    @adinabbbPosted almost 3 years ago

    Hello Hamidreza! Great work so far, see my feedback below:

    1. Try to get as close as possible font-wise - you can use "font-weight:" and choose the different types of front given in the styling file.
    2. There is no hover effect for some of the applicable elements (ex: main "Equilibrium" text and creator name).
    3. Although this is not a mistake, as a best practice tip you can make another file "style.css" where you can put all the css information there in order to keep everything organised.

    Keep up the great work and on to the next challenge :)

    Marked as helpful
    0