Divyanshu• 45
@divyanshu0x16
Submitted
Tell me what do you guys think 😃🚀
Luiz Claudio Lopes
@lc-dev90
All comments
- Luiz Claudio Lopes• 505
@lc-dev90
Posted
Hello Divyanshu, i have one question, when I hit the details countrie page and refresh it I don't lose the data, the page was refresed. How can I made this? Thank you
0 - Mzer Emmanuel Terdoo• 145
@terdoo-mzer
Submitted
Please mates review and leave comments please. Thanks!
Luiz Claudio Lopes• 505@lc-dev90
Posted
Good work!
Some improvments:
In your desktop breakpoint, you can change your container properties to:
.container{ width: 600px; max-width: 90%; }
In your 768 media querie break point, you can change your container properties to: .container{ width: 350px; max-width: 95%; }
Don't forget to put a :hover effect in your signup button and the cursor pointer property!
Happy coding
0 - pierre-pellegrino• 970
@pierre-pellegrino
Submitted
I had to hide the footer when screen width is less than 375px, otherwise it would display in front of my page. I don't know how to fix it yet. Update : Fixed thanks to @lc-dev90!
Luiz Claudio Lopes• 505@lc-dev90
Posted
Great job!
Try setting out you max-height: 800px; value in your container to move you footer down.
You can change your media querie breakpoint from 375px to 500 +
But overall looks good!!
1 - Luiz Claudio Lopes• 505
@lc-dev90
Submitted
It's my last newbie challenge, and I made some improvments that I learn to made this challenge. I have used some Css animations, flexbox, and a little of Javascript.
Any suggestions are welcome!!!!
Luiz Claudio Lopes• 505@lc-dev90
Posted
thank you!!!! ^^
0 - Martina• 20
@maroctt
Submitted
Hey, I just finished this challenge. Any feedback and suggestions are welcome! ✨
Luiz Claudio Lopes• 505@lc-dev90
Posted
overall looks really good!
0 - David Rhyne• 270
@davidrhyne
Submitted
Thank you for reviewing the project! I have the following questions and I would really like some feedback to help improve this and future projects.
-
a. Was the theme switch intended to be 'draggable'? I was thinking it should have been because the slider has a hover effect, but I am not sure that is possible with the current input/radio solution.
-
b. Is there a simpler way to implement this switch and if so, would you please share a keyword to research?
-
For the hover behavior on the keypad buttons, from the spec, it looks like the button face and not the shadow should change with the hover. I got a version of that working, but I am thinking there may be an easier way to implement this and would like to know what I missed.
Thank you for any feedback, it is greatly appreciated!
Luiz Claudio Lopes• 505@lc-dev90
Posted
overall looks very good!
0 -
- Aldi Anugra Pratama• 115
@LordAldi
Submitted
any tips? whatever it is, maybe some best practice. thanks
Luiz Claudio Lopes• 505@lc-dev90
Posted
You can easily do this challenge using Css grid layout and the grid-template-area propertie.
-1172px your center cards are broken
0 - Raul A.• 130
@rarg22
Submitted
So far I think this is my cleanest challenge. Any feedback is much appreciated :) !
Luiz Claudio Lopes• 505@lc-dev90
Posted
You can use grid css property, mix grid template areas like this:
.content{ display: grid; grid-template-area: grid-template-areas: "left center1 right" "left center2 right"; } /* and align each of item with align-self and justify self*/0 - N3SHOW• 40
@Muxsin
Submitted
Luiz Claudio Lopes• 505@lc-dev90
Posted
You need to check your breakpoints @media queries. But still very good, you rock!
1
