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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Observer simple version #35

Open
wants to merge 63 commits into
base: master
Choose a base branch
from
Open

Observer simple version #35

wants to merge 63 commits into from

Conversation

brkristoffer
Copy link

let's hope for the best

nicolaa5 and others added 30 commits January 21, 2018 11:52
linked css file to index
just added them to select-dish page
- Added Sidebar / Navbar
- Mobile expansion
- Confirm DInner
- Searchfield
- Dropdown
- Change from Bootstrap 4 to 3
getFullMenu and getAllIngredients are now functioning. I used line 80 in exampleView to test the output.
Merging "Master" & "Dinner model" Branches
-Top left logo/homepage return
-Alignment & color changed
-Typography added
I replaced all the reference ID's in the HTML so this renaming works out.
nicolaa5 and others added 30 commits January 24, 2018 23:34
- Chosen recipes (hardcoded)
- Print recipe button + link
also made prevented the amount of guests to go below 0
- Added ingredient list and description
- Buttons
- href to pages
- Typography

Removed sidebar.js as it had no functionality
- Change of color in css
-Resizing images for mobile
- Marker to return to if we want to go back to Lab 1
This reverts commit c7c9929.

- I reverted this commit in order to do it the right way as specified at the end of lab 1
for example, this.guests was replaced with var guests
plus some minor syntax changes
- Changing names
- Changing references to names in the code

All the changed names are shown in the image here:
https://docs.google.com/document/d/1Z2MNflXovnR7FcYBNQHnqfbk_AHhWiDvL4lXIiYpnHk/edit
Implementation of the MVC structure  for changing the amount of guests. The "STEP" comments refer to the steps made in the "Model-View-Controller Overview and Timeline": https://kth.instructure.com/courses/4645/pages/model-view-controller-overview-and-timeline?module_item_id=62195
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