Skip to content
OverviewSolutions
11
Comments
7
NOMAN ZAHOOR
@zahoornoman0

All comments

  • Tom Golding•250
    @ComanderPotatoSubmitted over 2 years ago

    Js and css code is a little scuffed probably a bit of repeat and unnecessary code written, wasn't too sure whether to change the class or js style attributes so I just did attributes so there is a big clump of code for the different themes down the bottom. I couldn't figure out how to format to 100,000 I got close though ahah. also i didnt know how to change hover color in js, I assume the way around it was to set an event listener for mouse over and have it set that way. Give me some criticism

    Calculator exercise

    1
    NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    you have missing calculator functionality

    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    A very nice arrangement of the layout as the expected by the frontendmentor. Keep it up. I am also willing to learn CSS Js and react (inshallah).

    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    Blunder in the design. Easy only if you make exact match of the design. Hint:define width of the card to make exact same as design

    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    Such an amazing solution, definitely gonna download and learn tricks. Currently I don't know scss and i am willing to learn it. I know only css and submitted only using only css. I did some amazing with this challenge see it in my solution. This is not well aligned with the screen but amazing placeholder with chat scroll. I don't have pro account so i was not able to make as exact.

    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    Try to center the card to the screen. Hint: Define height of the body:100vh Then center it using display flex

    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    Try to position it in the center of the screen. Hint: Body{ Height:100vh; Display:flex; Justify-content:center; Align-items: center; }

    Marked as helpful
    0
  • NOMAN ZAHOOR•160
    @zahoornoman0Posted over 2 years ago

    Blunder in the designing and html rules

    0