@michelcubSubmitted 3 months ago
MorganMartin12
@MorganMartin12All comments
- @MorganMartin12Posted 3 months ago
I would consider using more semantic html for some of your elements. Instead of using a div with background image, you should just use a img tag with an alt. This allows for better accessibility and SEO. Also consider using the time tag for the date for the same reasons. I would consider making the flex direction column to make this easier to control the layout.
0 - @RubsnewgateSubmitted 9 months ago@MorganMartin12Posted 3 months ago
I am very green to frontend but I loved looking through your project especially liked the sass things you were doing with the flexbox factory and defining variables. Sorry I couldn't find any criticisms.
0