Amon• 2,560
@A-amon
Posted
Hello! Great work. The responsiveness looks good too~ 😀
Maybe just change the .card min-height value to fit-content. Else, there is quite some space on mobile size (this change works fine for larger sizes too).
As for the accessibility report:
- It seems that you can only nest
<dt>and<dd>tags (and other tags as mentioned in the report) under<dl>. - As for the other issue, you should put the main content of the page into
<main>. For better understanding on what to do, you may refer this
P.s. Thanks for helping me learn about <dt> and dd tags 😉
Marked as helpful
1
Randy Brilliant• 115
@RandyBrilliant
Posted
@A-amon wow the min-height thing is sure worth trying, thanks for the advice mate, appreciate it. 🤗🤗
0
Amon• 2,560
@A-amon
Posted
@RandyBrilliant No prob! Am always glad to help~
0
