Skip to content

Develop#707

Open
VadimBatsko wants to merge 10 commits intomate-academy:masterfrom
VadimBatsko:develop
Open

Develop#707
VadimBatsko wants to merge 10 commits intomate-academy:masterfrom
VadimBatsko:develop

Conversation

@VadimBatsko
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@vitalii-bondarenko-ua vitalii-bondarenko-ua left a comment

Choose a reason for hiding this comment

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

Provide demo link to PR description

Copy link
Copy Markdown

@vadiimvooo vadiimvooo left a comment

Choose a reason for hiding this comment

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

There is still no link for your deploy, please add it to your PR description. There are some things to fix. Please feel free to ask for help in fe_chat.

  1. Font in Header has to be larger, please check mockup
Image 2. Same for button Image 3. The section background should span the entire width of the page Image 4. Same here Image 5. Menu items has wrong font size and in general entire menu should be centered. Check mockup Image Image 6. On tablet size and lower it seems that menu is completly broken Image 7. There is some space between corner of page and image. It should be fixed. Image

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.

3 participants