Skip to content

Dylan Harris Project Reddit Submission#234

Open
DylanHarris12 wants to merge 1 commit intoprojectshft:masterfrom
DylanHarris12:master
Open

Dylan Harris Project Reddit Submission#234
DylanHarris12 wants to merge 1 commit intoprojectshft:masterfrom
DylanHarris12:master

Conversation

@DylanHarris12
Copy link

No description provided.

<html>
<head>
<title>Rereddit</title>
<link href="http://maxcdn.bootstrapcdn.com/bootstrap/3.2.0/css/bootstrap.min.css" rel="stylesheet">

Choose a reason for hiding this comment

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

very old version

<div class="posts">
</div>

<form style="margin-top:30px;" onsubmit="event.preventDefault();">

Choose a reason for hiding this comment

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

event.preventDefault should be on JS file

Comment on lines +11 to +12
var name = document.getElementById('name').value;
var message = document.getElementById('message').value;

Choose a reason for hiding this comment

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

too generic, be more specific, its confusing with the comment later

Comment on lines +4 to +6



Choose a reason for hiding this comment

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

watch your spacing

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

Comments