Responsive CSS masterpiece

Community feedback

• 

  Tom Roche•330

  @TomrocheDevPosted 4 months ago

  Hi Robert,

  Great take on this project! You asked for feedback on a more simple way of writing code. First of all, your code is very consistant in my opinion, so you are doing a great job.

  A tip I could give you is to use more explainatory classes. In your code I read a lot of classes like "lefttextbox" and "lefttextbox1". That is something I personally would avoid, because if you want to change something, it could get hard to find the exact element you want to select (especially on bigger projects). The chronology could also be disrupted when you, for instance, add a new element in between two elements that are named for instance "class1" and "class2". What I would recommend is try to use more classes that discribe what the element is or does. It makes it a lot more readable and professional. I also would recommend to use dashes between words in classnames. I would write "lefttextbox" as "left-text-box". That is more readable.

  For positioning of elements I would recommend to do some research on flexboxes and grids. These technologies are great for the positioning of elements. At first it can be hard to understand, but give it some time. It will help you A LOT with postioning.

  Resources for flexbox and grid: https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_grid_layout https://developer.mozilla.org/en-US/docs/Web/CSS/flex https://www.youtube.com/watch?v=rg7Fvvl3taU -- (A video by Kevin Powell -- This is also a great channel to follow on YouTube)

  Hope this helps!

  Marked as helpful

  0