@Medido1
Submitted
@tufcoder
@Medido1
Submitted
@tufcoder
Posted
Hello! First congratulations on finishing the challenge!
I'm having a little difficulty giving some specific code feedback, it seems to me that you know well what you're doing.
What I can say is that the features are not 100% ok. I had a little trouble getting the App to work, I think you just need to test different ways for the user to enter data and ensure a good flow. For example, I will list some difficulties below:
NaN
(Not a Number) and I have to click on ResetI believe that fixing this flow would make it much better to test/use your calculator!
@HusniddinAyubjonov
Submitted
@tufcoder
Posted
First congratulations on finishing the challenge, the size of the container was a little small. I looked at your code and have some general observations:
type="module"
in your script tag, you don't need to put it at the end of the body
and it also makes it easier to load data.json
.border-radius
of the back container and leave the front one with the normal result, this way the edge of the back container "disappears" and there is no effect at the end (although in the example it had that).Stay up the good work!
Marked as helpful
@mellingh
Submitted
What are you most proud of, and what would you do differently next time?
That I finished the project.
What challenges did you encounter, and how did you overcome them?
JavaScript
What specific areas of your project would you like help with?
.
@tufcoder
Posted
I really liked your project!
The gradient on the button is beautiful. The spacing was respected and its layout is practically identical to the example, excellent work!
Your code is very concise and well done, I only have one thing to say about your JavaScript.
You can use the defer
attribute within the script tag and do not need to use DOMContentLoaded
within your script. It's just an alternative.
Keep up the good work!
@AndreyDimitrov3
Submitted
@tufcoder
Posted
Hi, I think your work is good but I have some observations:
Keep coding!
@Maobugichi
Submitted
What are you most proud of, and what would you do differently next time?
Proud that i was able to decipher this in way less time than i thought!
What challenges did you encounter, and how did you overcome them?
my grid sizing
What specific areas of your project would you like help with?
nothing
@tufcoder
Posted
I have some observations about your code:
section
or article
instead div
.Well done, keep coding!
@uzzalyafi
Submitted
What are you most proud of, and what would you do differently next time?
What you might be proud of:
What challenges did you encounter, and how did you overcome them?
Here are some potential challenges you may have encountered while working on your project, along with strategies for overcoming them:
@tufcoder
Posted
Very good work!
The only thing that I observe is the <br />
tag in the title. Because you use <span>
the default display is inline. To avoid use <br />
try to apply in the span
the display: inline-block
or display: block
so the span will be in a "new line".
To me your HTML is Semantic, if you like I have this link to more information.
@BankuBa
Submitted
What challenges did you encounter, and how did you overcome them?
centering the image to resamble the challenge was difficult
What specific areas of your project would you like help with?
centering items while preserving the background with tailwind css
@tufcoder
Posted
Hi, I take a look in your work, its good but I need to do some observations:
I don't know if you are using Tailwind to practice or not, but for a simple layout I think less is more to avoid adding complexity. But if you are practicing Tailwind, just ignore me.
What are you most proud of, and what would you do differently next time?
I'm proud because I managed to complete the challenge
What challenges did you encounter, and how did you overcome them?
I encountered problems when centering the elements, but I managed to solve it
What specific areas of your project would you like help with?
I would like to receive feedback on my code
@tufcoder
Posted
Your work is great! The only thing that I saw is the lack of Semantic Tags in your HTML. Your file has only a main tag, but you could use more like a section, header etc. This link helps a lot https://learntheweb.courses/topics/html-semantics-cheat-sheet/
Thats it!
@KanjiSaiko
Submitted
What are you most proud of, and what would you do differently next time?
I would add a responsive layout, something I haven't mastered yet
What challenges did you encounter, and how did you overcome them?
No complex challenges I ended up encountering
What specific areas of your project would you like help with?
Best practices with CSS, ideas on how to implement it more effectively and less "polluted"
@tufcoder
Posted
Very good work, you know the path to continue evolving.
The only thing that I observe in your css styles, is to avoid using "ids", prefer use "classes" for multi styles. Practice will bring the confort to use classes.
Your html has the class "nut" in the table, but that class isnt present in your css file. Maybe you remove then, but remains in your html.
Marked as helpful
@anonymousbazooka
Submitted
@tufcoder
Posted
Hi, I'll list bellow some considerations:
Thats all, good work!
@AcharaChisomSolomon
Submitted
What are you most proud of, and what would you do differently next time?
Just found out about CSS clamp().
What challenges did you encounter, and how did you overcome them?
None
What specific areas of your project would you like help with?
None
@tufcoder
Posted
Good work, bellow is my considerations:
@ChurchillV
Submitted
What are you most proud of, and what would you do differently next time?
Pleased with how I was able to quickly learn SASS and use it for this project.
What challenges did you encounter, and how did you overcome them?
Struggled in centering the qr code components on the screen, read on article on centering divs to figure it out.
Creating a responsive layout for the component proved a bit challenging. Ultimately I only made a minor change to enable responsiveness. (Literally just changing the entire background colour to blue)
What specific areas of your project would you like help with?
@tufcoder
Posted
Very good layout, but I think you need to pay more attention in:
That's my improvments to you, good work and happy coding!