-
Notifications
You must be signed in to change notification settings - Fork 131
Project 7 - GitHub tracker - EmmaBerg #106
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
base: main
Are you sure you want to change the base?
Conversation
…anch and a link to the GitHub repo
Added a link to Netlify
There was a problem hiding this 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> |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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.
No description provided.