Advice generator - CSS Grid, Vanilla JS

Solution retrospective
Hi,
Another great challenge, I enjoyed working on it to practice API integration and CSS grid. I decided to add a theme toggler to the design and also a typewriting effect to the text, I hope you like it. Feedback welcome!
Cheers
Please log in to post a comment
Log in with GitHubCommunity feedback
- @tanereren
Hey!
An improvement that you can make is your
.theme-toggler
button has uneven padding which makes clicking on it quite awkward.theme-toggler { border: none; background-color: inherit; color: var(--primary-300); display: flex; align-items: center; gap: 0.5em; font-weight: 700; padding: 2em 1.5em 0 0; <-- change this to 2em only cursor: pointer;
Also when you click the dice button the words aren't being rendered correctly - they're illegible so you may want to check the function - the typewriter effect is good idea though!
Let me know how you get on
Marked as helpful
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord