Skip to content
OverviewSolutions
22
Comments
15
Gideon
@GiDDeRo

All comments

  • Gideon•500
    @GiDDeRoPosted about 1 year ago

    Nice work bro. When hovering the button (button:hover), the text should still be bold.

    0
  • Gideon•500
    @GiDDeRoPosted over 1 year ago

    Hey there. Nice Design. You forgot to change the font in the button. Also, it is a good practice to make projeect responsive (at least) for major screen. Good Luck!

    Marked as helpful
    0
  • Gideon•500
    @GiDDeRoPosted over 1 year ago

    Hey there, Kudos to you on the design. Like it. You forgot to change the font in the button

    0
  • Brunno Hibbeln•220
    @BrunnoHibbelnSubmitted over 1 year ago

    Hey guys, in my last projects I received several feedbacks advising me to make a more semantic html. I did the best I could but I'm still confused when to use <section> and <article> for example. I also tried to use "rem" instead of pixels, but I still have doubts about how it works. And finally, is there any better way to implement the icon overlay?

    Thanks for the feedbacks, they help a lot!

    Responsive component using html and css with media queries

    2
    Gideon•500
    @GiDDeRoPosted over 1 year ago

    Hey There,

    <article> Generally text-based content, <section> For any other...
    1
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution.

    • Learn on how to change to other external fonts
    • To center the card. body { display: grid; place-items: center; height: 100vh; } or body { display: flex, height: 100vh; align-items: center; justify-content: center}
    Marked as helpful
    0
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution.

    • Make the <p> text lighter
    • I think you forgot to delete something (smiles)

    Happy Coding!

    Marked as helpful
    0
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution

    • I'll say learn to use semantic tags
    • Make h4 text more bold.

    Happy Coding!

    Marked as helpful
    1
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution 😃 Almost exact For aligning the text to the center: { text-align: center}.You could add that to the body or parent container

    0
  • Gideon•500
    @GiDDeRoSubmitted about 2 years ago

    Do you know what an accordion is in HTML? I didn't know. Not only until this project. According to W3schools, an accordion is used to show (and hide) HTML content.

    Added a little hover effect on the button. You should check it out

    So... What do you think?

    FAQ Accordion Card (HTML + CSS + JS)

    1
    Gideon•500
    @GiDDeRoPosted about 2 years ago

    *box

    0
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    What a nice solution. Love the little custom animation there. When the hovering the container. The main text becomes less visible. You should choose a color that would fit in the situation (maybe a black).

    More of a Feedback than Critique

    Happy Coding!

    1
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution.

    • You forgot the active-states.
    • To center the card. body { display: flex; height: 100vh; align-items: center; justify-content: center}
    • Div(s) are great as containers. For easy navigation, accessibility and clarity, learn to use semantic tags instead
    Marked as helpful
    0
  • Gideon•500
    @GiDDeRoPosted about 2 years ago

    Nice Solution. You should have given the rates (1-5) and the button a cursor: pointer; hover effect. .rates:hover{ cursor: pointer; }

    Happy Coding!

    Marked as helpful
    0