-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
Add comparison to single emission screen #68
Comments
Display the tree calculation like on the picture. The number of trees should be in blue / bold. |
Hello my friend! can i take this issue to do? |
should the content be displayed in the app the same way as the image, or should it just have the same information? |
Sorry for taking so long to answer, at the end of this week I will finish this task Thanks a lot to updated this design! About this task, I'll go deeper into it during the week and talk to you if I have any questions. |
Add comparison to single emission screen
Use shift project data for calculation of the charge smartphones.
In
EmissionItemScreen.tsx
add a component like this<Comparison emission={emission} />
before theButton.Primary
If it's a emission by car, do not display the comparison with car.
Add a line after the comparison with charge smartphone (not visible on the picture) saying :
if(emissionModelType === shortHaulFlight || emissionModelType === mediumHaulFlight || emissionModelType === longHaulFlight ) {
You could have decrease your carbon footprint by XX % if you had use a train
}
if(emissionModelType === car ) {
You could have decrease your carbon footprint by XX % than if you took a train
}
if(emissionModelType === bus ) {
Good job, you saved XX kgCO2 eq by not driving a car
}
if(emissionModelType === train ) {
if(value < 400){
Good job, you saved XX kgCO2 eq by not driving a car
} else {
Good job, you saved XX kgCO2 eq by not flying a plane
}
}
This is a big task. Use ramda whenever it's possible, and add tests to this Comparison "local" component (local meaning that I don't think it needs to live in
app/components
because I don't see where I could be used anywhere else).The text was updated successfully, but these errors were encountered: