uxdesign, landing, page, login page and database task#170
Conversation
html, css and javascript
mysql task
|
|
||
|
|
||
| </div> | ||
| <div class="box"> |
There was a problem hiding this comment.
landing page correction on code line 63, close div tag 65.
please check for the correction through the docs link below. thanks.
| <div class="box"> | |
| <div class="box"> | |
| ```suggestion | |
| <div class="box"> | |
| `` |
| </ul> | ||
|
|
||
| </nav> | ||
| </span> |
There was a problem hiding this comment.
This span closing tag does not have an opening tag. You should remove it if it's not needed.
|
|
||
| <div class="product"> | ||
| <div class="product_box"> | ||
| <div id="product_image"> |
There was a problem hiding this comment.
This id="product_image" is used in a couple more places. An id should be unique. If you need to make use of it in more than one place, you should consider using a class instead.
| </div> | ||
|
|
||
| <div class="product_box"> | ||
| <div id="product_image"> |
There was a problem hiding this comment.
same here as the previous comment.
| </div> | ||
|
|
||
| <div class="product_box"> | ||
| <div id="product_image"> |
There was a problem hiding this comment.
Here as well, id should be unique. Consider using a class instead.
|
|
||
|
|
||
| </div> | ||
|
|
There was a problem hiding this comment.
There is supposed to be a closing div tag here for the div class="section" listed above.
| <main> | ||
|
|
||
| <section> | ||
| <div class="section"> |
There was a problem hiding this comment.
This div does not have a closing tag.
correction/review of landing page
Task:
I
I