some tips or resources to increase my javascript skills will be helpful and if i made any errors in this particular challenge any tips to correct them will be greatly appreciated
Hector Ramirez
@hectorlil48All comments
- @Sandaruwan7056Submitted about 1 month agoWhat specific areas of your project would you like help with?@hectorlil48Posted 30 days ago
You can improve the code by using event delegation to minimize the number of event listeners, enhance performance, and simplify the logic. Refactoring your error handling into reusable functions will also help eliminate redundant code. Using more descriptive variable names for clarity would be beneficial, like renaming Element to tipButton. Lastly, breaking down logic into smaller functions would make the code easier to maintain. Keep up the good work!
Marked as helpful0 - @ClaudiaRamirezDSubmitted about 1 month ago@hectorlil48Posted about 1 month ago
Good job on the dashboard. It looks good in every size. The only thing I would say is on the card components; you might want to add a border radius to the top of the dark blue cards. Other than that, keep up the good work!
0 - @Gehad28Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Using JS to validate the form and apply style for errors
What specific areas of your project would you like help with?Any feedback is appreciated :)
@hectorlil48Posted 3 months agoYou did a good job. I noticed that your image disappears when you reach your breakpoint. I recommend making the breakpoint larger and keeping it in the mobile design. I added a breakpoint at 1000px so that when it switches, it would look good without affecting the image. Also, when it's on desktop, you should add a min-width so it won't stretch with larger screens.
Marked as helpful0 - @Medido1Submitted about 1 year ago
any feedback is welcomed
@hectorlil48Posted 3 months agoGreat work on this, but you may need to adjust the spacing. On desktop, there's too much space between the share div and the paragraph. Also, don't forget to change the border-radius on the image.
Marked as helpful1 - @hectorlil48Submitted 4 months agoWhat are you most proud of, and what would you do differently next time?
I'm proud that I was able to make it responsive. And finally getting to use the calc() function, makes me understand it better.
What challenges did you encounter, and how did you overcome them?I don't have much experience making a website responsive. I got to work with media queries and use the CSS calc() function to make my fonts responsive.
What specific areas of your project would you like help with?All feedback is welcomed!
@hectorlil48Posted 3 months agoThank you for the comment. When I was doing the project, I looked up how to make my font sizes more responsive and that's what I found. But from doing more projects I discovered that just using media queries to change the font will be easier and confusing. For the box shadow, I use this generator online and try to make it look as close as possible, they provide it with the -webkit and the -moz. I will look into my project's height but have begun using clamp and media queries to make my projects more responsive.
0 - @Stephanie-DennehySubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am proud that I was able to make the design adjustments as the page shifts from desktop to tablet and to mobile.
What challenges did you encounter, and how did you overcome them?I had some issues with getting the correct images to show up on the correct layouts. I was able to hide the images using media queries so that I only got the images needed based on the layout size.
What specific areas of your project would you like help with?I would love to know if there is a different way to lay out the images and have them swap at the different size breakpoints. (I used media queries and hid images at different breakpoints). Any suggestions for other ways to do this would be appreciated!
@hectorlil48Posted 3 months agoGreat work on this project. I encountered some difficulties with the images when trying to hide them. I noticed that the dividers expand based on the screen size in your solution. I suggest putting step one and two in a div that wraps both the number and the bar. After making this change, I set the circle's height and width to ensure it stays the same size.
Marked as helpful0 - @Ayyubiy90Submitted 4 months ago@hectorlil48Posted 4 months ago
Hey, it's looking good. Good job! You might want to add another breakpoint. Maybe make it a 2-column layout for tablet size. Using animation will help smoothen the transition between breakpoints.
0 - @DiegoAlveesSubmitted 4 months ago@hectorlil48Posted 4 months ago
It looks good. Good job on making it with React. I just learned it and I hope to use it on my next project.
0 - @TarekGawish1Submitted 5 months ago@hectorlil48Posted 4 months ago
Hey it’s looking good. Just need to make it responsive. Try using responsive units for font sizes with em or rem units. To make the design 1 column, you need to use media queries to set the breakpoints on when you want it to change. Over all it looks good, just keep at it.
Marked as helpful0 - @MarqCodesFromZeroSubmitted 4 months ago@hectorlil48Posted 4 months ago
Hey, it's looking good. I would suggest that you wrap everything inside a div like you have but inside of the main tag. Having proper semantic HTML tags will help screen readers navigate through your page. Also taking a look at the style guide and taking the time to add all the colors to a :root tag would be helpful. That way you always know what colors to use from the design.
0 - @FrankBonannoSubmitted 4 months ago@hectorlil48Posted 4 months ago
Great job on completing the social links project. I just finished it too and I was looking at your code. I noticed that you have two resets going on and you don't need both. I also noticed that you are targeting some of your HTML tags more than once and that can probably throw you or someone looking at your code off. Try using classes to target some of your tags. And you should look into using semantic HTML tags, that would help a visually impaired person navigate through your website. If you have any questions, please let me know.
0 - @DolbyTheSheepSubmitted 4 months ago@hectorlil48Posted 4 months ago
It looks good, you should consider using a div tag to separate the content from the image instead of using another main tag. The main tag is semantic and should be used to separate the main content of the page and the div tag is not semantic and should be used to group content related to each other.
0