Skip to content

Develop#547

Open
ruslan-kasinets wants to merge 12 commits intomate-academy:masterfrom
ruslan-kasinets:develop
Open

Develop#547
ruslan-kasinets wants to merge 12 commits intomate-academy:masterfrom
ruslan-kasinets:develop

Conversation

@ruslan-kasinets
Copy link
Copy Markdown

Copy link
Copy Markdown

@Denys-Kravchuk9988 Denys-Kravchuk9988 left a comment

Choose a reason for hiding this comment

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

Excellent job!

A few things to improve:

  1. I would recommend to add cursor: pointer; when user hovers on interactive elements on the page. For example, these interactive elements
Image Image Image Image
  1. The upper line in animation is hidden
Image
  1. For better quality of images I would recommend to export them in 2x scale from Figma and convert them to .avif or .webp format
Image Image
  1. I would recommend to remove autofill styles
Image

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.

lgtm 🔥

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