@KoiHast
Latest solutions
Latest comments
- @Mouradis
i havent made this project yet but i made a simular one i also struggled with thomething simular my solution was to put the main image as a background and above you can add the blue box with low opacity and the eye icon and make them display: none; and in the hover proprety i make them display: flex; with a little transition
Marked as helpful - @nhilt38@Mouradis
very good code but my only comment wich is not really a mistake but instead of using the container div use <main></main>
Marked as helpful - @DMikaia@Mouradis
You can give the buuton disolay flex/ and align items center / justify contentent center and the margin right about 10px(or however you like)
Marked as helpful - @MichaelAgyebeng@Mouradis
my advice is you may have started the media to early because in this small projects you can make desktop aspect the same as the tablet one but that is not a mistake in general but the mistake the styles of your website dont work on mobile try use max-width instead of min-width in the media query i think it should fix the problem
Marked as helpful - @KoiHast@Mouradis
the quick advice i can give you is that you made your media query too early about 900px you should but it about max-width : 480px (mobile size) or if you want 780px (tablets size)
- @Arbaz-79@Mouradis
you dont have mistakes you can just remove the border from the image and i personally prefer that you put the <h1> and the <p> each in it own div and put those tow divs in a section for bigger porjects its better , again its not a mistake it is just my prefrences and you may benifit from them
Marked as helpful
