Skip to content

Conversation

@lario0913
Copy link

Completed The restaurant info page, menu page and review page , Please Kindly review and drop your comments
Screenshot_8
Screenshot_9
Screenshot_7

Joggz and others added 30 commits October 21, 2019 10:52
- uses create-react-app
- uses bootstrap as a css framework
- uses sass preprocessor for styling
- installed radium incase of any inline styling
- adds components folder in src folder, all Components created should sit in this folder
- adds assest folder for assest like images to be used in the project
- adds container folder.
- adds README
- changes the import directory
made a change on line 1
- component and assest folder did not appear on github on my first push
- each component should contain its own scss file
- adds prop type validation to component
adds reusable FormInputs and buttons
- changes Footer/Footer.js to Footer/index.js
- adds Footer component in the app.js for testing purpose
Copy link

@Mlg29 Mlg29 left a comment

Choose a reason for hiding this comment

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

Boss. I went through your branch now, but i didnt see any of your page. I only see the navigation, auth nav and searchbar in the app component. cant find any of your task. Kindly look into it or Am i missing out on something

@Joggz Joggz self-requested a review January 22, 2020 10:39
Copy link

@Joggz Joggz left a comment

Choose a reason for hiding this comment

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

well done bro, i have a couple of feedback:

  • the text inside the should be in the middle of the button
  • the inputs of the food catergory has some padding and seem to be at the center of the box
  • the discount should be position absolutely on the black and white diagram, with the view offer button right below the diagram.

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.

9 participants