@legion40216
Posted
Good job on this challange, i would like to point some improvement
1.remove the width property of 1150px instead let the flex streach and shrink accourding to the size of the view port for better responsiveness on testmoninal-box and add property of flex-wrap to wrap
2.I liked your implementation of margin: 28em auto in .text-service i was trying myself to find another solution by doing it without using postion: absolute and good use of custom classes
3.You should add background-position: center in your service section images with background-repeat: no-repeat this will inhance responsiveness
4.You should reduces the padding of the left and right of text-about classes using calc or min max properties for responsiveness using vw or vh units
Overall well done loved the detailed thought you put on it
@brokenShinobi
Posted
@legion40216 really appreciate that you took some time to point out my mistakes and also how I can improve them. The site has some issues while displaying in small screens and I'm looking into fixing them. Being mostly a backend guy, css is very new to me so I'm taking baby steps. Thanks again Suleman :)
