Conversation
ckesurf
left a comment
ckesurf
left a comment
There was a problem hiding this comment.
I gave a glowing review on your final project, Asel. Great job, keep up the good work, and good luck on your new pursuit as a stellar front-end dev.
|
|
||
| let messageForm=document.getElementsByName('leave_message') | ||
| leave_message.addEventListener('submit',function(e){ | ||
| leave_message.addEventListener('submit',(e)=>{ |
| </a> | ||
|
|
||
| <!-- <a href = "mailto:asel.karagazieva@gmail.com" target="_blank" class = "icon"> --> | ||
| <a href = "mailto:asel.karagazieva@gmail.com" target="_blank" class = "icon"> |
There was a problem hiding this comment.
using the target=_blank to open in new tab, expertly done
| #messages a { | ||
| display: inline-block; | ||
| width: 30%; | ||
| width: fit-content; |
| background-image: url(../css/blue\ grey.jpg); | ||
| background-size: cover; | ||
| background-blend-mode: multiply; | ||
| color: var(--clr-light); |
There was a problem hiding this comment.
the messages section looks very stylish in browser. great job here.
| let messageSection = document.getElementById("messages"); | ||
| let messageList = messageSection.querySelector('ul') | ||
|
|
||
| let removeButton= document.createElement("button") |
There was a problem hiding this comment.
this whole section looks cleaner. If I had anything nitpicky to say, it would be that some lines in this section, like 61 through 64, could be indented, and it would be nice to see consistent semicolons at the end of code statements, especially because you worked hard to break up certain lines so they could be more readable, such as in Line 56-57, which I appreciate.
2 participants
No description provided.