Skip to content
OverviewSolutions
5
Comments
6
Horshaw
@Horshaw

All comments

  • Horshaw•85
    @HorshawSubmitted almost 2 years ago

    Does anyone know why the text overflows the card in the screenshot of my solution? I can change the window size in any direction, but I never get this error on my Vercel hosted version.

    HTML 5 SCSS Flexbox

    3
    Horshaw•85
    @HorshawPosted almost 2 years ago

    Does anyone know why the text overflows the card in the screenshot of my solution? I can change the window size in any direction, but I never get this error on my Vercel hosted version.

    0
  • skullflowerss•10
    @skullflowerssSubmitted almost 4 years ago

    Helloo, is my first project doing this kind of stuff. I'm trying to make more to build a portfolio. Some parts were easy to land but I still have some problem with the background in both ways, because I still feel is kinda... stiff??? and in the mobil version i feels weird. Is there anything I can check up to improve more??

    Thanks for reading this and for the feedback tooo!

    Profilecardcomponent with simple html/css

    1
    Horshaw•85
    @HorshawPosted almost 4 years ago

    Hi,

    Managing the background is the complicated part of this exercise. You can't give a fixed value for the positioning of the background, it has to evolve according to the screen. There are several solutions for this :

    • you can have a look at Aleksandar's solution which uses calc()
    • another solution uses translate(x%).

    I hope that will help you.

    0
  • Horshaw•85
    @HorshawPosted almost 4 years ago

    Hi,

    In the "Why us" part, you should create a list or put <br> in your html. For screens with a height lower than the content, you can't scroll to see the whole content.

    I hope this will help you.

    0
  • Kenneth•95
    @KenmwSubmitted almost 4 years ago

    am having trouble centering the contents,particulary in mobile version.any tip on how to do this plus feedback on any other feature is highly welcome

    single-price-grid-component

    1
    Horshaw•85
    @HorshawPosted almost 4 years ago

    Hi,

    You should try to reconstruct it using grids. Grid is much more suitable for 2D layouts. By putting your grid in a flex, it will be easy to centre it with align-items: center and justify-content: center.

    I hope this will help you.

    Marked as helpful
    1
  • Horshaw•85
    @HorshawPosted almost 4 years ago

    Hi, You should try to rebuild it using grids. Grid is much more suitable for 2D layouts. Moreover, in your different @media you rewrite the same lines of code. If there are no changes, there is no point in repeating yourself. It slows down the page loading. I hope this will help you.

    0
  • Lia•75
    @liampcSubmitted almost 4 years ago

    Hello! I'm practicing CSS, SASS and responsive design. I really appreciate it if anyone could leave feedback. Thanks in advance :)

    Some of my issues:

    • how to match the number of words per line in the header paragraph
    • I'm not sure if I applied the margin-bottom efficiently throughout the page

    sass-social-proof-section

    1
    Horshaw•85
    @HorshawPosted almost 4 years ago

    Hi ! To respect the number of words per line, I put <br> in the <h1> of the header. Because the line was the same in the 1440px and 375px design. However, for the <p> of the header, the line feed varies between the two designs. So I fixed a width for mobile and desktop to respect the line width of both designs. I don't know if this is the best way to do it, but it works. I hope this answers your question.

    1