Treasure• 310
@oluwatobicode
Submitted
Sinka László
@dlxzeus777
All comments
- Sinka László• 470
@dlxzeus777
Posted
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 havedisplay: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-columnand add thisdisplay: 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 thegrid-template-columnsor thegrid-template-rows.0 - C Lewis• 110
@casserole27
Submitted
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.
Sinka László• 470@dlxzeus777
Posted
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 - Have-Samuel• 350
@Have-Samuel
Submitted
Hello Guys,
Ideas, suggestions, and compliments are welcome 🤝.
Sinka László• 470@dlxzeus777
Posted
Hello!
Your solution looks awesome!
I think you should give the body the
height:100vh;and thebackground: 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
@dlxzeus777
Submitted
Sinka László• 470@dlxzeus777
Posted
Any idea on how can I solve the hover state on the image?
0
