Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Log in with GitHub
OverviewSolutions10Comments37

All comments

  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Qazi! 👋

    Good work

    I suggest ,

    • You have to check again your responsive design while 1420px your design is portrait view.

    • Adding a footer

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Joaquin ! 👋 Nice work

    I suggest ,

    • You should fixe your footer It have to be on bottom page

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Rafael ! 👋 Nice work

    I suggest ,

    • You should fixe your footer
    • You have to check again your responsive design

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi David! 👋

    I suggest ,

    • You should check again your responsive design
    • Changing the font-family

    Keep coding 👍

    0
  • Pau S• 40

    @pcls25

    Submitted

    Hello! I chose to do this challenge because I wanted to get to know the CSS grid more. The card layout is done with it but I used CSS flexbox for the profile part where you can find the author image and name. I found it difficult to make it responsive with just CSS grid, having to set margins every time. If you can point out areas I can improve on please feel free to do so. Your suggestions are much appreciated. Thank you!

    Testimonials Grid Section with CSS Grid

    1

    Seck Mamadou• 325

    @doums85

    Posted

    Hi Pau! 👋

    Pretty nice code !

    I suggest ,

    • You should hidden horizontal scroll bar

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Aishah,👋

    Pretty nice work !

    I suggets,

    • You've to fixe your footer

    • Adding spacing between your contain and footer

    Keep coding 😁😁

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi David! 👋

    Nice work

    I suggest ,

    • You've to fixe your footer it should be on bottom page.

    • decreases p

    • check again the responsive design while 1110px your page is broken

    • decreases your user's picture while 768px

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Lucas ! 👋

    Nice work and yes, you can do it with alert on html after body

    I suggest ,

    • Adding a padding on your button

    • The background-image you should add on header (banner) section

    • You've to add a background-color on menu bottom and fusion it with call-to-action section

    • check again your responsive design while 1160px the design is broken

    • Adding a hover on nav

    Keep coding 👍

    1
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Burgos! 👋

    pretty nice work 👍

    I suggest ,

    • you've to add a footer on page

    Keep coding 👍

    1
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Saul! 👋

    Nice work

    I suggest ,

    • Adding size and wight on your titles and subtext

    • Adding overflow:hidden on body

    • check the responsive design while 800px there is a problem

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    salamou alékoum Diop ! 👋

    Je pense que je peux écrire en français avec toi 😁

    Très bon travail masha Allah.

    Je te conseillerais

    • Revois ton footer il doit être fixe en bas de page vu ton code je peux que t'as l'oublié.

    • Ton animation pour l'affiche est vraiment cool je pense que tu devrais le prend mais lent 😉.

    En tout je suis très content de retrouver un(e) compatriote sur ce site.

    Keep coding 👍

    1
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Raj! 👋

    Pretty nice work

    I suggest ,

    • You've to fixe your footer it should be on bottom page. You should decrease your font-size for the footer
    • Change your font-weight on the title

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Adam! 👋

    Pretty nice work

    You used react that's cool

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Felipe! 👋

    Very nice work

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi David! 👋

    Good !

    I suggest ,

    • Your background-image have to resize with backgroud-size.

    • I don't understand why you used span on your Title I think you have to use h1 and resize it

    • You should check your font-size

    • The footer have to be on bottom page.

    • You should add a shadow for your cards

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Gonzalo! 👋

    Pretty nice work

    I suggest ,

    • Shadows cards could be more subtle
    • Add your font-size for h2 and p

    Keep coding 👍

    0
  • Josef Tomecek• 20

    @joseftomecek

    Submitted

    Hi,

    I wanted to aks you how do you implement Font Awesome icons. I managed to sign upto get the html link but this didn't work so I'm using a bit clumsy download of the FA library. Are you using a different way or something else than FA?

    Thank you, JT

    Responsive Bootstrap 4.5 Approach

    1

    Seck Mamadou• 325

    @doums85

    Posted

    Hi Josef ! 👋

    Nice work

    I suggest ,

    • You should add space between logo
    • Check your footer it have to be on bottom page

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi David! 👋

    I suggest ,

    • Your background-image have to resize with backgroud-size.

    • You should check your font-size for h1 h4 p span. h1 is bigger than the others

    • The footer have to be on bottom page.

    • You should add a shadow for your cards

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Taofiq! 👋

    Very pretty nice work.

    @artimys I'm agree with you

    I suggest ,

    • You should made a shadow-box more subtile

    Keep coding 👍

    0
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Swiichy! 👋

    I suggest ,

    • Your width and height have to be 100%, your container should take the whole screen
    • This isn't responsive check again your code especially the media queries part. You should add media for tablet etc and not only just for 375px screens

    Keep coding 👍

    1
  • Seck Mamadou• 325

    @doums85

    Posted

    Hi Anton! 👋

    I suggest ,

    • You’ve to change your <code>font-size</code> on <code>h1</code> it should be bigger

    Keep coding 👍

    0
  • P
    ApplePieGiraffe• 30,545

    @ApplePieGiraffe

    Submitted

    Hey, everybody! 👋

    This was an awesome challenge that was a little trickier than I initially thought! 😅

    This was my first time creating a site with light/dark themes but I'm pretty happy with the result. I added this cool transition that ripples over the cards when the theme is changed! 😆

    As usual, feedback is welcome and appreciated! 😊

    And as always, happy coding! 😁

    Social Media Dashboard With CSS Grid And Light/Dark Theme Transition

    6

    Seck Mamadou• 325

    @doums85

    Posted

    Waouh Just amazing PERFECT 👏👏👏

    1