@omarkl95Submitted 7 days ago
Katarzyna Kaźmierczak
@KasiaKaz14All comments
- @KasiaKaz14Posted 6 days ago
Hey, I must admit it is a really good job, and it almost looks like the template! But I would recommend you one important thing: if you create root, and give names to every elements - don't do it this way: --white: hls(sth, sth, sth); Why? Because it sometimes happens that we need to change a color for an element or section, and it can't look like this: --white: #orange(new color), because the name will be confusing for another developer. It's better to do this way: --font-color: #color; --bgColor: #color; It's definitely more readable for everyone
Marked as helpful0 - @BrandonJr20Submitted 23 days agoWhat are you most proud of, and what would you do differently next time?
Aprendi a utillizar css grid y pues la introduccion sobre flexbox me dejo muy ansioso, quiero aprender mas!
What challenges did you encounter, and how did you overcome them?El uso correcto de grid column y grid rows
What specific areas of your project would you like help with?Posiblemente en la sombras de las cartas
@KasiaKaz14Posted 23 days agoHey, there are some elements we can improve:
- The background color. It is not white. You can find the color in the style-guide.md: Very Light Gray: hsl(0, 0%, 98%)
- Both texts: reliable and powered have the same font-size, and different font-weight: 200 and 600. Look also at the font color: it's this one - hsl(234, 12%, 34%);
- The same with the description under "powered..": hsl(229, 6%, 66%). Look also at the psoition of this text. You can add width and text-align: center
- The 4 containers are higher and more narrow
- Descriptions in container have this color as well: hsl(229, 6%, 66%)
- box-shadow also should look a bit different
0 - @luqmanx1998Submitted 24 days agoWhat are you most proud of, and what would you do differently next time?
I'm most proud of actually completing this project haha 😂. This challenge ended up being much harder than I'd expected for a few reasons, particularly in the HTML/CSS, but it's probably just due to my inexperience. I honestly wanted to give up , but at the end of the day a dev can't give up so I managed to push myself to pull through. Again, I made some silly mistakes that ended up making this challenge MUCH harder than it should be, but nonetheless, I'm satisfied with the end result !
What challenges did you encounter, and how did you overcome them?Surprisingly, the hardest part of this challenge wasn’t the JavaScript—it was getting the HTML/CSS layout just right! Working with
What specific areas of your project would you like help with?overflow: hiddenin the parent container of the slides added complexity to positioning and dimensions. I had to restructure my HTML and CSS multiple times to align with the design while keeping the slider functional. Initially, I struggled with positioning and image dimensions until I realized I’d forgotten to add a fixed height to the slider parent container. Without this, elements disappeared withoverflow: hiddenapplied. Lesson learned! 😂 I’ll never miss that step again. I tried getting this pixel perfect, but I couldn't without a figma file as it would've taken too long, and also to be honest I would have to restructure my HTML/CSS to accommodate for pixel perfection.Any feedback at all is much appreciated!
@KasiaKaz14Posted 24 days agoHey, that's a really good work, and great you did not give up, and made it to finish the whole project. You can really be proud of you! HTML a CSS are connected with JS, so if you do not know them yet - I recommend you to start learning. Btw, if you really do not know them/have no experience I am impressed you were able to finish! Coming back to the project. The Javascript works correctly, but remember you can write functions inside the addEventListener. You do not have to separate it. If it comes to the whole design, there are a few things you can improve in CSS. Just take a look at the quotation mark, the text, slider or the "purple something" behind the img :D I must admit it's a good job, keep working :)
Marked as helpful1 - @osuulolaSubmitted 24 days ago@KasiaKaz14Posted 24 days ago
Hey, there are a few issues you need to polish:
- The whole container is wider than in the template
- Img, name, location and title are a bit bigger
- The buttons are in the wrong place. They need to be in the center of the design. On the template they are also a little bit wider and higher. Maybe they seem smaller because the whole container is wider. So you need to think, if you want to leave the width of the container or make the buttons a little bit bigger
- Give a bigger border-radius That's all. In general - good job. Keep working!
Marked as helpful0 - @iMohamed-ESubmitted 26 days ago@KasiaKaz14Posted 25 days ago
Very good work. Your design almost look like the template. Really impressive. But I can give you a small tip for the future. Look at the 'title' and description. Your version is written more tight than in the template. Maybe you have heard about the line-height. This way you can expand the space between the text's lines. For instance, you can apply line-height 1.5; It will be enough. Congrats again and good luck in the future
Marked as helpful1 - @arthur-mendoncaSubmitted 29 days agoWhat challenges did you encounter, and how did you overcome them?
Designing correctly the UI and adjusting the logic.
@KasiaKaz14Posted 29 days agoHey, if I understand the task correctly - the calculator counts how old you are and displays the age on the screen after adding a date. When I entered my date of birth I received a wrong result. I did it a few times, and each time the app gave me another, wrong result. After every try I had to refresh the site because when I entered a date after date - the app was breaking and started counting like a crazy
Marked as helpful0 - @talxdoSubmitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Hey, not bad, but take a look on the template. The text is smaller and does not touch the border of the container. Just make the font-size smaller, and everything will be ok
0 - @rashed-miaSubmitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Hey, very good job. The design looks like the template! Really impressive. But there is a detail we can improve. Just look at the background. It should be lighter. If you could not find this info, here you are: #d5e1ef
0 - @ernieJohnSubmitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Hey, you did not write the code at all. No HTML, no CSS styles we can see here. You need to know the basics of the technologies mentioned above to start the project. If you know them, you have startup files that really help you to write the code. Just read the readme.md and style.guide files to arrange the design in a proper way. On the template you can see how it should look like
0 - @coding-brianSubmitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Hey, really nice work. I think I have nothing to add here. The project looks exactly like the template. Impressive.
0 - @AustinjnrSubmitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Hey. Great work, but there are a few details we can improve. Just look at the name of the girl in template. It is a little bit smaller. We can just apply h3 for instance. We can also make bigger container, and <a> links, and also larger gaps between <a> links. It just make the content more readable. Certainly, it's my opinion. You can apply it, if you want to. Reagards ;)
Marked as helpful0 - @Lpluiz123Submitted about 1 month ago@KasiaKaz14Posted about 1 month ago
Really nice project, and I think there is no elements to improve. Maybe the whole container could be a little bit longer. It can make it more readable. I can recommend you to create a project which will contain your own social media. This way people can meet you and maybe invite you to a collaboration? Consider it
0
