pinaska/instanews_one-page_api_app

Simple app to feature news from NY Times API

instanews_one-page_api_app's Issues

Consistency on quotations

Please try and commit to only using single or only using double quotes for your coding. I noticed you have indicated single quotes as your preference on eslint, however there is an inconsistency in your project resulting in double and single quotes in your coding.

HTML Validation Error: Remove Value on select element

Select Bar

The .select category on line 66 of your css. You can consider setting the background-color: #fff because once in mobile format of the site, the selection bar disappears into the background.

@font-face

I have noticed that you have named both of your @font-face fonts "Open Sans". This could cause confusion as to which font you were intending to use. It would be beneficial to differentiate the two fonts in order to prevent the wrong font to be displayed.

Check spelling

On line 47 of your fail function. Please ensure spelling is correct to prevent the user from being confused when there is no content available.

.fail(function() {
$('.story-list').append('

Sorry, it appears there is a problem. Try agai later

');
});