Skip to content

Helenas-github-tracker#14

Open
HelenaW86 wants to merge 7 commits into
Technigo:mainfrom
HelenaW86:main
Open

Helenas-github-tracker#14
HelenaW86 wants to merge 7 commits into
Technigo:mainfrom
HelenaW86:main

Conversation

@HelenaW86

Copy link
Copy Markdown

No description provided.

Comment thread code/script.js
drawChart(forkedRepos.length);
});
};
//Getting information from my github acount API

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your comments are very helpful

Comment thread code/script.js
Comment on lines +92 to +97
//Show the comment when clicking on the button
function toggle() {
this.classList.toggle("active");
}
document.getElementById(`my${repo.name}`).onclick = toggle;
});

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very nice Helena!

@themisk84 themisk84 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Excellent job Helena! You manipulated well the data from the github API and your page is both in functionality and styling great!Keep up the good job!

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