Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Log in with GitHub
OverviewSolutions28Comments10

All comments

  • 👾 Ekaterine Mitagvaria 👾• 7,640

    @catherineisonline

    Submitted

    Hello, Frontend Mentor community! This is my solution to the Crowdfunding product page.

    I appreciate all the feedback you left that helped me to improve this project. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.

    You are free to download or use the code for reference in your projects, but I no longer update it or accept any feedback.

    Thank you

    Crowdfunding product page

    10

    Dave Hillman• 830

    @dhillman1989

    Posted

    Very clever with the animated values. Nice little touch to a well implemented project. It shows your skill and creativity at the same time. Well done.

    0
  • Yonatan Afewerk Teshome• 720

    @Yehonatal

    Submitted

    I liked this challenge loved how the design and i thought i can recreate it was gonna add more animations but i don't know much so i have to learn it first, just did some from what i knew tried to make the cards animate in based on when they come to view i did mange but ill have to make them not hidden before so i have work to do there any suggestions, comments are welcomed :)

    News homepage

    #animation#bem#sass/scss#theme-ui

    2

    Dave Hillman• 830

    @dhillman1989

    Posted

    Very nice work. Only one accessibility issue which can be easily fixed. Keep up the hard work.

    0
  • Dave Hillman• 830

    @dhillman1989

    Posted

    Visually well implemented, I especially like the transition/animation of the dropdowns although they are perhaps a little clunky as one opens and another closes at the same time.

    From a coding perspective you have multiple items with the same ID. An ID should only be used once for unique elements. If you need to identify multiple elements use classes.

    Otherwise nice work.

    Marked as helpful

    0
  • Dave Hillman• 830

    @dhillman1989

    Posted

    A few accessibility issues + the colours are a little off, but this is an issue I had too so I understand.

    The functionality of the rating selection and hovers is implemented well. The reason your second page isn't loading is because you have an event listener on 'submit-rating' but this button doesn't exist. It should be an id of 'submit'.

    0
  • Dave Hillman• 830

    @dhillman1989

    Posted

    Nice work.

    I like the way that when you add items, the basket pops into focus, and the checkout loading screen is a nice touch. A suggestion would be that once you add to the basket the counter on the product reset to 0, just my thoughts.

    Responsive menu is well implemented at lower screen size, but maybe needs an animation or transition to make the opening/closing of the menu seem smoother and less jarring.

    Overall nice implementation of the functional design.

    0
  • maya• 60

    @nicoleanalisecox

    Submitted

    This is my first Front End Mentor Challenge! Would love some feedback on how I can improve my code/approach.

    Profile Card Component (HTML & CSS)

    1

    Dave Hillman• 830

    @dhillman1989

    Posted

    Accurate creation of the design, and no code issues. Good first piece.

    0
  • Kevin Canlas• 225

    @kvncnls

    Submitted

    Demolish my code!

    Also, I couldn't figure out how to get the custom error message for the form. Could anyone help me with that? Thanks in advance!!

    Fylo dark theme landing page

    3

    Dave Hillman• 830

    @dhillman1989

    Posted

    Great effort pal. not looking at the code, but the page is very close to the design minus some sizing issues. Well done.

    1
  • Dave Hillman• 830

    @dhillman1989

    Posted

    Having attempted this one myself I agree the placing of elements was tricky at times, but you have done a fantastic job here.

    1
  • Dave Hillman• 830

    @dhillman1989

    Posted

    Pretty much there. Couple of html issues to easily fix. Only real major difference I see is some header spacing at the top. Otherwise good job.

    0
  • Dave Hillman• 830

    @dhillman1989

    Posted

    Impressive. good work.

    0