Skip to content
OverviewSolutions
29
Comments
4
Sinka László
@dlxzeus777

All comments

  • Sinka László•470
    @dlxzeus777Posted about 2 years ago

    Hey Treasure, Nice solution however I have some tips for you!

    Instead of grid-template-columns: repeat(5,1fr) in your social-media-1 class change the number 5 to 4 because its a 4 column grid. The parent div with a social-media class should have display:grid; justify-items: center; so it gets centered.

    In your div with a class of social-media-dashboard-2 you should delete every children's grid-column and add this display: grid; grid-template-columns: repeat(4,1fr); grid-template-rows: repeat(2,1fr);, so if you want to make it responsive all you have to do is change the grid-template-columns or the grid-template-rows.

    0
  • C Lewis•110
    @casserole27Submitted about 2 years ago

    MY JAVASCRIPT ON THIS IS SO VERY CLUNKY! I am absolutely certain there is an easier way than what I did. Also, I know I have work to do on min and max-widths in my CSS. I'm submitting the project with documentation because it works, and I'm going to revisit my JS for some better solutions.

    Interactive Rating Component using vanilla JavaScript

    4
    Sinka László•470
    @dlxzeus777Posted about 2 years ago

    Hey C Lewis nice looking solution, however you could've just used a for of loop to loop through the buttons and get its value and apply it to the innertext.

    Like this for example:

    for(let btn of buttons) { btn.addEventListener('click', (e) => { const target = e.target.value; console.log(target); selectedNumber.innerText = target; }) }

    1
  • Sinka László•470
    @dlxzeus777Posted about 2 years ago

    Hello!

    Your solution looks awesome!

    I think you should give the body the height:100vh; and the background: rgba(10, 12, 28, 1); instead of the container class because on smaller screens the content just gets cut off.

    Other than that it looks good, nice work!

    Marked as helpful
    0
  • Sinka László•470
    @dlxzeus777Posted about 2 years ago

    Any idea on how can I solve the hover state on the image?

    0