Hi guys! It’s haven’t logged in for a while.
They all look really nice. Just maybe a few changes
@toles04 Maybe center the picture and reduce the indentation when the screen gets smaller. You can use that bootstrap class to center or that css width + margin auto combo for the image, and there is a property for that list indentation that you can use with a media query.
@PaulNewman-M add that img-responsive class so the image shrinks with the screen and maybe use another font for the large block of text so it’s easier to read.
@aaminu same as @toles04 with the indentation on smaller screens.
@maxbevans try using percentages for the margins so it adapts when you change screen sizes and it doesn’t take up so much space. You can use it on the height and widths of the divs too so text doesn’t overflow. Later you can check the twitter bootstrap classes that kinda does the same thing by just adding one extra word. There are more options but these usually solve most of the cases. Also, the title is very small and its almost the same color as the background.
@JorneM same as @toles04 with the indentation and maybe set a minimum width because the title font is too large for small screens. You could also set the font size to something other than pixels so the font scales depending on the screen size. Usually using a fixed percentage (or other relative units) will give you some problems because then the object, in this case the font, ends up either too big or too small when resizing. Because of this you can use a function called calc. There you can set some upper/lower limits to this size. Google it, it has a formula. It is a bit confusing at the beginning because you know… math is like that.