Skip to content
OverviewSolutions
7
Comments
9
C1SLR
@C1SLR

All comments

  • P
    Peetawit Vongchanapibul•160
    @pete13232
    What are you most proud of, and what would you do differently next time?

    This challenge contains many card themes, so I structured my HTML using the BEM (Block-Element-Modifier) methodology to keep the codebase maintainable and scalable. Additionally, I organized related CSS sections with comments for better readability.

    Here are some example:

    <div id="card-1" class="card card--purple">
  <div class="card__head">
    <img src="" class="highlight" alt="daniel_image">
    <div class="card__profile">
      <p class="card__profile-name"></p>
      <p class="card__status"></p>
    </div>
  </div>
  <div class="card__message">
    <h1 class="card__message-title"></h1>
    <p class="card__card__message-body"></p>
  </div>
</div>

    Testimonial grid section with responsive design

    1
    C1SLR•160
    @C1SLR

    Your all code and solution are out class and awesome my current skill is not good at that time to review your this project but one thing i want to ask how did you know that you should create a three responsive layouts or that was your personal decision .

  • C1SLR•160
    @C1SLR

    I love this types of Project but i would like to advice you some thing is that you write a lot of css it's 180 lines tailwind css is advance and good web dev tool for css which saves your times alot . But in this project there is a big flaw you didn't met mobile design requirements as given in solution which is not acceptable in terms of ui/ux and responsiveness

  • C1SLR•160
    @C1SLR

    seems like you don't love to code maybe because your code is not clean you didn't updated readme even you didn't manage your folders properly and your 2 cards are out of the position you need to work on this project again overall code rating 4/10

  • C1SLR•160
    @C1SLR

    instead of using so many links for fonts you should import through css when its so many links in head tag its make your code dirty and unhealthy make sure to use semantic way of coding and your css coding have also some problem like you given 100 percent height manually giving a height to any div is not a good practice overall your code rating is 4/10 may be you should work on ui also its important .

  • Alperen Aksoy•150
    @aksoyalpi
    What are you most proud of, and what would you do differently next time?

    styling tables and lists

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

    is the css like i implemented it good or too complicated?

    Responsive recipe page with table and lists

    1
    C1SLR•160
    @C1SLR

    i would like to say that you do a lots of mistakes is your code as such in your css body you didn't set min height properly and your project image path is not valid make sure when applying images you set the path correctly and after if you changes something in navigation or may be folders you have to check all the paths again if it's working or not another thing i would like to say about your ui as such in this project there is in bottom was a table with some lighten border in your code this is very darken which is interacting attentions which is not good in the sense of ui making may be you should focus on it overall i would like to rate your coding 5/10 and i would like to answer your question make your css class meaning full let's take example of your page if you have to create a class for your specific section maybe you should create a class name with their main heading like Nutrition for that just create nutrition class if there is multiple headings in same page then create their parents classes with name of nutrition parent or child so on! your css classes defined too short which is hard to understand for another coder to see or check your code .

    Marked as helpful
  • C1SLR•160
    @C1SLR

    your code need to be fix in way of semantic html you used li and ul tags instead you should use buttons tag at early stages its will show their impact but make you code neat and clean as much as possible and second i would like to say your css is looking good and well defined classes but hitting ul li types tags directly is not a good practice overall your code is 10/7 .

    Marked as helpful
  • Jose Ramon Caravaca•40
    @jrramoncp
    What are you most proud of, and what would you do differently next time?

    This is my first project using TailwindCSS and I was impressed by how flexible and fast is it. Next time i would make a more complex design

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

    The harddest part was to change my ussage perspective from CSS vanilla to this framework

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

    I need feedback on my classes and semantic ussage, and know if I can improve anithing for the future

    Social Links, made with semantic HTML and TailwindCSS

    1
    C1SLR•160
    @C1SLR

    First of all your code is not matching with the give solution but yeah that's not the problem As beginner i would like to say about the things which are missing and need to be fix in your code

    Your UI according to social cards or links is not good like you given your buttons so much extra margin and paddings there need to be reduced by 30-40 percent for a decent look overall your code is good and semantically looks clean i would like to rate your coding skill 7/10 considering only html and CSS with tailwindCSS.

    Marked as helpful
  • P
    skhbabez•60
    @skhbabez
    What are you most proud of, and what would you do differently next time?

    Next time, I am going to put more effort into the accessibility side of the project, using the accessibility features of the browser developer tools more effectively. I learned a lot about responsiveness from this project, and I am looking forward to diving deeper into this topic.

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

    "I focused on making the layout responsive, especially the font sizes. After doing some research, I decided to avoid media queries and use clamp() instead.

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

    I'm especially unsure about the layout using semantic elements. I'm not sure if I used them correctly, such as using <article> for the card and <section> for the main content.

    Blog card using html and css

    1
    C1SLR•160
    @C1SLR

    as my coding skills and review your code is perfect and i didn't find any mistakes in your code and to be honest most of the properties you used in your project i am seeing these properties first time in my life sure i will use in my further projects your code rating from my side is 10/10. over all your semantic html all is good and best possible option with the perspective of semantic html

    Marked as helpful
  • C1SLR•160
    @C1SLR

    At very first i have to say that you should work on it your read me file and one thing you skip in this solution is css property called box shadow and another feedback i should give you that targeting direct elements at early stages will make you in big trouble you should avoid targeting p or h1 elements directly you should create class to more customization over all 5/10 of your solution