Skip to content
OverviewSolutions
13
Comments
11

Peter Bachman

@PeterBachman100200 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • P
    Anton Vasilache•580
    @Antonvasilache
    What are you most proud of, and what would you do differently next time?
    • Managed to dynamically generate the html content with relative ease
    • Switching between the different data sets using JS
    • Would probably think more about how to insert the html content, before building the page, could probably make a more simple structure
    What challenges did you encounter, and how did you overcome them?
    • Switching between the different views (daily, weekly, monthly) - did it by emptying the html containers and repopulating them, using the index to match the data with the respective container
    data.forEach((item, index) => {  
            containerWrappers[index].innerHTML = '';
            appendItem(containerWrappers[index], item, view);                           
        })
    • Applying an active state to the currently selected view - I did it by deleting all the states when clicking, before applying the current one
    card.addEventListener('click', ()=>{        
        cardSelect.forEach(otherCard => otherCard.classList.remove('active'));

        card.classList.add('active');
    What specific areas of your project would you like help with?

    Could probably use a way of handling the dynamic content with less loops.

    Time Tracking Dashboard with CSS Flex, Grid

    1
    Peter Bachman•200
    @PeterBachman100

    Looks good! I especially like that the desktop layout is responsive to narrower widths.

  • Peter Bachman•200
    @PeterBachman100

    Looks like you submitted the wrong project! You should be able to update the project information to fix it, or delete it if that doesn't work.

  • P
    yinnie•320
    @wcyin9
    What are you most proud of, and what would you do differently next time?

    It's my first time writing JS for a project, so I'm proud of successfully executing it after studying JS for a bit. I think there's more ways to make my code concise, so next time I'll definitely apply that.

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

    The most difficulty I had was with the JS itself. Since I'm fairly new to JS, I had little knowledge of what code to use, and how to successfully link it to the html. After a bit of struggle, I finally successfully executed what I wanted it to do by using classList

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

    As it's my first time writing JS, I'm unaware of the best practices, or how to condense my code in a better way. Please give me any heads up, pointers, or tips, and let me know if there's anything I could have done better.

    In addition, in the figma design the image is cropped differently in desktop view, and I am unsure as to how to crop the image exactly as it's portrayed in the design.

    Article Preview Component

    1
    Peter Bachman•200
    @PeterBachman100

    Nice work!

  • Peter Bachman•200
    @PeterBachman100

    Nice work!

  • P
    mrcordova•1,360
    @mrcordova
    What are you most proud of, and what would you do differently next time?

    I am proud of completing the project, and I wouldn't not change anything.

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

    I was having trouble placing elements with grid because I was used to flexbox and working in 2d was a different experience but I figured it out. I also hadn't used background image so I was unaware how to use attachment. I looked it up and figured out I was using fixed instead of local.

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

    I would welcomed any feedback especially how I used css grid. I was wondering how you would center your grid items to the center of the screen responsively.

    Testimonials Grid Section using grid and flexbox

    1
    Peter Bachman•200
    @PeterBachman100

    There is a super easy way to center the whole grid in the center of the screen. Just set the parent container of the grid to display: grid; place-items: center;

    Hope that helps!

    Marked as helpful
  • Peter Bachman•200
    @PeterBachman100

    I would consider using different display settings for different parts of the page, as some parts of this content are well suited for grid (the cards), and the top section doesn't really need it.

View more comments