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

All comments

  • coder-abdo• 280

    @coder-abdo

    Submitted

    What are you most proud of, and what would you do differently next time?

    I am using zustand as the main state management for the project with custom hooks and organized files.

    What challenges did you encounter, and how did you overcome them?

    complex states which I overcome by using zustand and custom hooks.

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

    in the building stage, there is a minor error that makes my solution not appear properly.

    coder-abdo• 280

    @coder-abdo

    Posted

    it works now and the live site is now live.

    0
  • tortaruga• 450

    @tortaruga

    Submitted

    What are you most proud of, and what would you do differently next time?

    It works! The code is a bit messy but I'm happy that it works

    What challenges did you encounter, and how did you overcome them?

    I struggled with the mobile menu, the cart menu, the lightbox, the general layout, making each button do whatever each button had to do, it'd be quicker to ask what i didn't struggle with. but my perseverance will triumph. I bend, but do not break. I whine, and complain, and cry and throw tantrums, but eventually, by sheer luck, through some deus ex machina and never through my actual skills, I reach my goal. i'm so sorry for anyone who actually reads these i'll stop wasting your time

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

    is there an easier way to make that side menu thing so that the rest of the screen gets darker and paler when it is open? right now i use a dialog for the menu, and style the rest of the content through js whenever the dialog is showing, because the ::backdrop only works if i make the dialog a modal, but if it is a modal it doesn't stay styled at one side of the screen, but it appears in the center no matter which saint I light a candle to.

    coder-abdo• 280

    @coder-abdo

    Posted

    your solution is amazing, I love the way you handle the entire application only using native javascript so you have done great, but here are little notes:

    • you can make the overlay when clicking the menu icon by using z-index with a high value to cover the whole application.
    • the lightbox has a vertical scrollbar you need to remove it and you can use overflow: hidden to prevent this problem. you did an amazing job.

    Marked as helpful

    0
  • P
    BillRozy• 170

    @BillRozy

    Submitted

    What are you most proud of, and what would you do differently next time?

    I am proud of overlay side navigation bar fully accessible and it also handles focus well, allows to escape from the modal pressing ESC

    What challenges did you encounter, and how did you overcome them?

    all this accessbility thing take some time to test, now I started using NVDA and it goes better comparing with ChromeVox

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

    no

    coder-abdo• 280

    @coder-abdo

    Posted

    you used React which is an amazing library and you did an awesome job I have a little suggestion:

    • always split the logic from UI by splitting the logic in a custom hook and using it, it will help you in testing and readability of the code.
    0
  • Jordan Davenport• 660

    @jjdavenport

    Submitted

    What are you most proud of, and what would you do differently next time?

    Mostly my JS, although I think i use too much global scope the functions break down the problem and clean up my code a little. I built the site mobile first and would do the same next time, this is also my first site with accessibility from scratch although I still have more to learn to make it more accessible.

    What challenges did you encounter, and how did you overcome them?

    The main problem was integrating the radio inputs with the buttons that contain them.

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

    Any feedback is welcome particulary regarding form validation and any tips to improve accessibility.

    coder-abdo• 280

    @coder-abdo

    Posted

    your code is awesome; you just need to replace the README.md file with your thoughts and main criteria.

    Marked as helpful

    1
  • Boris• 2,870

    @mkboris

    Submitted

    What are you most proud of, and what would you do differently next time?

    Proud of the outcome.

    What challenges did you encounter, and how did you overcome them?

    None.

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

    All feedback is welcomed. Thanks!

    coder-abdo• 280

    @coder-abdo

    Posted

    your work is awesome but here are some tips and tricks to enhance the solution a little bit:

    1. don't make the alt attribute empty you have to add a readable alt for accessibility and SEO.
    2. you can use the dl tag or detail tag to describe the accordion this is what I found in many articles the div tag is not bad but sometimes you need to achieve the best thing to achieve the right tag.
    0
  • Simona Pizio• 110

    @SimonaPiz

    Submitted

    What are you most proud of, and what would you do differently next time?

    All! I don't know if I would do it differently another time

    What challenges did you encounter, and how did you overcome them?

    I had difficulty with some accessibility specifics, such as aria-role and keyboard navigation

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

    Is my solution optimal for web accessibility?

    coder-abdo• 280

    @coder-abdo

    Posted

    • your solution is amazing I love it. 1- your code is very semantic and accessible. 2- your solution is responsive on desktop and mobile. 3- your code is readable and well-structured. overall you done amazing
    0