Skip to content

solution#741

Open
andreira-chilin wants to merge 4 commits intomate-academy:masterfrom
andreira-chilin:develop
Open

solution#741
andreira-chilin wants to merge 4 commits intomate-academy:masterfrom
andreira-chilin:develop

Conversation

@andreira-chilin
Copy link
Copy Markdown

Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, but seems like your page has not been deployed correctly, check the DEMO Link

Copy link
Copy Markdown

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

good job! 👏

here are some improvements:

  1. also add some hover effect for these navigation items. Make sure that you have added transition for your effects
Image
  1. add some content container
Image

Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, check next:

  1. All phone buttons should initiate a call.
  2. Add some hover effect like transition to all pictures.

Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

great job!

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.

4 participants