Conversation
home.css
Outdated
| @@ -0,0 +1,226 @@ | |||
| body{ | |||
home.css
Outdated
| } | ||
|
|
||
|
|
||
| .imge { |
home.css
Outdated
| list-style-type: none; | ||
| /* Remove bullets */ | ||
| padding: 0; | ||
| /* Remove padding */ |
home.css
Outdated
| } | ||
|
|
||
|
|
||
| .conatiner1{ |
There was a problem hiding this comment.
Please don't use 1 or 2 in names.
Instead give some proper names.
home.css
Outdated
| height: 50px; | ||
| } | ||
|
|
||
| .sideimage{ |
There was a problem hiding this comment.
Please give css class and ids names in kebab case. Like "image-container".
Please make changes at all places in css files
home.html
Outdated
| <link rel="stylesheet" href="home.css"> | ||
| </head> | ||
|
|
||
| <body> |
There was a problem hiding this comment.
Please give proper & consistent indentation in all files.
home.html
Outdated
| </nav> | ||
|
|
||
|
|
||
| <div class="conatiner1"> |
There was a problem hiding this comment.
Please don't add 1 or 2 in naming. Instead please specify proper names. Like "main-container", "body-container" etc.
home.html
Outdated
|
|
||
| <div class="conatiner1"> | ||
| <div class="center"> | ||
| <P class="paragraph"> |
There was a problem hiding this comment.
Why P is capital ? Please correct
home.html
Outdated
| <h3 class="color3">Happy Independent Day</h3> | ||
| <p class="color3">Lorem ipsum dolor sit amet, consectetur adipisicing elit. Et modi nihil cumque | ||
| consectetur voluptatum harum explicabo at facilis similique libero, earum, | ||
| <br /> |
There was a problem hiding this comment.
Please don't use br tag for new line as its not a good practice. Use css.
home.html
Outdated
| </div> | ||
| </div> | ||
|
|
||
| <div class="createflex"> |
There was a problem hiding this comment.
Please give proper naming. createflex is not a good name.
And, for all class and ids name use kebab case.
2 participants
No description provided.