fawzy351• 130
@fawzy351
Submitted
waiting for your feeds bros
Gideon
@GiDDeRo
All comments
- Gideon• 440
@GiDDeRo
Posted
Nice work bro. When hovering the button (button:hover), the text should still be bold.
0 @wobin1
Submitted
How does my use of colors feel, and how can I make my next design feel better?
Gideon• 440@GiDDeRo
Posted
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- Gerardominic• 80
@Gerardominic
Submitted
Gideon• 440@GiDDeRo
Posted
Hey there, Kudos to you on the design. Like it. You forgot to change the font in the button
0 - Brunno Hibbeln• 220
@BrunnoHibbeln
Submitted
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!
Gideon• 440@GiDDeRo
Posted
Hey There,
<article> Generally text-based content, <section> For any other...1 - Hamza Muhammed• 10
@hamza4916
Submitted
there's no html and css tags for beginners like me.
i'd really appreciate if you added it, thanks😊
Gideon• 440@GiDDeRo
Posted
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 - giovanni.rizzolo• 10
@giovannirizzolo
Submitted
Gideon• 440@GiDDeRo
Posted
Nice Solution
- I'll say learn to use semantic tags
- Make h4 text more bold.
Happy Coding!
Marked as helpful
1 - Gideon• 440
@GiDDeRo
Submitted
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?
Gideon• 440@GiDDeRo
Posted
*box
0 - zwiro• 290
@zwiro
Submitted
Gideon• 440@GiDDeRo
Posted
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 - Francisco Franco• 20
@FranciscoFranco014
Submitted
Hi everyone, this is my solution of this challenge, im open for your feedback this is very important for me.
Thanks.
Gideon• 440@GiDDeRo
Posted
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 @ondickson
Submitted
I'm very happy with how this challenge turned out except for the mobile view. I'm yet to fully learn and understand all the tiny details using @media efficiently. I will definitely come back to this project to finish that. At the moment, here's my solution. Any feedback is welcome.
Gideon• 440@GiDDeRo
Posted
Nice solution. To center the card (vertically and horizontally): body { display: flex, align-items: center, justify-content: center, height: 100vh }
Marked as helpful
1- pvkom26• 50
@pvkom26
Submitted
Is it OK to turn the <body> element into a flex for a challenge like this? Just to center the card? Or would it be better to wrap the card into a <div> for that?
Also, if the card is the <main> element and it gets wrapped into a <div>, would it be more semantically correct to make the wrapper <main> and turn the card into a regular <div>? I doubt it but want to see what the community thinks.
Love this website, helps me a lot!
Gideon• 440@GiDDeRo
Posted
This is a nice solution.
-
Actually yeah, you could do that. Look at it this way, you are gonna be saying (coding), the body to center its element. And the only and main element is the card. Even tho, what you gonna do about the footer, if the card is all wrapped in a div. Personally, I think you done a great job
-
Depending on your approach. Personal preference: But since the card is literally the "main" element, another <div> container wouldn't be needed. Yunno, it gives the code simplicity and clarity.
Happy coding
Marked as helpful
1 -
