@FoxMalder-coder
Posted
Hi! To make grid row the same height use grid-template-rows: 1fr 1fr or repeat(2, 1fr). Also I'v suggest to use some intermediate grid between small screen and desktop, and center content on large screen. About js - first cut styling from js to css (h6.daily, h6.weekly, h6.monthly have same styling for passive state and active state - use one class for passive, for example, selector, and another one for active, for exemple, selectorActive; and then just toggle class in js). Another thing is that your doing the same in all three function - try to pass value (daily, monthly, weekly) inside just one function. p.s. strange to see camelCase in html...
Marked as helpful
@TheShonuff
Posted
@FoxMalder-coder Thank you for taking the time to look at my code and write a response. I'll attempt to implement your suggestions in a fork soon. As for the camel casing in HTML... yeah it's an ugly habit I need to break. I camelcase everything.