Skip to content

Conversation

@EmmaaBerg
Copy link

No description provided.

Copy link

@SofiePellegrini SofiePellegrini left a comment

Choose a reason for hiding this comment

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

Great job Emma! You fulfilled all the general requirements and I really like the design in the desktopversion. Looks good with the picture and the chart next to eachother. Also good commenting on the code!

<h3> <a href = ${repo.html_url}> ${repo.name}</a></h3>
<hr>
<ul>
<li> <p> Latest push: ${new Date(repo.pushed_at).toLocaleString('sv-SE', { dateStyle: 'short', })}</p> </li>

Choose a reason for hiding this comment

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

Great to work with a list here!

/* Global CSS variables for colors */
:root {
--primary: #F1FAEE;
--secondary: #A8DADC;

Choose a reason for hiding this comment

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

Good to work with the colors in this way instead of writing them every time.

fetch(API_URL_REPOS)
.then((resp) => resp.json())
.then((allRepos) => {
//A function for filtering out the forked projects from technigo.

Choose a reason for hiding this comment

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

Good comments on the code! Easy to follow and easy to understand.

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