Update range display

[Indigo744]

Implements #110 second option!

Usage:

opt = {
    hiddenOpacity: 0.3,                 // Opacity to use when hidding element
    animDuration: 200,                  // Animation duration for showing/hiding element
    afterShowRange: function() {},      // User callback
    ranges: {                           // The ranges
        plot: {
            0: {                        // valueIndex
                min: 1000,
                max: 1200
            },
            1: {                        // valueIndex
                min: 10,
                max: 12
            }
        }, 
        area : {
            {min: 10, max: 20}    // No valueIndex, only an object, use 0 as valueIndex (easy case)
        }
    }
};
$(".mapcontainer").trigger("showElementsInRange", [opt]);

Provided are 2 advanced examples using NoUISlider (http://refreshless.com/nouislider).

As a nice effect, I factorized some function using the new setElementOpacity() function.

[Indigo744]

This is weird. UT were passing in Firefox...

[Indigo744]

At last :)

Travis/PhantomJS doesn't seem to like $(".selector").is(":visible")
I changed it for $(".selector").css('display') !== 'none'

[neveldo]

Thank you, this is a great new feature ! Why did you force the installation of phantomJS 2.0.0 for unit testing ?

[neveldo]

Does the argument ranges shouldn't be called options because we can set options like hiddenOpacity or animDuration in this object ?

[Indigo744]

Right! It was from an earlier version where I only passed a ranged array. I forgot to update this part after extending the event.

[Indigo744]

I commited the new names.
Regarding phantomJS 2.0, I though this was what cause problems in the UT. It turned out it is not the case (see explanation above). I should then remove it.

[neveldo]

As the function is not expected to return any value, maybe we should just do return; here ?

[Indigo744]

Actually, no. This return true; is related to the $.each() function callback.
Returning true will go to the next item (like a continue statement). Whereas false will end the loop (much like a break statement). See jQuery doc: http://api.jquery.com/jQuery.each/

[neveldo]

You are right, thank you !

[neveldo]

Seems to be redundent with L761 elem.mapElem.show();

[Indigo744]

Fixed :)

[neveldo]

Maybe the 'animDuration' should be added to the arguments of fnShowElement() ?

[Indigo744]

Why? It is available in the parent onUpdateEvent() function as a closure.

[neveldo]

My bad, you are right !