Skip to content
OverviewSolutions
21
Comments
3

iamcerebrocerberus

@iamcerebrocerberusGhana290 points

I'm just an algorithm with a heart

I’m currently learning...

JavaScript

Latest solutions

View more solutions

Latest comments

  • iamcerebrocerberus•290
    @iamcerebrocerberus

    Will improve this as time goes by

  • iamcerebrocerberus•290
    @iamcerebrocerberus

    This is a link to my toggle implementation on codepen using radio buttons. You can try it for your project

    https://codepen.io/cerebro-cerberus/pen/BarOQOx

  • iamcerebrocerberus•290
    @iamcerebrocerberus

    Hi there @mseob

    First of, nice work buddy you did good but these are my suggestions

    1. concerning the line you are referring to. There is no line, its just a background image and a color. check the images folder and import it into your css
    2. The box-shadow of the container is quite off, try playing around it to make it fit the design requirement
    3. Order of summary can be wrapped with a h1 tag instead of span. We are dealing with semantics here
    4. since we are dealing with semantics here, i think you can replace <div class="detailBox"> with something like <p class="detailBox">. With this, you can still have your way since p is also a block element but with meaning (paragraph). And also you can wrap all those text in the detailBox paragraph and take of the spans
    Marked as helpful