Skip to content
OverviewSolutions
8
Comments
4
Abhishek Kumar
@abhixdev

All comments

  • DevEtudaye•10
    @DevEtudayeSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    Am proud i was able to attempt the challenge but however i would love to work on my stylings next time to get the exact card design and also to make the css less cumbersome as possible

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

    I was faced with the timing challenges as i have busy schedules.

    What specific areas of your project would you like help with?
    1. Color transparency usage and dimming to get the exact colors used during the challenge
    2. Styling and the html used to achieve the spacing between the result summaries e.g (the space between "Reaction and 80/100) and the entire divs especially the background colors of the entire divs

    Result summary component using html and css

    1
    Abhishek Kumar•100
    @abhixdevPosted 7 months ago

    Hey, congratulations on completing the challenge! I had a chance to review your code and noticed a couple of issues that I'd like to address. In your repository, I observed the presence of two HTML files. To streamline the structure, I suggest deleting the current index.html file. Subsequently, rename your existing HTML file Result summary solution.html to "index.html" to serve as the main entry point of your website.

    Additionally, for consistency and clarity, I recommend renaming your CSS file from Result summary solution.css to "styles.css". Once renamed, ensure that it is properly linked to your index.html file using <link rel="stylesheet" href="./styles.css" /> By making these adjustments, you should be able to preview your website accurately. Let me know if you have any questions or need further assistance. Happy Coding!

    0
  • Abhishek Kumar•100
    @abhixdevPosted 7 months ago

    Hey, really nice work! I noticed some irregularities in your code. The color of the box is pure white. i.e .card {background-color: white;}. The text color of title "HTML & CSS FOUNDATIONS" should be black, but on hover, it should change to yellow. First, remove style="color: hsl(47, 88%, 63%)" and add a new class for the title. I suggest adding a class named "title" inside your style.css

    .title:hover {
  color: hsl(47, 88%, 63%);
}

    Don't forget to add the "title" class to your HTML element.

    0
  • Abhishek Kumar•100
    @abhixdevPosted 7 months ago

    Hey, great job! I hope you're enjoying the Frontend Mentor challenge. I checked your code and found some small bugs. First of all, the HTML boilerplate template, asset images, fonts, and colors are available for download in a zip file. You just need to download the zip file and make changes inside it. There is a file named style-guide.md which will tell you all you need to know about the colors, fonts, and text sizes used in the design. You need to import fonts into your project. Also, the image of your project is not linked properly. You need to mention the relative directory of the image inside your <img> tag. "<img src="./handsome-bearded-guy-posing-against-white-wall_273609-20597.avif>" should do the job for you. Happy learning!

    0
  • Sabine•260
    @LadyoftheCaveSubmitted 7 months ago
    What specific areas of your project would you like help with?

    Because I am a newbie on JS. All tips are welcome.

    Age calculator html css js

    1
    Abhishek Kumar•100
    @abhixdevPosted 7 months ago

    Hey there, great job on your work! I noticed a tiny bug in the code related to error handling. Specifically, when a user enters an invalid month or year, the error message should prompt them to "Enter a valid Month" or "Enter a valid Year". To address this, I suggest making changes to lines 86 and 93 within the app.js file.

    Additionally, I observed that the form is currently accepting dates greater than 31, and month greater than 12. To ensure accuracy, you might want to consider adding a condition like "if(day > 31)" or "if(month>12)" in the logic before proceeding to calculate the age.

    Marked as helpful
    0