Weather fcc project

Weather fcc project
0.0 0

#1

not bad for a newbie phew :sweat_smile:


#2

You are using this js code:

today = new Date();
var date =
	today.getFullYear() + '/' + (today.getMonth() + 1) + '/' + today.getDate();
var time =
	today.getHours() + ':' + today.getMinutes() + ':' + today.getSeconds();
var dateTime = date + ' ' + time;

but your code document.getElementById('clock').innerHTML = dateTime; does not work because you don’t have an element in html with that id.

Otherwise the app is functional, it seems responsive but you definitely may improve your design.


#4

A little easy tweak you might do to improve your design is changing the Celsius button color to white so it gets contrast. It almost hidden now.

Good job!


#5

I like it !

Some minor items to change, the spacing between miles and the number given, as well as spacing between the C/F button and the current temp.

:slight_smile:


#6

hope it is better now ! thanks a lot


#7

tysm! I fixed that. appreciate your feedback :smile:


#8

Thank you so much for your feedback, of course I had the clock id in the navbar I just needed to change the color.

hope my design gets better in future :slight_smile: