Skip to content

W7 - Jin Cho#81

Open
jiiinCho wants to merge 16 commits intoTechnigo:mainfrom
jiiinCho:main
Open

W7 - Jin Cho#81
jiiinCho wants to merge 16 commits intoTechnigo:mainfrom
jiiinCho:main

Conversation

@jiiinCho
Copy link
Copy Markdown

Copy link
Copy Markdown

@Kyparn Kyparn left a comment

Choose a reason for hiding this comment

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

Awesome job Jiin, i just don´t what it all means yeat in your JS files :)

I just lookt over your css, hmtl chart and main js file, once again u did a awesome site, did just wanna know how many hours u spend on it :)

<!-- <script src="script.js" type="module" defer></script> -->
</head>
<body>
<div id="root"></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.

Clean hmtl 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.

The chart looks nice, tok some time to understand it all, maybe right some comments on how it works also :)

Comment on lines +2 to +18
/* colors */
--color-black: #0e1116;
--color-light-black: #171b21;
--color-superlight-black: #20242b;
--color-deep-grey: #21262c;
--color-grey: #353941;
--color-light-grey: #2a2f35;
--color-deep-white: #44474c;
--color-white: #f1f7fa;
--color-light-white: #8d949c;
--color-primary: #6ca3f8;
--color-blue: #6ca3f8;
--color-red: #d35635;
--color-yellow: #eee170;
--color-pink: #e8a5a9;
--color-violet: #5f2ec6;

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 with the root folder for all the color.

Comment on lines +1 to +8
import { createElement } from "../utility/createElem";

export default function LoadingStatusComp() {
const spinnerContainer = createElement("div", "spinner-container");
const spinner = createElement("div", "spinner");
spinnerContainer.appendChild(spinner);
return spinnerContainer;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Did u have a spinner on the site, did not se one :)

Copy link
Copy Markdown

@Kyparn Kyparn left a comment

Choose a reason for hiding this comment

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

Awesome work Jin :)

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