Resolves 38 issue #44
Conversation
|
@metaversemoon try adding a |
| @@ -2,11 +2,22 @@ | |||
| <html> | |||
| <head> | |||
| <title>Document Two Sections</title> | |||
| <link rel="stylesheet" href="style.css"> | |||
| <link rel="stylesheet" href="style.css" /> | |||
There was a problem hiding this comment.
No need to add / here. Can you remove it?
| <link rel="stylesheet" href="style.css" /> | |
| <link rel="stylesheet" href="style.css"> |
| <li>first point</li> | ||
| <li>second point</li> | ||
| <li>third point</li> | ||
| </ol> | ||
| <video src="http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4" controls width="300"></video> | ||
| <video |
There was a problem hiding this comment.
Try to not change code unrelated to the issue at hand.
You should focus on the issue and what it's asking. If the issue doesnt require a code change here, you shouldnt be changing it. Even if you think the style is prettier the way you write here. You should strive to do the least amount of work as possible and avoid "diff noise" by changing unrelated code just because you feel a certain way about how it looks.
Update: I think you might be using an automatic code formatter. Maybe dont use it.
| src="https://github.com/nbktechworld/introduction-to-html/raw/abcdac58651ba724c40665c82dd55f171534115f/document-two-sections/barking.mp3" | ||
| controls | ||
| ></audio> | ||
| <ol class="order"> |
There was a problem hiding this comment.
I see you added the class order here. Did you forget to commit the definition of that class?
Resolves 38 issue