portfolio #17
portfolio #17
Conversation
There was a problem hiding this comment.
Ditt projekt är riktigt bra och uppfyller de kriterier som efterfrågas (plus lite till). Koden är lätt att följa och du har gjort ett bra jobb med att namnge klasserna på ett tydligt sätt. Effekterna du satt på korten är riktigt snygga och de lyfter verkligen användarupplevelsen!
Det som skulle kunna utvecklas är att göra HTML-filen lite tydligare genom att använda semantiska taggar som <header> och <main>, de ger en klarare struktur.
En annan liten förbättring är att länkar till externa resurser, som Google Fonts, bör ligga i HTML-filen istället för i CSS-filen för bäst prestanda och bäst praxis.
Bra jobbat!
index.html
Outdated
| <div class="hamburger-menu"> | ||
| <a href="./index.html">Home</a> | ||
| <a href="#">About</a> | ||
| <a href="#">Portfolio</a> | ||
| <a href="./contact.html">Contact</a> | ||
| </div> | ||
| <div class="nav-links"> | ||
| <a href="./index.htm">Home</a> | ||
| <a href="#">About</a> | ||
| <a href="#">Portfolio</a> | ||
| <a href="./contact.html">Contact</a> | ||
| </div> |
There was a problem hiding this comment.
Jag ser att du har två separata menyer för hamburgermenyn och den "vanliga menyn", som jag förstår att du anpassar beroende på skärmstorlek. En idé för att göra koden lite mer flexibel och lättare att underhålla skulle vara att använda en enda meny som du manipulerar med hjälp av JavaScript och CSS istället för att duplicera HTML-strukturen.
På så sätt slipper du upprepa samma menyinnehåll på två olika ställen, vilket gör koden renare och enklare att hålla koll på. Dessutom blir det lättare att göra förändringar i menyn i framtiden, eftersom du bara behöver justera den på ett ställe.
There was a problem hiding this comment.
Generella kommentarer till din html fil:
- Ett förslag för att göra koden ännu mer strukturerad: överväg att använda
<header>och<main>för en tydligare uppdelning och bättre läsbarhet. - Bra jobbat med klassnamnen, de är tydliga och beskrivande!
There was a problem hiding this comment.
Good job with this project, you’ve successfully created a responsive business site using flexbox and CSS grid. Regarding your question about the footer, I think the quickest fix would be to set a min-width of 100vh to the body.
PS. I really enjoy being able to click outside of the hamburger drop down to close it, nice touch 👍
PS2. Remember that all things code related should be in English (even code reviews)
https://linda-schonfeldt.netlify.app/
I have some problems with the footer not being pushed all the way down on the "Contact" page.
I got some great comments from Linda tho, and I made some changes since then. :)