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

Suggestions for Level 3 #5

Merged
merged 1 commit into from
Nov 4, 2016

Conversation

ajhyndman
Copy link
Contributor

No description provided.

//we will get text '@NodeGirlsSydney'

ourTwitter.textContent;
// we will get text '@NodeGirlsSydney'
Copy link
Member

Choose a reason for hiding this comment

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

we will get text "Twitter: @NodeGirlsSydney @NodeGirlsMelb" as we had updates in html

// kittens. Let's create a new image that sources our original logo file, and
// put it into header.
//
// P.S. Wou also can give styles to the new node that you create.
Copy link
Member

Choose a reason for hiding this comment

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

mistype You also ...

@ButenkoT
Copy link
Member

ButenkoT commented Nov 4, 2016

Thank you a lot for polishing instructions! I very appreciate your help :)

@ajhyndman
Copy link
Contributor Author

@ButenkoT I think I fixed your two suggestions.

I'm glad you found it useful. :)

@ButenkoT
Copy link
Member

ButenkoT commented Nov 4, 2016

Those changes were right in time, as we run one event tomorrow.

@ButenkoT ButenkoT merged commit d10f8ff into muses-code-js:master Nov 4, 2016
@ajhyndman
Copy link
Contributor Author

Oh, I didn't even realize that.

I hope it goes great!

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