bryansaylor / homework-portfolio

0 stars 0 forks source link

Feedback on assignment #4 #4

Open barryross opened 7 years ago

barryross commented 7 years ago

FEWD Week #4 Project: JavaScript Basics


Description

The team from Relaxr is back...again! They want their page to be more interactive and asked you to add JavaScript to their source code. You've been given the HTML and CSS but will need to create a JavaScript file and add a few different pieces of interactivity: Users should be able to click a link and have the remainder of the blog's content slide down and appear on the page, both on the main content column and the side bar; users should also be able hide the content when they are finished reading. See a detailed technical explanation below.

Student: Bryan Saylor

Project URL: https://github.com/bryansaylor/homework-portfolio

Technical Requirements Does Not Meet Expectations (0) Meets Expectations (1) Exceeds Expectations (2)
Use in-line-block or floats in your CSS to achieve a two-column layout- Prevent a form submission with the event.preventDefault() function x
Use the $.ready() handler to delay your code from executing until all DOM assets have been loaded x
Select the appropriate DOM elements with CSS selectors upon a user's click using the $.click() handler x
Have the text in the <p>tag slide down along with a "Read Less" link in the blog post using,$.slideDown() and $.show() x
Hide the "Read More" link using $.hide() x
Have the <p> slide up and hide the "Read Less" link using $.slideUp() and $.hide() x
Show the "Read More" link using $.show() x
Using the same functions as above, if a user clicks the "Learn More" link in the sidebar, have the '' inside that '

' slide down and hide the "Learn More" link using $.slideDown() and $.hide()

x
TOTAL: 16/16
barryross commented 7 years ago

Bryan, awesome work with this! Excellent job hitting the tech requirements and finding a solution for the "jerky" slideUp() issue you were describing. That's the solution I was going to suggest...using a callback in conjunction with the slideUp() method...excellent work identifying and tackling a problem/something that was bugging you about the UX 💯 . Please see inline comments for additional feedback and keep up the great work!