Krushna Sinnarkar• 1,080
@krushnasinnarkar
Posted
Hi @nj93777,
Congratulations on successfully completing the challenge!
Your solution looks nice, though there are a couple of things you can improve, which I hope will be helpful:
-
Large Gap Between Sections:
- There is a large gap between the image section and the aside. You can solve this issue by removing the
flex: 1property from the main section element. - Also, change the
justify-content: space-between;property tojustify-content: center;. This will prevent the image section from growing and keep both sections centered.
- There is a large gap between the image section and the aside. You can solve this issue by removing the
-
Stretched Layout on Large Screens:
- Add
max-width: 1440px;to the body element to avoid stretching on large screens. - Use
margin: 0 auto;to keep the content centered.
- Add
-
Carousel Button Alignment:
- The carousel buttons are not properly aligned for screen sizes between 900px to 650px. Adjust their
leftandrightproperties to ensure they are perfectly aligned within this range.
- The carousel buttons are not properly aligned for screen sizes between 900px to 650px. Adjust their
I hope you find this helpful.
Feel free to reach out if you have more questions or need further assistance.
Happy coding!
Marked as helpful
0
nj93777• 240
@nj93777
Posted
@krushnasinnarkar
It was really helpful! I made the changes.
1
