Hi .
- The best place to start is with the feedback from the report above, where some issues are to do with duplicate id's - you can opt to use a class instead. You can only use an id name once .
- The idea of the project is to be able to filter cards when you click on the referred items in the designs - at present this is not happening.
- Notably , the background image is missing on your mobile presentation
- The search is not displayed well on mobile as the important text is cut off, I don't know what to search and probably it would be helpful to give a hint to users on what needs to be done
- An li tag is used to represent an item in a list, i see yours has a couple of other elements notable an img , div and a nested ul. You need to relook on this as i believe it is not it's proper use.
- The buttons in the ul tag as well cannot be standalone without the li tag. The above issues will have an impact to screenreaders as information will be confusing. Generally the display is good. Hope you will find this useful
2
lythanhhai• 150
@lythanhhai
Posted
@ChamuMutezva Thank you very much , the judge and feedback of your really so useful to me , about problem with duplicate id's , i will improve it . filter cards when you click on i am making it more perfect . I really thank you about this comment . hope get more feedback of your
0
