Skip to content

Password#19

Open
kenjelly22 wants to merge 3 commits intolearn-co-curriculum:masterfrom
kenjelly22:password
Open

Password#19
kenjelly22 wants to merge 3 commits intolearn-co-curriculum:masterfrom
kenjelly22:password

Conversation

@kenjelly22
Copy link
Copy Markdown

Practiced using event handlers in React

Comment on lines +4 to +6
if (event.currentTarget === event.target) {
console.log("Good!")
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No need for the conditional here:

Suggested change
if (event.currentTarget === event.target) {
console.log("Good!")
}
console.log("Good!")

Comment on lines +10 to +12
if (event.currentTarget === event.target) {
console.log("Hey! Eyes on 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.

Suggested change
if (event.currentTarget === event.target) {
console.log("Hey! Eyes on me!")
}
console.log("Hey! Eyes on me!")

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