-
Notifications
You must be signed in to change notification settings - Fork 78
Solution #32
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
base: master
Are you sure you want to change the base?
Solution #32
Conversation
danheim
left a comment
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.
Пересоздай в ветку master своего репозитория
irenhh
left a comment
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.
fix paddings for inputs in the header, they have to be equal for the left and right sides
logo should be a link
|
logo is a link to the main page |
irenhh
left a comment
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.
- make the language in the footer be available to choose
- add some hover effect to the footer links
- you marked the 5th task as done but I don't see restaurant page on your website
- and the page is not adaptive
Please, mark those tasks done that are really done
Link
Сторінка повністю адаптивна, додані ховер ефекти, логотипи мають посилання, падінги виправлені, константа для посилання додана, мови в селекті доступні для вибору.