Karen Robertson• 190
@indigorose
Posted
This is a nice solution and follows the design brief.
Whilst accessible, the solution does not match the mobile styles.
It is tricky as I struggled with the mobile styling, however using the body element selector in your css and changing the background-color property will help to remove the white border in your solution and modify with @media when required.
The code is clear, efficient and reusable. I used the li::marker as well to access the li colour changes.
As you keep trying other challenges, you will be able to return to this and refactor. Keep going and look forward to seeing your other solutions.
Marked as helpful
0
Dharmendra Rathod• 190
@rathoddharmendra
Posted
Thanks for a detail review @indigorose :)
0
