-
Notifications
You must be signed in to change notification settings - Fork 299
chore(developer-hub): reward calculator and image import in ois #3057
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
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
|
98ff252
to
3a9689c
Compare
const RewardSimulator: React.FC = () => { | ||
const [publisherStake, setPublisherStake] = useState(200); | ||
const [delegatorStake, setDelegatorStake] = useState(300); | ||
const [maxCap, setMaxCap] = useState(500); | ||
const [delegatorFee, setDelegatorFee] = useState(20); | ||
const [rewardRate, setRewardRate] = useState(10); | ||
|
||
const [publisherReward, setPublisherReward] = useState(0); | ||
const [delegatorReward, setDelegatorReward] = useState(0); | ||
const [publisherRewardRate, setPublisherRewardRate] = useState(0); | ||
const [delegatorRewardRate, setDelegatorRewardRate] = useState(0); | ||
|
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.
this file is referred from the reward simulator of the documentation.
import OISSlashingInvestigationDark from "./images/ois_slashing_investigation_dark.png"; | ||
import OISSlashingInvestigationLight from "./images/ois_slashing_investigation_light.png"; | ||
import OISSlashingPoolPreDark from "./images/ois_slashing_pool_pre_dark.png"; | ||
import OISSlashingPoolPreLight from "./images/ois_slashing_pool_pre_light.png"; | ||
import OISSlashingPoolPostDark from "./images/ois_slashing_pool_post_dark.png"; | ||
import OISSlashingPoolPostLight from "./images/ois_slashing_pool_post_light.png"; |
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.
using image import and moved the images from the public folder inside the folder where they are used.
apps/developer-hub/src/components/RewardSimulator/index.module.scss
Outdated
Show resolved
Hide resolved
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.
Most important change is to remove the misuse of effects
apps/developer-hub/src/components/RewardSimulator/index.module.scss
Outdated
Show resolved
Hide resolved
apps/developer-hub/src/components/RewardSimulator/index.module.scss
Outdated
Show resolved
Hide resolved
apps/developer-hub/src/components/RewardSimulator/index.module.scss
Outdated
Show resolved
Hide resolved
b640939
to
84350cc
Compare
This reverts commit 84350cc.
$font-family: ( | ||
"monospace": ( | ||
"ui-monospace", | ||
"sfmono-regular", | ||
"consolas", | ||
"Liberation Mono", | ||
"menlo", | ||
"monospace", | ||
), | ||
); | ||
|
||
@function font-family($family) { | ||
@return map-get-strict($font-family, $family); | ||
} |
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.
I think this is great but at some point we should probably have a specific font we ship for mono text, cc @partyparrotgreg not sure if you already have one in mind
Summary
Rationale
How has this been tested?