Skip to content
OverviewSolutions
4
Comments
3
Androo Frowns
@AndrooFrowns

All comments

  • Androo Frowns•60
    @AndrooFrownsSubmitted 3 days ago
    What are you most proud of, and what would you do differently next time?

    I did everything with HTML/CSS, so I'm happy with that. It does mean however I didn't have a great way to match the way errors were being reported in the spec.

    What challenges did you encounter, and how did you overcome them?

    I found getting the email address/notify me buttons to wrap correctly challenging

    What specific areas of your project would you like help with?

    If there's a better way to keep the design responsive with fewer media calls I'd love to hear it. It seems like it should be possible for none to be required with the right min/max width settings.

    HTML/CSS only page

    1
    Androo Frowns•60
    @AndrooFrownsPosted 3 days ago

    oh no! the footer didn't look like that when I was testing things. oopsy.

    0
  • Wren•80
    @wren3nSubmitted 4 days ago
    What are you most proud of, and what would you do differently next time?

    Tried to use Semantic HTML5 elements again. I am getting good probably. This time mobile first workflow was problematic (I think). The provided design images had different card width on mobile and desktop. Also all the elements had different properties too. I only noticed that after finalizing everything (in my mind). Then added some media query 😄. Also tried using BEM class naming format.

    Blog preview card solution

    1
    Androo Frowns•60
    @AndrooFrownsPosted 4 days ago

    Looks really nice, I don't think I have any comments in particular about the appearance.

    In terms of semantic HTML you may want to consider using the <img> tag and the <author> tag. (I'm also getting used to semantic html elements so I might be wrong about using them, so take it with a grain of salt.)

    BEM style looks pretty organized, maybe I'll need to try it.

    You might be interested in learning about "CSS Generated Content" to for example add the "Published" word from CSS rather than by adding it to the Html.

    Also, I'm not sure if it's itentional but your info at the bottom of the page involves scrolling slightly and still has the white background.

    Marked as helpful
    0
  • Androo Frowns•60
    @AndrooFrownsPosted 4 days ago

    I love the big bold text you used for the call to action.

    May want to consider trying to vertically align the component if that's something you don't have practice with..

    0