All comments

• 

  P

  Daniil• 450

  @DaniilGurski

  Submitted 21 days ago

  What are you most proud of, and what would you do differently next time?

  This was my first project with test writing which was one of the main learnings for me. I learned how to use vitest and testing library a little bit and I think it will be useful in future, more large-scale projects.

  What challenges did you encounter, and how did you overcome them?

  I found it a bit difficult to find resources and examples in terms of testing, so mostly used AI to answer some not-so-obvious questions.

  What specific areas of your project would you like help with?

  I'd like to know how I can improve my tests. For this project I wrote one large unit test because it was difficult to figure out how to do it with the context provider involved.

  

  Responsive React Tip calculator

  2

  20

  

  SaakarX• 320

  @saakarx

  Posted 4 days ago

  Hey Daniil, Congratulations on completing the challenge

  • You should check for invalid inputs for the calculateTip function you have in your utils directory. When I tried to change the input to text and input letters it gave NaN error on the result section. Maybe add validation for the function arguments if they are not valid numbers, it should return 0 for both tip and total per person.

  • Also, on mobile devices the website does not look like the design provided for the challenge.

  Hope this helps, overall your solution looks great ✌️.

  0
• 

  Ian Rioba• 450

  @Rioba-Ian

  Submitted almost 2 years ago

  I faced these challenges while doing the nft preview challenge:

  • Being able to change the nft image color to cyan while hovering on it. Well on this part, I was able to change it's opacity but changing its color was a different thing.
  • being able to add a rounded colored radius around the avatar. One can achieve a border radius around the image by setting it to 50%, how can one also add a colored border radius ?

  I appreciate any feedback, Thank you.

  These challenges seem to always have something quite new under their sleeves.

  

  An nft preview card component using html and css

  2

  40

  

  SaakarX• 320

  @saakarx

  Posted almost 2 years ago

  Hello Ian Rioba, Congratulations on completing the challenge

  • You can add the color over the image by creating an overlay using a psuedo-element like .classname::after or .classname::before for reference pseudo-elements.
  • To add the border you can use border: 2px solid white you have used border: #fff; you haven't specified the border width specifying it will create the avatar image border needed.

  Hope this helps.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Ahmad_Mana• 320

  @UserAhmad2001

  Submitted about 2 years ago

  Hello Fellow Devs 👋, Any feedback is appreciated 😁, Just finished the easy bank landing page challenge, I tried to challenge myself by completing the website using react JS, it turned out to be harder than I thought. In this project, i:

  • Used ReactJS to organize the website content into multiple sections .
  • Used SASS the same way to style the website.
  • Made the site mobile-compatible.

  I had a hard time with:

  • Linking the CSS file inside the React application.
  • Finding out how to preview my webpage in github pages.
  • I couldn't find how to change the title of the webpage in react.

  

  Mobile-friendly Landing page of easy bank website

  2

  30

  

  SaakarX• 320

  @saakarx

  Posted about 2 years ago

  Hello AHMAD_MANA. Congrats on completing the challenge. You can change the title of the website under: public > index.html. Hope this helps, overall the website looks great. 🙏

  1

  This comment has received 1 replyShow
• 

  Xhanti Singatha• 30

  @Singatha

  Submitted over 2 years ago

  How do I make the hr thinner?

  

  NFT preview card component

  3

  31

  

  SaakarX• 320

  @saakarx

  Posted over 2 years ago

  Hello Xhanti Singatha, good job on the solution and great feedback above.

  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • And for making the <hr> thinner, use border-width: 1px 0 0; on the hr tag to make it thinner.

  Good luck with that, have fun coding! 💪

  0
• 

  Nazemrap• 200

  @Nazemrap

  Submitted over 2 years ago

  Got some trouble here and there, but in the end just one detail i couldn't fix is the alignment of the ETH and clock with there respective text... maybe i have played to much with some position/display things.

  

  Flex Grid for NFT preview

  3

  20

  

  SaakarX• 320

  @saakarx

  Posted over 2 years ago

  Hello Nazemrap 👋

  Good effort on this challenge! and great feedback above, In addition to programming feedback I would suggest you to:

  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • Change the alt attributes for the #profilo, as they don't any extra context for screen reader users. Changing to alt="" will make screen readers skip those avatar images.

  Good luck with that, have fun coding! 💪

  Marked as helpful

  0

  This comment has received 1 replyShow