Feat: added 8-column card for card pattern - #5809

Demo

Jenkins

demos.haus

Thanks Madhur - copying my comments from the other PR:

• Use h1 styling for the article heading
• Limit the title to 3 lines as we've done with the other card and showing the description for the featured article without having to hover over it (but keeping the hover interaction with the card border the same)

I mocked up the different variations here if it's helpful

Reviewed this with @mattea-turic and had the following comments -

• The line height (i.e. the space between each line) in the heading looks to be bigger than Vanilla
• Perhaps with this ^ fix, we can keep the description to 3 lines instead of 2
• Also mocked up the tablet and mobile versions with author, date and description here

Thanks @eliman11 @mattea-turic
Made changes as suggested in the comment

• For the line-height, this was something already there in the styling for the card heading, are you sure we need less of it now?

Thanks so much @immortalcodes! Just one more tiny change, can we reverse the order the information appears on tablet and mobile to image -> heading -> author/date -> description? To stay consistent with the way the other card variants resize on tablet and mobile. Thanks a lot!

Thanks so much @immortalcodes! Just one more tiny change, can we reverse the order the information appears on tablet and mobile to image -> heading -> author/date -> description? To stay consistent with the way the other card variants resize on tablet and mobile. Thanks a lot!

Sorry but is it not already the case?

@immortalcodes hm maybe the changes didn't load for me yesterday. LGTM then :)

+1ing too as Elaine and I shared the same comments