Skip to content

Github tracker by Isabell Wästfelt#80

Open
isabellwastfelt wants to merge 11 commits into
Technigo:mainfrom
isabellwastfelt:main
Open

Github tracker by Isabell Wästfelt#80
isabellwastfelt wants to merge 11 commits into
Technigo:mainfrom
isabellwastfelt:main

Conversation

@isabellwastfelt

Copy link
Copy Markdown

No description provided.

Comment thread code/script.js
.then((data) => {
console.log(data);
profileInfo.innerHTML += `
<img src=${data.avatar_url}' alt='image of isabellwastfelt at GitHub'>

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 way to fetch the image and profile info

Comment thread code/script.js
};
getProfile(); //invoking

const getRepos = () => {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Impressive job making this part so clean and simple.

Comment thread code/script.js
getRepos();

// Pull requests for each project
const getPullRequests = (repos) => {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same with this part, but maybe even more so. I know how tricky this was and you made it look really simple. Super easy to follow. WELL DONE!

Comment thread code/style.css
-webkit-box-sizing: border-box;
-moz-box-sizing: border-box;
-webkit-font-smoothing: antialiased;
text-rendering: optimizeLegibility;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Learned a new thing here

@HWallberg HWallberg 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.

Since I know how hard this was, I'm really impressed by your javascript file! You made it look really simple and it was easy to follow the code.
Nice use of grid and the responsive design! Awesome that you manage to spend some time on the css. It looks really well nice.
If there was time you could work a bit more on the grid to maybe make the boxes not stretch over the entire page when using a big screen.

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