design for 1600px

[KolodnytskaV]

I finished design for 1600px but I have some problems with @media and import in main.css

[etojeDenys]

if you have any issues, feel free to ask for help in the chat, because it's hard to communicate here

[etojeDenys]

demo link does not work. please fix it and re-request the review

[vadiimvooo]

There are still no demo link in the description of PR and deploy. Please feel free to ask for help in fe_chat.

[2pasha]

good job! 👏

here are some improvements:

1. fix moments from fe_chat
2. these links don't work

3. add hover effects for your elements like these

4. it's better to use icons in svg format with the same style and you may add some hover effect

5. add form validation

• all fields should be required
• add email validation
• after submitting form form should be cleaned, without any side effects
  to implement these you may use this:

<form onsubmit="this.reset(); return false;">

6. tablet and mobile views look broken

[KolodnytskaV]

I can't fix a problem with the slider in the header. I also have a problem with smooth transitions between different screen sizes.

[brespect]

Almost done, but this header swiper looks broken, if you have any questions you may write in fe_chat

[Anton-Kuchmasov]

Almost done!

Please fix CSS Autofill styles in your Form
no additional styles (e.g. background-color or whatever else changes) allowed

[KolodnytskaV]

I resolved this problem, but I still have a problem with image in header

[etojeDenys]

almost done

1. all fields should be required

2. these should be links

3. centre this section

[etojeDenys]

1. there is still a problem with steps 1 and 3 from the previous review

2. fix the style of the links (remove base underline and text color)

[KolodnytskaV]

can't center testimonials_cards and fix a problem with image in header

[brespect]

Please provide the DEMO LINK in Pull Request description, and make sure it is working, if you have any problems ask in fe_chat

[Anton-Kuchmasov]

DEMO_LINK: https://kolodnytskav.github.io/layout_dia/

hey, to proceed with your landing page, you should make some changes:

1. Centering the Testimonials cards: use justify-content: center for the testimonials__cards div:

2. Fixing the layout of Slider:

2.1 Add margin-top on div with class container strategic__container with correct value (retreive it from Figma mockup - i've added 100px as an example):

2.2. Add correct width to div with strategic__image class. I've added 50%, but again - you should check the right value in Figma

2.3. Add border-top-left-radius: 200px to slightly round your Slider. Do it again with with .header to style background of your border (border-bottom-left should work)

2.4. Disable width: 400% on .slides div

2.5. Make section class ="strategic" as relative (just add position: relative).
Let's position container strategic__container div:

3. Analyze further styles to make your solution perfect.
   You can use this solution as a reference

[Denys-Kravchuk9988]

Good job!

A few things to improve:

1. It's better to align slider according to the design

2. I would recommend to remove default button's border width

3. Part of the visit us text is hidden

4. It's better to export icons and logos in .svg format

[etojeDenys]

almost done

1. now is prone version looks a bit broken. adjust these sections for the phone version

[brespect]

Almost done, but i think:

1. this text should be inside header image:

2. Check the swiper buttons, as for me, not working properly

[brespect]

Lgtm 🚀