#light {display: none;}
#fade {display: none;}
id имеет приоритет, над class.
Да и посмотри как у других сделаны попапы. Чтоб закрывался не только при нажатии на Х а просто вне попапа.
p.s.
visibility: hidden - не рекомендуется, если позиция не абсолютная и если в попапе есть видео. При исчезновении, видео продолжит играть. А display: none; его отрубит.
