This is my version of the quiz app project. I'm looking forward your feedback ! :)
MaximilianoDanielGarcia
@MaximilianoDanielGarciaAll comments
- @AlinaAlexandraVizireanuSubmitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi Alina, great job!
One little thing, I've noticed that when you submit an answer without making a selection this is moving on to the next question.
The rest looks excellent. Congrats!
1 - @paupalazzesiSubmitted 10 months ago
"I'm finding it difficult to make it responsive. Could anyone help me with this task?"
@MaximilianoDanielGarciaPosted 10 months agoHi María Paula, good job!
In response to your question. You need to use media queries to change its design on different screen sizes. Here an example:
@media screen and (width <= 650px) { .container { display: flex; flex-direction: column; } }
The code above changes the properties of your
.container
class when the screen width is less or equal than 650px. Visit w3schools for more details about media queries.I hope these was helpful to you.
Marked as helpful0 - @camigamboaSubmitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi @camigamboa, great job!
There is an very useful browser extension called Perfect Pixel that allow you compare with the design image and thus see the exact dimensions. I use it and I recommend it to you.
I hope this suggestion is useful for future projects.
Marked as helpful0 - @AbrahamTegeneSubmitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi @AbrahamTegene, good job!
If you are wondering how to center it, let me show you 2 easy ways:
- Using Flex Box
body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; }
- Using CSS Grid
body { display: grid; place-items: center; min-height: 100vh; }
For the attribution part, you can do this:
.attribution { position: absolute; bottom: 25px; }
I hope these was helpful to you.
1 - @SpecticallSubmitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi @Specticall, great job!
I've noticed that the main image is cut off and that's due you are using
object-cover
instead ofobject-contain
tailwind class. After you have replaced that class you will need to add abackground-color: #0f0f0f;
and az-index: -1;
onheader
tag.After you apply these suggestion it will better. The rest looks very good.
Marked as helpful0 - @Rohanbasnet12Submitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi Rohan, good job!
Here are some suggestions that might interest you:
-
I noticed that your
#profile-card
should be taller, try withmin-height: 612px;
. -
Swap the background colors of
#profile-card
and buttons. The card should be dark-grey and buttons grey. -
You should change the buttons' height, try with
height: 45px;
. -
Semantically, buttons should be anchor tag because they are redirectioning you to another sites.
After you apply these changes It will look better.
Marked as helpful0 -
- @imonBlazeSubmitted 10 months ago
Explanation guidance wasn't very clear.
@MaximilianoDanielGarciaPosted 10 months agoHi @imonBlaze, good job!
Check this code to improve your background image and to center the card:
body { display: grid; place-items: center; min-height: 100vh; background-size: contain; background-color: hsl(225deg 100% 95%); background-position: center -25%; }
The
background-size: contain;
property resizes the background image to make sure the image is fully visible.After you apply these it will be look better.
0 - @EverezzeSubmitted 10 months ago
Besides the fact that I forgot to adjust the error messages If you have any insight to improve the JS part let me know (there might be a more efficient way or whatever) or even the CSS part! It was a fun project anyways
@MaximilianoDanielGarciaPosted 10 months agoHi Lamar, great job!
If you're wondering how to center it, you can use either Flex Box or CSS Grid.
- Using Flex Box:
body { display: flex; justify-content: center; align-items: center; min-height: 100vh; }
- Using CSS Grid:
body { display: grid; place-items: center; min-height: 100vh; }
And if you want to adjust your background image to the screen size just add
background-size: cover;
After you apply these suggestions it will look better.
1 - @EverezzeSubmitted 10 months ago@MaximilianoDanielGarciaPosted 10 months ago
Hi Lamar, great job!
If you're wondering how to center it, you can use this:
body { display: grid; place-items: center; min-height: 100vh; }
Then, remove the padding from
body
that isn't longer necessary.I hope these was helpful to you.
2 - @laplatacryptosSubmitted 11 months ago@MaximilianoDanielGarciaPosted 11 months ago
Hi @laplatacryptos, great job!
If you are wondering how to centered, you could use CSS Grid:
body { display: grid; place-items: center; min-height: 100vh; }
Then, you should remove the
margin-top: 225px;
from yoursection
tag.After you apply these It will look better. Regards!
0 - @cloudpc7Submitted 11 months ago
In this solution what I found difficult was how to make it better using less code and create the symantic html that is most simple and basic. I know that this should be an easy challenge so I wanted to make it less code and simpler. I am unsure of the width and the height of the code.
I believe I did a better job this time around. if there is any critiques that need to be made as far as detail I would like to improve. thank you for all the feedback.
@MaximilianoDanielGarciaPosted 11 months agoHi @cloudpc7, good job!
Just 2 little things:
- If you want to look it centered, you should add this:
body { justify-content: center; min-height: 100vh; }
- I think the margin top on image should be
margin-top: 1.5em;
If you want to know the exact measurements for the design, you have a figma file in the starter folder.
I hope these was helpful to you. Regards!
Marked as helpful0 - @ivan2600Submitted 11 months ago@MaximilianoDanielGarciaPosted 11 months ago
Hola Iván, buen trabajo!
Tengo unas pequeñas sugerencias para mejorar tu desarrollo.
- Para que se vea centrado sólo te falta agregar
min-height: 100vh;
a los estilos delbody
. - La fecha calculada debería estar alineada a la izquierda. Para hacerlo sólo debes reemplazar la propiedad
margin: 0 auto;
(esto hace que se centre) que tenés en.results p
y agregarle un espaciado para que no se pegue al contenedor, algo así:
.results p { padding-left: 60px; /* margin: 0 auto; quitar */ }
- Agrega
cursor: pointer;
al botón principal.
Otro detalle que noté es que no hay validaciones en los inputs. Podrías utilizar inputs de tipo
number
que te permiten restringir el rango de números válidos. Te dejo este link que contiene ejemplos muy útiles de cómo utilizarlo.Espero esto te sea de ayudar. Saludos!
Marked as helpful0 - Para que se vea centrado sólo te falta agregar