Skip to content

Homepage UI fix#481

Open
Aeesh wants to merge 8 commits intotrunkfrom
aisha
Open

Homepage UI fix#481
Aeesh wants to merge 8 commits intotrunkfrom
aisha

Conversation

@Aeesh
Copy link
Copy Markdown

@Aeesh Aeesh commented Apr 19, 2021

No description provided.

Comment thread package.json Outdated
"scripts": {
"start": "REACT_APP_VERSION=$(git describe --abbrev=0) REACT_APP_GIT_SHA=$(git rev-parse --short HEAD) react-scripts start",
"serve": "REACT_APP_VERSION=$(git describe --abbrev=0) REACT_APP_GIT_SHA=$(git rev-parse --short HEAD) BROWSER=none react-scripts start",
"serve": "react-scripts start",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please revert this change

Comment thread src/components/Home/Banner/index.tsx Outdated
<Text>Blockchain Analytics Tool </Text>
<LogoCon>
<Text> Brought to you by </Text>
<img src={CryptonomicLogo} alt="img"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

set alt attribute to something like Brought to you by Cryptonomic


<BottomDiv>
<Title2>Discover Arronax</Title2>
<img src={DownArrow} alt="img"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

alt, same here if you can

Comment thread src/containers/Homepage/index.tsx Outdated
<img src={PlaceholderImage1} alt="img"/>
<AnchorTag className={classes.alignRight}>
<Link to={topBakersUrl}>
<AnchorTag style={{marginTop: '-7px'}} className={classes.alignRight}>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please set custom styles dynamically in AnchorTag or add in classes

Comment thread src/containers/Homepage/index.tsx Outdated
<AnchorTag>
<Link to={{ pathname: "https://periscope.arronax.io/blocks?q=priorityBlocks"}} target="_blank" >
View Graph on Periscope
<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if possible please save svg as file in images and import to component

Comment thread src/containers/Homepage/index.tsx Outdated
<AnchorTag style={{marginTop: '-7px'}} className={classes.alignRight}>
<Link to={{ pathname: "https://harpoon.arronax.io/"}} target="_blank" >
View Baker Stats on Harpoon
<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same svg

Comment thread src/containers/Homepage/index.tsx Outdated
<AnchorTag>
<Link to={{ pathname: "https://periscope.arronax.io/accounts"}} target="_blank" >
View Graph on Periscope
<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same svg

Comment thread src/containers/Homepage/index.tsx Outdated
</Grid>
<Grid item xs={6}>
<img src={PlaceholderImage1} alt="img"/>
<img style={{marginBottom: '-18px'}} src={PlaceholderImage1} alt="img"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please pass margin bottom in class to avoid inline styles

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