3 ms·
Looks good. But I have a design suggestion for you. Based on general heuristics, lightbox image button on top-right corner (http://imgur.com/a/LoQ6D http://img
by dm8 15y ago
Looks good. But I have a design suggestion for you.
Based on general heuristics, lightbox image button on top-right corner (http://imgur.com/a/LoQ6D http://imgur.com/a/LoQ6D) should be to close lightbox image rather than expand it. Its confusing and there is no other way to close it apart from clicking in the background page.
- sgdesign 15y agoAgreed, but I used an existing lightbox script (http://www.no-margin-for-errors.com/projects/prettyphoto-jquery-lightbox-clone/ http://www.no-margin-for-errors.com/projects/prettyphoto-jqu...) and didn't have the courage to tweak it too much.
- minikomi 15y agohttps://gist.github.com/d5882b9ce4c52cb56cf1 https://gist.github.com/d5882b9ce4c52cb56cf1 I changed the click event bound to the expand button to the close function for you.. Chrome developer tools does a pretty good job at unminifying js