Elina Eriksson Hult - business-site #368
Conversation
There was a problem hiding this comment.
Hi Elina! Great job on your business site 🎵 Some tips and considerations coming up.
HTML
- Including a video in the header is a nice touch! You’ve correctly added attributes like controls, autoplay, loop, and muted, which enhance the user experience. However, the alt attribute you used on the
<source>tag is unnecessary since the<source>tag doesn't support it. You might want to remove it or provide descriptive text elsewhere. - The label elements are correctly wrapping their respective inputs, which is great for accessibility.
CSS
- The form is well-styled overall, but I would consider adding some more padding to the input fields, as well as changing the color of the link.
- Really like the right alignment of the labels, make it look very neat 👍
Keep up the great work, and continue building your skills! 🎉
PS. Committing your code more often will make it easier to track your changes.
There was a problem hiding this comment.
Så snygg hemsida!
Jag testade även ditt formulär, och det fungerade som det skulle!
Jag såg även att formuläret och hemsidan fungerade på flera olika plattformar så som mobil och surfplatta :-)
Koden va tydligt skriven och i kronologisk ordning. Den innehöll också många kommentarer som gjorde det ännu lättare att förstå var i koden dina element fanns!
Bra jobbat! :-D
| <!-- video content --> | ||
| <video | ||
| id="video" | ||
| controls autoplay loop muted> |
There was a problem hiding this comment.
Riktigt snygg video som passade temat perfekt! :-)
Jag tror att om du tar bort "controls" från din videokod så kommer förmodligen "play" och "stop" knapparna att försvinna på videon.
| <label> | ||
| <span>Song</span> | ||
| <select name="Song"> | ||
| <option value="Shura - 2Shy">Shura - 2Shy</option> | ||
| <option value="Griff - Miss Me Too">Griff - Miss Me Too</option> | ||
| <option value="SKAAR - 24">SKAAR - 24</option> | ||
| <option value="Nao - Another Lifetime">Nao - Another Lifetime</option> | ||
| <option value="IDER - Girl">IDER - Girl</option> | ||
| </select> | ||
| </label> |
There was a problem hiding this comment.
Något som jag lyckades hitta i sådär sista sekund innan jag själv lämnade in var att man kan lägga in en till <option> ovanför de andra som som gör att det inte finns något värde förifyllt. dvs den är bara tom och om man klickar på den så ser man hela listan med val.
Så här såg min kod ut:
<label>
<span>Country</span>
<select name="country" required>
<option value="" disabled selected></option>
<option value="SE">Sweden</option>
<option value="NO">Norway</option>
<option value="DK">Denmark</option>
<option value="FI">Finland</option>
</select>
</label>
| ::-webkit-input-placeholder { | ||
| color: #989898; | ||
| } | ||
|
|
||
| ::-moz-placeholder { | ||
| /* Firefox 19+ */ | ||
| color: #c2c2c2; | ||
| } | ||
|
|
||
| :-ms-input-placeholder { | ||
| /* IE 10+ */ | ||
| color: #c2c2c2; | ||
| } | ||
|
|
||
| :-moz-placeholder { | ||
| /* Firefox 18- */ | ||
| color: #c2c2c2; | ||
| } |
There was a problem hiding this comment.
coolt, inte sett det här ännu! Ska googla sen och se vad de gör i din styling så jag kanske lär mig något nytt! :-D
No description provided.