Skip to content

project-github-tracker-justine#115

Closed
JustineZwiazek wants to merge 12 commits intoTechnigo:mainfrom
JustineZwiazek:main
Closed

project-github-tracker-justine#115
JustineZwiazek wants to merge 12 commits intoTechnigo:mainfrom
JustineZwiazek:main

Conversation

@JustineZwiazek
Copy link
Copy Markdown

No description provided.

Comment thread code/index.html
</head>
<body>
<div id="profile" class="profile"></div>
<img src="images/bag.png" alt="a bag of repos" class="img-bag" />
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LOVE this. Did you create it yourself?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ah thanks! I made it in Canva!

Comment thread code/index.html
Comment on lines +21 to +23
<div class="wave"></div>
<div id="repoGrid" class="repo-grid"></div>
<div class="wave"></div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Also this, so creative!

Comment thread code/script.js
@@ -0,0 +1,78 @@
// DOM selectors:
const repoGrid = document.getElementById("repoGrid");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I like how you name the variables. Easy to read!

Comment thread code/script.js
const API_REPOS = `https://api.github.com/users/${userName}/repos`;

// Profile information function:
const getProfile = () => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Clean code, mine were a lot messier haha. Great job!

Comment thread code/script.js
getRepos();
};

// Fetch repositories function:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great with the comments

Comment thread code/script.js
});
};

// Fetch pull requests function:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I had a hard time with this one. Yours was a lot easier to read! Thanks for clearing things up in my head with the comments

Comment thread code/script.js
)
.then((res) => res.json())
.then((data) => {
const myPullsOnly = data.find(
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good name for variable

Comment thread code/style.css
body {
background: #FFECE9;
} 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.

Great with *, I haven't added that in mine, I always use body for that. Thanks!

Comment thread code/style.css

/* Profile information: */
.profile {
background-color: pink;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Love this color hehe

Comment thread code/style.css
background-position: 0;
background-size: auto 100%;
background-repeat: repeat-x;
animation: animateCloud 20s linear infinite;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Such a cool animation! Where did you find the wave-picture?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Canva again :)

Comment thread code/style.css
}

/* Tablet */
@media screen and (min-width: 600px) and (max-width: 992px) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice clean code in here!

Comment thread code/style.css
object-fit: cover;
animation: shake 2s infinite;
}
@keyframes shake {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wow this was new to me!

Copy link
Copy Markdown

@emmajosefina emmajosefina left a comment

Choose a reason for hiding this comment

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

I adore your page, Justine! It looks very professional. You have made a page that's functional, responsive and fulfills all the requirement for blue level. Also I love your repo bag and animation, really great job! So impressed by your performance this week :)

@JustineZwiazek
Copy link
Copy Markdown
Author

Thank you for your kind words Emma!

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