Skip to content
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

Eco #357

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Eco #357

wants to merge 2 commits into from

Conversation

ehordyenko1
Copy link

@ehordyenko1 ehordyenko1 commented Dec 22, 2023

Copy link

@loralevitska loralevitska left a comment

Choose a reason for hiding this comment

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

Looks like your styles didn't deploy, please fix it and re-request review
image

Copy link

@l4st1m0za l4st1m0za left a comment

Choose a reason for hiding this comment

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

Still same problem with styles. Please write in FE chat, if you have any issues

image

Copy link

@l4st1m0za l4st1m0za left a comment

Choose a reason for hiding this comment

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

  1. Add hover effects to all the interactive elements on the page. (Logo link, menue links, insta and facebook logos, anything on you can click and something will happen)

  2. Also make all animations smooth with the help of time in transition

  3. This button, should lead to the contacts

image
  1. Remove autofill styles from all the inputs. Here is a hint of how you can do it
image

Copy link

@anastasiiavorobiova anastasiiavorobiova left a comment

Choose a reason for hiding this comment

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

Great work!
Screenshot 2024-04-28 at 18 00 22
Screenshot 2024-04-28 at 18 00 37
Check alignment here. Also, the page has a horizontal scroll on mobiles. Please, add hovers for interactive elements in the footer and fix the mailto link https://css-tricks.com/snippets/html/mailto-links/
Screenshot 2024-04-28 at 17 59 35
Screenshot 2024-04-28 at 17 59 41
Check alignment here

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