hey @Harshitsoni2000 well done on what I am guessing is your first challenge? š
I think you did a good job on this one and the design looks good and responds well at different viewport sizes. Here's some small suggestions for you to consider:
-
Try to avoid adding inline styles in your HTML. They are difficult to override and it's generally not a good idea. In this case you could just create a class called 'accent' or something and apply that to your
span
. This makes things easier to change later too. -
Don't forget the alt text on your images is meant to be read aloud by people using screen readers or to show up if the images fail to load for some reason. Therefore the text should really describe the image something like "A group of colleagues working and having fund together" might work in this case.
-
Similar to point #1 above, if using capitals try to do this through the CSS
text-transform
property rather than actual capitals in the HTML. Again this is just a good habit and makes it easier to swap things around later on.
These are all small points but should help you as you carry on with the challenges. Good luck and great work here! š Cheers š
@Harshitsoni2000
Posted
THANK YOU VERY MUCH. I WILL WORK ON THESE ERRORS AND HOPE YOU WILL FURTHER ASSIST ME IN MY UPCOMING PROJECTS ON FrontEnd Mentor. I really needed someone to guide me on these projects.