Skip to content
OverviewSolutions
10
Comments
9

Alperen Aksoy

@aksoyalpi150 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
    Emmanuel Lopez•460
    @EmLopezDev
    What are you most proud of, and what would you do differently next time?
    • I am most proud of pushing through it, there were times where I couldn't figure out how to solve a part of the solution but rather than give up I tried and tried until I figure it out.
    • Getting a better understanding of some of the core concepts of JavaScript like event delegation , the .reduce() method and Promises.
    • Taking in the extra time to create a loading state
    What challenges did you encounter, and how did you overcome them?
    • Buttons not working after fetching data. I forgot that fetching data re-renders the HTML causing any event listeners to be removed, therefore I had to use event delegation to get it to work. I figured it out by using stack overflow and reading up on the fetch() method.
    What specific areas of your project would you like help with?

    I would like some feedback on the main.js file was there a better or more efficient way of writing the js.

    Time Tracking Dashboard

    #sass/scss
    1
    Alperen Aksoy•150
    @aksoyalpi

    Your HTML and CSS look very good (your solution is also very accurate), but your js is not that performant, because you add and remove a lot of elements. Try to use more css to change the data (for example with data attributes, that helped me a lot; see this: https://developer.mozilla.org/en-US/docs/Web/HTML/How_to/Use_data_attributes).

    My Solution is also not perfect, but you can check it out, if you want to see how to use the data attributes and minimal js :)

    Marked as helpful
  • Alperen Aksoy•150
    @aksoyalpi

    Your solution looks very good, but try to not use DOM manipulation too much. Instead of adding and removing DOM elements try to just set a css class (with element.classList.toggle(...)) that for example has display: hidden, so that it toggles between hidden and not hidden. This is a lot more efficient.

  • Alperen Aksoy•150
    @aksoyalpi

    Looks very good, thumbs up

  • Alperen Aksoy•150
    @aksoyalpi

    Looks good, but you could use mixin for better scss usage

  • Hiren Tumbadiya•110
    @HirenTumbadiya
    What are you most proud of, and what would you do differently next time?

    Proud to learn flexbox will keep learning

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

    merging rows and seperated with div

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

    will work on web sockets and webrtc

    Responsive Card with FlexBox

    1
    Alperen Aksoy•150
    @aksoyalpi

    Looks very good, only thing that i would recommend is to set the body to 100vh height so that the background color is all over the website and the footer is on the bottom. I also like the hover effect.

  • Alperen Aksoy•150
    @aksoyalpi

    Wow, I c no difference. You're code is also well structured. And I like the modular architecture of your scss.

View more comments