Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revamp calculator #37

Merged
merged 3 commits into from Aug 29, 2019
Merged

Revamp calculator #37

merged 3 commits into from Aug 29, 2019

Conversation

hendricius
Copy link
Owner

Over time I have been enjoying sourdough pizza more and more. However the calculator was not initially set up to cope with sourdough. This PR adapts the calculations for the sourdough.

The calculator has been split in 2 sections:

  1. Yeast or Fresh yeast
  2. Sourdough

image

We still need to fix the initial page of the calculator. It should be 2 links to the page, passing the parameters from the form 馃檲. I've not worked that much with react before if someone else is interested, please give it a shot.

image

Over time I have been enjoying sourdough pizza more and more. However
the calculator was not initially set up to cope with sourdough. This PR
adapts the calculations for the sourdough.

The calculator has been split in 2 sections:

1) Yeast or Fresh yeast
2) Sourdough
Replaces the button to calculate with links on the homepage
@hendricius
Copy link
Owner Author

Update - I adapted the design:

image

Would be good to have a review by someone with more experience.

@hendricius hendricius merged commit 00861e2 into master Aug 29, 2019
@hendricius hendricius deleted the revamp-calculator branch August 29, 2019 16:07
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.

None yet

1 participant