Skip to content

Portfolio Lab#1

Open
Denae716 wants to merge 2 commits into
Return-Ready-2021-JavaScript-Evening:mainfrom
Denae716:main
Open

Portfolio Lab#1
Denae716 wants to merge 2 commits into
Return-Ready-2021-JavaScript-Evening:mainfrom
Denae716:main

Conversation

@Denae716
Copy link
Copy Markdown

No description provided.

@kburd kburd self-requested a review October 29, 2020 14:11
Comment thread page.1.txt/testing.txt
@@ -0,0 +1 @@
testing pg 1
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Only commit important files. This text file doesn't have any value

Comment thread pages.html/pg1.html
<header>
<a href="pg2.html">Contact</a>
<h1>Denae Brown</h1>
<img src="C:\Users\denae\Portfolio-Lab\page.1.HTML\Me.JPG" alt="photo of me"/>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Image did not load correctly, you used an absolute file path. If you used a relative one like Me.JPG it would have loaded on mine as well as yours (-1)

Comment thread pages.html/pg2.html
<h1>Contact Me</h1>
<hr>
<label for="firstname">First Name</label><br>
<input required type="text" id="firstname" name="firstName"> <br><br>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing 30 character limit (-2)

Comment thread pages.html/pg2.html
<input required type="text" id="firstname" name="firstName"> <br><br>

<label for="lastname">Last Name </label><br>
<input required type="text" id="lastname" name="lastName"> <br><br>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing 30 character limit (-2)

Comment thread pages.html/pg2.html
<button>Submit</button>

</body>
</html> No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing How did you hear about us dropdown (-5)

80/100
Great work! My only recommendation is to make sure you do everything the lab asks for. You missed some points on silly mistakes.

Comment thread pages.html/pg2.html


<label for="interest">Skills of Interest</label>
<select>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be checkboxes instead of dropdown (-3)

Comment thread pages.html/pg2.html


<label for="comments">Comments</label><br>
<textarea name="comments" cols="50" row="10" maxlength="300"></textarea> <br>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be a max length of 250, not 300 (-1)

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