Skip to content

initial commit#228

Open
builtbyteran wants to merge 12 commits intoprojectshft:masterfrom
builtbyteran:master
Open

initial commit#228
builtbyteran wants to merge 12 commits intoprojectshft:masterfrom
builtbyteran:master

Conversation

@builtbyteran
Copy link

No description provided.

root/index.html Outdated
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>simple weather application</title>
<link rel="stylesheet" href="style.css">
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0-alpha.6/css/bootstrap.min.css" integrity="sha384-rwoIResjU2yc3z8GV/NPeZWAv56rSmLldC3R/AZzGRnGxQQKnKkoFVhFQhNUwEyJ" crossorigin="anonymous">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this version? is alpha

root/index.html Outdated
Comment on lines 23 to 24
</div>
</form>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check your alignment

index.html Outdated

<div class="container">
<div class="forecast row">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spacing and alignment

root/main.js Outdated
weekday: 'long'
});

var dayOne = {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all these could have a loop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants