https://github.com/FACN3/Domain-Finder/blob/master/dom.js#L22

what is li and idx ? the first argument should be the name of the value you are getting and the second argument is index.

and theres gotta be better way to do this but im not sure what would it be 🤔

I think you guys should add more tests,

and the existing test code doesn't really text anything in your API 😓

https://github.com/FACN3/Domain-Finder/blob/master/style.css#L15

We will need to slowly eliminate the use of pixels in CSS cause it's really bad for responsiveness/mobile compatibility, it's much more encouraged and favored ( sounded poetic there a bit ) to use rem/em.

Its not really that clear to how use the search menu

in my opinion instead of clicking the button to go to the domain it should go automatically

or maybe the button text should be different to prevent confuse 🤔

push these up thanks!

BEAUTIFUL README ❤️

i would only add local instructions

Lets get the api xhr requests and api_call object functions pushed up.
Thank you : 0 )

Please upload the outline CSS. Thanks

not sure if is_it_there is a good var name 😆
also here https://github.com/FACN3/Domain-Finder/blob/master/dom.js#L59
https://github.com/FACN3/Domain-Finder/blob/master/dom.js#L70
https://github.com/FACN3/Domain-Finder/blob/master/dom.js#L68

i guess you got it by now 😅

also loads of console log

We should insert a small text that explains the purpose of the app

Searching for a 'futuristic' date shouldn't give any results 😅

(Unless your planing to predict the future) 👽

Thank you

When the search results didnt find anything it returns 'undefined'
in my opinion there should be a message saying "Info not found" or something instead of

IP address: 80.178.83.138
City: Tel Aviv
State/Region: Tel Aviv
Country: Israel
Zipcode:
Timezone: Asia/Jerusalem
Coordinates: 32.0666, 34.765

We should start writing the readme file.

kinda empty eh? 😅

We need an index file with the basic divs and sections we will use.

you thought this is an issue to compliment your beautiful code didn't you?

you could really shorten these line of code by defining a variable that equals to data.archived_snapshots.closest

Pls fix thanks 👍

callbacks like this should always be error first callbacks, cause currently when an error occurs in your xhr request you are returning,
and then nothing would happen after it, it's just going to hang and the user won't know the heck just happend.

that's why in callbacks they take error first and always check if there's an error so it can handle it depending on how you told it to handle it.

ps: same goes to the archive api request

We need to work on mobile-first and html5 tags.

https://github.com/FACN3/Domain-Finder/blob/master/main.js#L18

why did you need this object ?? why not just define the functions?

please add labels for input tags
💛 💙 💜 ❤️ 💚 💔 💓 💗 💕 💞

please check for console errors
😄 😆 😊 😃 ☺️ 😏 😍 😘 😚 😳 😌 😆 😁 😉 😜