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
architecture(web): init #7
Conversation
web/README.md
Outdated
@@ -1,3 +1,17 @@ | |||
# CKB Explorer Web | |||
# introduction |
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.
introduction
to Introduction
web/README.md
Outdated
CKB block explorer web | ||
# Build Setup | ||
|
||
prerequisite: nodejs |
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.
Prerequisite
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.
better to indicate version of NodeJS
web/README.md
Outdated
|
||
yarn i | ||
|
||
yarn start // for development |
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.
comments in shell should start with #
web/src/App.tsx
Outdated
@@ -0,0 +1,55 @@ | |||
import React, { useContext, useEffect } from 'react' | |||
import Routers from './routes' | |||
import Loading from './components/loading' |
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.
Component File should be named PascalCase
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.
Agree. Component name should be PascalCase
web/src/App.tsx
Outdated
appContext.resize(window.innerWidth, window.innerHeight) | ||
} | ||
window.addEventListener('resize', resizeListener) | ||
// appContext.toastMessage('hello', 12999933) |
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.
meaningless comments
web/src/components/loading/index.tsx
Outdated
return ( | ||
<LoadingDiv className="loading"> | ||
<div | ||
className="loading_content" |
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.
class name should follow the camelCase
or BEM
link
web/public/manifest.json
Outdated
@@ -0,0 +1,15 @@ | |||
{ | |||
"short_name": "React App", | |||
"name": "Create React App Sample", |
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.
short_name
and name
should be CKB Explorer
feat: change nonce precision to 50
No description provided.