Open SalwaMugh opened 5 years ago
https://github.com/FACG6/emotion-detector/blob/722171050724feb8aedef4fb487e6922f6d32d5d/src/handlers.js#L39 always declare your variables at the start of you js file also you could have done more on modularization part like separating the request functionality in other file and make some changes on it.
I liked your project , you have a nice api guys . also I liked how you handled the red square part over the img at the dom file , very clever :+1:
ty alot
aminalakhsham
commented
5 years ago aminalakhsham
commented
5 years ago
https://github.com/FACG6/emotion-detector/blob/722171050724feb8aedef4fb487e6922f6d32d5d/src/handlers.js#L39 always declare your variables at the start of you js file also you could have done more on modularization part like separating the request functionality in other file and make some changes on it.
I liked your project , you have a nice api guys . also I liked how you handled the red square part over the img at the dom file , very clever :+1: