Closed DCNick3 closed 5 years ago
Looks good, minor nitpicks:
Please enable JavaScript execution in your browser
show_image
, function show_image(index) {
for (var i = 0; i < images.length; ++i)
images[i].className = (i === index) ? "container" : "hidden container"; // or its equivalent if-style
}
Looks sane
Sane enough, merging.
Relative urls are more flexible, so absolute ones were replaced with them.
Greatly improved browser support by replacing lamdas with classic function definition.
Replaced css modification to hiding or showing image with css classes, which has less overhead.
Removed unnecessary css libraries, which created 2 additional requests to external sites.