Skip to content
OverviewSolutions
7
Comments
7
NalsDev
@NalsDev

All comments

  • NalsDev•80
    @NalsDevPosted 15 days ago

    Its good. But the sub title in the grids are bit too large except that it is perfect.

    0
  • Abdulsalm Taiwo•120
    @SymplyteeziySubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    I will trying g to focus on using grid in my next challenge.

    What challenges did you encounter, and how did you overcome them?

    I’m unable to use grid which is really important.

    What specific areas of your project would you like help with?

    Using grid.

    Responsive four card feature using CSS

    1
    NalsDev•80
    @NalsDevPosted 4 months ago

    The project more over looks similar to the design except the font-size and heading levels. You can add font weight property to heading tags refer to the style guide in the given folder.

    Grid For more practice you can checkout the courses offer by front end mentor in building flexible layouts(course before the project) checkout the links you will find resourses for grid and flex.

    0
  • P
    kareem sakr•190
    @kareemsakrSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    Minimized the code in media queries section

    What challenges did you encounter, and how did you overcome them?

    The picture/img had a bit of strange whitespace underneath it that took me a bit of time to figure it out.

    What specific areas of your project would you like help with?

    If my css code adheres to best standards when it comes to responsivity.

    Also if my HTML is semantic or could be improved

    1st Responsive design challenge

    1
    NalsDev•80
    @NalsDevPosted 4 months ago

    The website looks too good It perfectly matches the design.

    0
  • NalsDev•80
    @NalsDevPosted 4 months ago

    great work!. The page looks good. But i guess you forgot add background color to preparation time container.

    0
  • NalsDev•80
    @NalsDevPosted 4 months ago

    The output looks good. Great work!.

    0
  • IsaacKel•70
    @IsaacKelSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    found it easy for the most part

    What challenges did you encounter, and how did you overcome them?

    i couldn't line the label up with the side of the image properly. if i added a container i probably could've done it.

    What specific areas of your project would you like help with?

    using h1, p tags will adjust my margins a bit. i was originally just using a div. i should use the proper tags and just adjust the margins they use.

    Why aren't my images showing on the github pages website - but when i press show live server on vs code, they do?

    Flex box layout, organisation of text

    1
    NalsDev•80
    @NalsDevPosted 4 months ago

    the page looks good but images. the image path may be incorrect or you didn't upload the images folder from assets into git. once check those and re-run the code.

    maybe you can add the image inside the div and set its height and width. then you can set width to 100%.

    I think using the approriate tags like p and h1 in code is necessary. using div may look but but it need more css than approriate tags does.

    It is just my point of view. All the best.

    0
  • NalsDev•80
    @NalsDevPosted 4 months ago

    your solution is too good but not in the right place. you can hold entire within the body with a container and add some properties with css. use flex box and set properties justify-content and align-items to center and don't forgot to mention the height and width of the container otherwise they will not align in center. This is just feedback on you code. All the best

    1