Skip to content
OverviewSolutions
6
Comments
4
Neha Nalini Kennedy
@nehanalinik

All comments

  • Neha Nalini Kennedy•100
    @nehanalinikPosted almost 3 years ago

    Did you just forget about the hover part?

    Also it'll be good if you add alt attribute to your images and add description to them too !important. 🐬

    1
  • Aneta•180
    @anetaanetteSubmitted almost 3 years ago

    Hi! 👋 This time I had some problems with the grid on mobile version but I figured it out after a few days of break from this project 😛 Let me know if there is anything I could improve! Thanks Aneta ❣️

    Single Price Grid Component

    2
    Neha Nalini Kennedy•100
    @nehanalinikPosted almost 3 years ago

    Hey! I guess you did not applied font-family in the button element., other than that everything seems great!

    Marked as helpful
    0
  • Sushank49•60
    @Sushank49Submitted almost 3 years ago

    Overlay on the image is not polished, would love feedback on how to do it in a better and more efficient way. My code on that overlay is dirty.

    CSS Flexbox, insets, positioning

    3
    Neha Nalini Kennedy•100
    @nehanalinikPosted almost 3 years ago

    Add alt text in the overlay image, put ur code in one container say "main"

    0
  • Neha Nalini Kennedy•100
    @nehanalinikPosted almost 3 years ago

    Play with background-position, with viewport height and width. I tried this: (add values in place of viewport-width and viewport-height) right viewport-width bottom viewport-height, left viewport-width top viewport-height;

    Marked as helpful
    1