Skip to content
OverviewSolutions
9
Comments
13
@valchiz
@yemmighto

All comments

  • @valchiz•150
    @yemmighto

    oh thanks for the suggestions, I was about to figure the html correction well enough but I don't seem to understand the javascript part, how will I take the event.preventDefault(); out of the conditions block, can you write a sample code of that part only as a comment,

  • P
    Andrew A Lee•380
    @drewlee
    What are you most proud of, and what would you do differently next time?

    I had to refresh my memory on how to make form interactions accessible, which includes leveraging aria-required, aria-describedby, and aria-invalid attributes, along with programmatic focus management so that the user's place of focus isn't lost. I also implemented a custom keyboard focus indicator, as it wasn't explicitly provided as an active state on in input in the design spec.

    Newsletter Sign-up Form with Success Message

    2
    @valchiz•150
    @yemmighto

    I really love this, checking out what you did, gave me some insight and corrections on what I did, thanks

  • @valchiz•150
    @yemmighto

    this is nice but need some correction, after clicking the refer icon, the word "share" isnt showing, also cant find the mobile version. love the fact that i can click any part of the page and the icon will close

    Marked as helpful
  • @valchiz•150
    @yemmighto

    this is well coded, really learn some stuff from the way you code this. thanks

    Marked as helpful
  • @valchiz•150
    @yemmighto

    thanks for your feedback and the video recommendation, will learn a lot from it

  • @valchiz•150
    @yemmighto

    I really appreciate your feedback, I really don't know how to make use of the linear gradient part yet but will try to figure it out.

    about the height and width on main element, which make it unresponsive, where can I apply the height and width to make it responsive since I divide my work in sections.

    the max width character really help a lot and also the shadow part, it really help

  • @valchiz•150
    @yemmighto

    wow this is really well done, I had to go through the code, yet to really understand the grid layout but use flex for mine. will have to read through you code to understand how it is done

  • @valchiz•150
    @yemmighto

    love the responsiveness and the transition while reducing the page size but the font size and font family aren't in line, it look bigger especially in mobile view, color is also different, you can find the right one in the style section in the folder

  • @valchiz•150
    @yemmighto

    well done but the font size is somehow bigger than 16 when i checked using the live site

  • Elango Ravi•20
    @elangoravi
    What are you most proud of, and what would you do differently next time?

    I've used psuedo root selector to define custom properties and manage it in css effectively. I've created a little too many css classes, but instead should learn to make better use of selectors for the compact styling

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

    I need help/review and feedback regarding the height/width of layouts (right now its all trail and error). I'm still learning about how the parent and child containers interact and trusting my luck with overflow.

    Social card layout with flexbox and grid

    1
    @valchiz•150
    @yemmighto

    this is perfectly done, love the hover effect

  • @valchiz•150
    @yemmighto

    thank you, was looking for way to see the figtree font and I notice it didn't really apply but this really help. thanks a lot

  • @valchiz•150
    @yemmighto

    Really well coded but the max-width is kind of small to the main design and the font is also not the same.

  • @valchiz•150
    @yemmighto

    This is well code, love this