Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TTT #145

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

TTT #145

wants to merge 2 commits into from

Conversation

JoseDiazCodes
Copy link

(1 to 5, but not 3) I completed the challenge: 4.
(1 to 5, but not 3) Feeling good about my code: 4.
Got any specific feedback for me?

So, I started off using a bunch of stuff I found on Google, like .bind() and trying to get all OOP with it. But honestly, the whole .bind() thing was tripping me up, so I went back to what I knew and it made a lot more sense. I had a chat with Leon, and he helped me see that it's okay to be a beginner and take things step by step.

I'm still wrapping my head around the whole this keyword and constructors thing. I ran into this weird issue where my event listener was just spitting out 'undefined' when I tried using the this keyword in the function. Super weird.

I didn't go all out on the styling part (hence why the 4 and not a 5) , mainly because I wanted to really get the JavaScript down. I pulled in some stuff we did in class, but I could definitely use more practice, especially with constructors and how to use the 'this' keyword in real web apps.

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.

1 participant