Blessing Chusaru - Completed Test Task

[chusarublessing]

No description provided.

[bbucek]

Cards:

• Good responsive, although could have one more break with two cards
• No hover state
• No coloured labels

CTA:

• Text could span the whole width when on mobile
• Could use icon instead of text for chevron

[rafzielinski]

Hi and thank you for your submission.
We like your implementation but we want to ask for few tweaks before we get to the final decision.

1. CTA button should have an icon instead of text chevron, it's also missing a hover state.
2. CTA text, try using max-width with rem value instead of width: 50% to limit text width.
3. Card is missing a hover state.
4. Card label is missing color variants
5. Listing could display cards on different screen sizes like 3 > 2 > 1 instead of jumping straight from 3 to 1.

[chusarublessing]

Thank you for your feedback Rafal. Let me sort it out and get back to you.

…

On Wed, May 31, 2023, 18:54 Rafal Zielinski ***@***.***> wrote: Hi and thank you for your submission. We like your implementation but we want to ask for few tweaks before we get to the final decision. 1. CTA button should have an icon instead of text chevron, it's also missing a hover state. 2. CTA text, try using max-width with rem value instead of width: 50% to limit text width. 3. Card is missing a hover state. 4. Card label is missing color variants 5. Listing could display cards on different screen sizes like 3 > 2 > 1 instead of jumping straight from 3 to 1. — Reply to this email directly, view it on GitHub <#21 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ANSY5X7UXOUMWVLJKB34WN3XI5ZUXANCNFSM6AAAAAAYRGF7K4> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[chusarublessing]

Hie Rafzielinski

I have implemented changes as advised.