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.

Sign up for GitHub

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

Jump to bottom

Develop #356

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

Develop #356

wants to merge 11 commits into from

Conversation

yanatoto
Copy link

loralevitska
loralevitska suggested changes Dec 22, 2023
View reviewed changes
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.

You did an amazing job! Looks good! Let's do some improvements (check comments)

  • remove outline, it breaks design
    image
  • disable textarea resize, it breaks design
    image
  • add hover effect to this links
    image
    image
  • it should be link with hover effect, by click open google map in new tab
    image
  • change to current year
    image
  • create links to socials
    image
  • for mobile mode stretch form to full screen
    image

SanyaBratashchuk
SanyaBratashchuk suggested changes Dec 23, 2023
View reviewed changes
Copy link

@SanyaBratashchuk SanyaBratashchuk 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 for images

VitaliyBondarenko1982
VitaliyBondarenko1982 approved these changes Dec 26, 2023
View reviewed changes
Copy link

@VitaliyBondarenko1982 VitaliyBondarenko1982 left a comment

Choose a reason for hiding this comment

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

Looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VitaliyBondarenko1982 VitaliyBondarenko1982 VitaliyBondarenko1982 approved these changes

@loralevitska loralevitska Awaiting requested review from loralevitska

@SanyaBratashchuk SanyaBratashchuk Awaiting requested review from SanyaBratashchuk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yanatoto @VitaliyBondarenko1982 @loralevitska @SanyaBratashchuk