@emestabillo
Posted
Hey @grizhlieCodes, congrats on completing this project! I'm getting some text overlap on medium widths, starting at 420px or so, down on the services and testimonial section. Some of the text actually disappears right before the next breakpoint. A few more:
-
The CTA on your nav should be left as an anchor tag instead of a button wrapped in an
a
tag. -
For the images, I think there's no need to create several divs and hide/show depending on screen size. Look into
srcset
and maybe<picture>
tag to handle image changes. I'm actually not sure if the images in the services section are meaningful. I'd probably choose to keep them as background images. Then you wouldn't have to worry about grid and overlaps. -
The class nesting looks quite deep.
section.services article.service.graphic-design .text-container .service-heading
, for example, is very specific. Overriding it will be difficult. I suggest to nest sparingly, keeping classes at the root level and nest only pseudoclasses and pseudoelements.
Hope this helps!
Marked as helpful
@grizhlieCodes
Posted
@emestabillo Darn, you betcha this helps. As soon as I read that button
shouldn't be in <a>
I recalled reading this in the past lol, think it's invalid HTML according to the spec ๐
??
As regards to srcset I think I didn't like it recently since it's heavily based on width
. And then when a desktop image was actually of lower width than the mobile version it wasn't behaving how I wanted it to.
Perhaps I messed up the sizes
media query? Perhaps you could sprinkle how you'd approach that sort of problem too ๐ฌ?
Never used <picture>
so I'll look into it, thanks. Think that's another clue of my cluelessness of images in html.
"I'm actually not sure if the images in the services section are meaningful." -> Super interesting point, never really went that 'deep' into HTML images. I think you're right, I'll apply this logic in the next image-containing project, thank you ๐
As for nesting - normally I'd agree but for this particular project I didn't care (๐คซ) because of its' 1 page nature. My Designo solution was as you recommended, where deep-nesting would truly mess up a lot of the component-logic that Designo offers the developer to use. This project I wanted to spend time on images, so your feedback is invaluable for me!
Super useful feedback, thanks so much Emmilie, super happy to get image-based advice as for whatever reason 12months later I haven't spent enough time on them, go figure!
@emestabillo
Posted
@grizhlieCodes Hahaha don't beat yourself up. I also have a hard time deciding whether to include images in html or not. In this case, I think I can place the cherries and orange as background images because they're not really connected to Graphic design or Photography, like I can understand them well without those two images.
As far as changing images with <img>
and <picture>
(and even <figure>
), my idea is that if the images have different aspect ratios, I go for <picture>
. Otherwise, it's <img
with srcset
. Still gray on some things, I'm still learning just like you.
I'm gonna have to look at your Designo to get ideas once I'm brave enough to attempt it lol :-)
@grizhlieCodes
Posted
@emestabillo Oohhhh, alrighty, that makes a ton of sense to me now.
And Designo is awesome to do!! Still my favourite project, even comparing to the audiophile or invoice apps. I canโt recommend it enough for:
- Planning HTML and CSS/SCSS
- Thinking through components
- It was my first time using an API (leaf js for the maps).
- Really getting into making useful SCSS mixins. I still use many of them to this day.
- Managing layouts, especially the more โglobalโ ones across different pages. Like โbodyโ, โmainโ, etc.
Many more Iโm sure but I forgot ๐ฅฒ
I used designo to start thinking how to make components via HTML and SCSS. Reuse code etc. Also practices sass architecture which was a breath of fresh air ๐. Structure makes me happy.
Iโll definitely attempt a few demos in codepen to understand how you use โimgโ and โpictureโ, I like that you have a distinction there. I havenโt come across it before.
Thanks again for the useful info ๐๐