Since this project was quick to code, I challenged myself to reduce the CSS to 100 lines or less. I focused on treating my CSS like a design system, extracting repeated patterns into small, focused classes. This approach is a little more time-consuming, but I can see how it makes a component like this easy to maintain and scale over time.
Reza Fauzi Augusdi
@fbtwitterAll comments
- @SheGeeksSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?@fbtwitterPosted 3 months ago
I appreciate your attention to detail, especially the addition of titles to images and the use of the <small> element for accessibility. However, based on best practices in web development, wrapping these elements in a <figure> tag would provide a clearer structure, as it semantically groups the image with its associated content. This approach enhances both accessibility and the logical organization of the page.
0 - @fbtwitterSubmitted over 2 years ago
By doing this, i think what i have done just tried to make the apps more looks like the design and kinda follow the css structure about the spacing and font size scale. I still kinda a bit needs what to focus. am i have to follow strict rule or not ?
@fbtwitterPosted over 2 years agoThank you for the feedback, Yes, I already add some additional value for the alt image and to be sure i wanted to ask, is there any particular number for the line height ? things i has learned that all values above 16px surely had to get number below than 1.5 so i put 1.25 there. is there any fixes for the number ? or that's just an opinionated visual look perspective?
0