Skip to content
OverviewSolutions
11
Comments
13
P
jasoneczek
@jasoneczek

All solutions

  • Submitted 6 months ago

    Time Tracking Dashboard

    • HTML
    • CSS
    • JS
    1

    I am always looking to improve accessibility best practices.

  • Submitted 7 months ago

    Newsletter Signup

    • HTML
    • CSS
    • JS
    1

    For the success screen, I used JavaScript to toggle the display from none to flex, and the component from grid to none. I don't know if this is best practice for this type of component and if it meets accessibility standards. Any advice on this, or anything else would be greatly appreciated.

  • Submitted 7 months ago

    Article Preview Component

    • HTML
    • CSS
    • JS
    1

    Besides what I mentioned above, accessibility standards. I don't think my focus is correct for the social media links. I'm not sure if focus should be trapped within this div when it is active. Of course, I am always open to all feedback and suggestions for improvement.

  • Submitted 7 months ago

    Meet Landing Page with Grid, Flexbox, and Fluid Type

    • HTML
    • CSS
    1

    I would really like to know if my use of and tags is ok for this project. Especially this piece of code in the hero section:

    and within the hero I have this ``:

    Group Chat for Everyone

  • Submitted 7 months ago

    Testimonials Grid Section

    • HTML
    • CSS
    1

    I used the and elements, and I'm wondering if how I used as the wrapper for the entire card is OK, or if the cards also need to be's in a `` for this challenge.

  • Submitted 8 months ago

    Responsive Grid Four Card Feature Section

    • HTML
    • CSS
    1

    There were several decisions I made that I had doubts about being the best practice, so I will explain my thought process, and if you see how I can improve please let me know:

    I used flexbox in the body, and grid only in the wrapper which contains the 4 cards. I needed just a one-directional vertical alignment to stack my and. Inside my element, I have an which contains my and. I didn't see any need to include those in and complicate the grid.

    I do wonder if this is good use of the element. I thought since there is no navigation here, wouldn't be correct.

    For the cards I used and for each card. Is this the right situation for use a list for cards, and if so, when is it correct to add the attribute role="list"?

    Implementation of the grid itself. It was easy to get the layout here, but it could have been done also using `grid-template-areas'. I like using the shorthand:

    .card-1 {
    grid-area: 2 / 1 / 4 / 2;
  }

    But if this is not recommended then I want to correct my ways.

    Thank you!

  • Submitted 8 months ago

    Product Preview Card

    • HTML
    • CSS
    1

    A few doubts I had are whether the default image should be the mobile or desktop version. I went with mobile, which works, but I don't know if that is the best practice. Button type: "submit" or "button"? I chose button because I don't think adding to cart submits data to the server. I've been trying to use BEM more often. Any feedback on how I named my classes would be of great help. Any other suggestions or improvements are also very welcome.

  • Submitted 8 months ago

    Recipe Page with custom properties, dynamic padding and layout

    • HTML
    • CSS
    1

    I'm grateful for any feedback on how to improve. One thing in particular I couldn't figure out is how to vertically center the bullet points on the unordered lists. In the design they appear to be center aligned with the text in the list item rather than aligned to the top of their box.

  • Submitted 8 months ago

    Social Links Profile

    • HTML
    • CSS
    1

    Besides the problem mentioned above, any feedback on accessibility, best practices or any ways to improve in any way are massively appreciated.

  • Submitted 8 months ago

    Blog Preview Card with fluid text sizes

    • HTML
    • CSS
    2

    Definitely the fluid text. At this point it "works", but I would really love feedback on my code for this, because I am sure it could be improved. But also any feedback on any other areas in my code would be greatly appreciated.

  • Submitted 8 months ago

    QR Code Component

    • HTML
    • CSS
    1

    Naming conventions: Is it ok to use div here instead of an element like article? Are my names for classes appropriate? How about my CSS variable names?

    CSS selecting: For nested elements, is it right to select just the class name for the element I want to select (e.g., .card-header), or should I include the parent's class? (e.g., .card .card-header, .footer .footer-link), etc.

    Keeping the footer on the bottom: My solution was position: fixed, bottom: 0. Is there a better way? Adding flex-grow 1 to the main element also worked, but it seemed like not the best method.