Skip to content
OverviewSolutions
14
Comments
6
BadT
@saodinh

All comments

  • Ishmael•330
    @IshmaelsealeySubmitted almost 3 years ago

    Hi. My question is about the border-radius: ; aspect of the section. I tried putting the border-radius: 5px; on the container of the entire section, but that didn't work.

    To resolve that I went ahead and made each section have a separate border and change that using a media query for desktop screens.

    While this works fine, my issue with this is that it's repetitive and would be more intuitive if I could put the border on the containing section.

    Any comments on what I could do will be appreciated :D

    Responsive Landing with CSS Grid

    1
    BadT•230
    @saodinhPosted almost 3 years ago

    You can set the <main> element {border-radius: 5px ; overflow: hidden}. The overflow hidden is what you are looking for here.

    Marked as helpful
    1
  • 👾 Ekaterine Mitagvaria 👾•7,600
    @catherineisonlineSubmitted almost 3 years ago

    Hello, Frontend Mentor community! This is my solution to the Huddle landing page with a single introductory section.

    I have read all the feedback on this project and improved my code. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.

    You are still free to download or use the code for reference in your projects but I longer update it or accept any feedback.

    Thank you

    Huddle landing page with a single introductory section

    4
    BadT•230
    @saodinhPosted almost 3 years ago

    You did a great job, though some things can be improved. Since the "Register" button is actually a link to the register page and doesn't belong in a form , you should use <a> tag for it. The three social icons should be wrapped in <a> tags as well (because they are also links to the social media pages).

    0
  • 👾 Ekaterine Mitagvaria 👾•7,600
    @catherineisonlineSubmitted almost 3 years ago

    Hello, Frontend Mentor community! This is my solution to the 3-column preview card component.

    I have read all the feedback and tried to update my code accordingly. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.

    You are free to download or use the code for reference in your projects, but I no longer update it or accept any feedback.

    Thank you

    3-column preview card component

    2
    BadT•230
    @saodinhPosted almost 3 years ago

    Those "learn more" buttons are actually links, so you got to use <a> tags for those.

    1
  • BadT•230
    @saodinhPosted almost 3 years ago

    It looks great. One thing though, your links and buttons should have a hover and focus state.

    0
  • Aleksander Tysklind•860
    @outerpreneurSubmitted almost 3 years ago

    Hi! Im back at it after the break!

    here is my solution to this challenge. Im starting to do some more effort into responsiveness.

    For this challenge, I don't know how to make the image adapt to the size of the screen of a smaller device. Any ideas?

    I also struggle with the fontawesome icons. For some reason, they don't appear on my project. I have added the stylesheet and the CDN address.

    Huddle landing page with single introductory section

    1
    BadT•230
    @saodinhPosted almost 3 years ago

    About the font-awesome icon, trying changing the 'href' to this link instead "https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.0.0-beta3/css/all.min.css".

    Marked as helpful
    0
  • BadT•230
    @saodinhPosted almost 3 years ago

    Oh, I didn't know that, thanks. I'll fix it

    0