JastAbdullrahman
@Alord81
All comments
- JastAbdullrahman• 180
@Alord81
Posted
Hello @nanatotibadze,
Congratulations on completing this app✨.
I will give you some notes that I saw when I tested your app:
-
You can use the Placeholder for input fields without giving it a preset value.
-
You should have taken the time to think about the equation that might lead you to the same results found in the design. After experimenting with the numbers that were used in the attached pictures in the project, I did not reach the same result.
-
The last note is also related to the input field, as it does not accept the
Floatingtype.
Thank you,
0 -
- Vadym• 30
@LLUXER
Submitted
I like it very much, but please don't pay attention to the fact that I didn't optimize for other screens, I don't know how to do it and it's difficult, but thanks if you write what flaws you saw
JastAbdullrahman• 180@Alord81
Posted
Hi @LLUXER,
I think I can help you with the flaws I saw :
You can make the main tag
<main>contain all the elements instead of making them dependent on the body directly if all elements belong to the same section.Also if you want to optimize for other screens you can search for CSS Media Queries.
If you want to make the box in the middle of the page you can add
min-height:100vh; display: flex; align-items: center: justify-content: center; to the body.HOPE I COULD BE OF HELP :)
HAPPY CODING!
Marked as helpful
1 - JastAbdullrahman• 180
@Alord81
Submitted
Hello there 👋. This is the first project that I use JavaScript with it. And still, I don't finish my JS course, but I like to test my level in Js.
The project took me about 5 hours to finish
The project took me about 5 hours to finish creating the page and programming it with little breaks.
And I'm happy for this result.
If you have any comments on anything I hope to give me any comment to help me to improve my level or reduce the codes or the project in general.
have a nice day.
JastAbdullrahman• 180@Alord81
Posted
I think I forget to change the width of the box ;{
0
