Welcome to the Treehouse Community
Looking to learn something new?
Steven Parker220,937 Points
I'm not sure what you mean by "everything seems fine" if it doesn't work.
But here's a few issues I noticed:
- getElementsByClassName returns a collection, but you treat the result as if it were an element
- you could subscript the result if you want an element, or use querySelector instead
- you forgot to enclose your class name in quotes when passing it as an argument
- your modal reveal function needs to change the display style to make it visible