Abdul-Azeez / jqueryfundamentals

0 stars 1 forks source link

Exercise 7.1 #8

Open Abdul-Azeez opened 9 years ago

Abdul-Azeez commented 9 years ago

https://github.com/Abdul-Azeez/jqueryfundamentals/blob/master/jqfundamentals-master/exercises/chapter7/product.js

Level 3)Product Display.We have product data in csv format and images as followsGiven the csv file and images at this url - http://dl.dropbox.com/u/628209/exercises/javascript/product_data.tar.gzHere is the JSON for the same data.. http://dl.dropbox.com/u/628209/exercises/javascript/product.jsonCreate a page which loads this json data and renders a store front. All the products should be displayed by default. Show a list of Brands in the left. Upon clicking the checkbox in front of a brand, only that particular brand products should be show, hide all others. If no checkbox is selected show products belonging to all brandsSimilarly have a filter for colorAlso create a toggle for All producs and available products - if user clicks “Available Products” show only products that are not sold out.The filters should work together - so I should be able to see Only Yellow products of Brand C which are Available.

yuktikhurana commented 9 years ago
  1. https://github.com/Abdul-Azeez/jqueryfundamentals/blob/master/jqfundamentals-master/exercises/chapter7/product.js#L6 please use var
  2. https://github.com/Abdul-Azeez/jqueryfundamentals/blob/master/jqfundamentals-master/exercises/chapter7/product.js#L34 Why do we need this function ?
  3. https://github.com/Abdul-Azeez/jqueryfundamentals/blob/master/jqfundamentals-master/exercises/chapter7/product.js#L103 Can we set the class attribute in a single line ? instead of calling addClass 3 times
  4. https://github.com/Abdul-Azeez/jqueryfundamentals/blob/master/jqfundamentals-master/exercises/chapter7/product.js#L109 #productList being used again and again
  5. filterProducts can be optimised. Can we work it out using Jquery filter method ?
  6. Can we work a little on the css please. the products are listed like a long line. should be a grid
  7. also add an option to clear all filters