Add selenium tests for Slider and RangeSlider

Any chance to have that fixed or is there a workaround?

from bokeh.models import TextInput, RangeSlider
from bokeh.layouts import widgetbox
from bokeh.io import curdoc
from bokeh.client import push_session

def on_slider_change(attr, old, new):
    text.value = str(new[1])

def on_text_change(attr, old, new):
    try:
        n = int(new)
        n = max(n, -10)
        n = min(n, 10)

        slider.range = (0, n)

    except ValueError:
        return

text = TextInput(
        value=str(2),
        title='Population Beta:')
text.on_change('value', on_text_change)

slider = RangeSlider(
        start=0,
        end=10,
        range=(0, 2),
        step=1,
        title='Population Beta')

slider.on_change('range', on_slider_change)

curdoc().add_root(widgetbox(text, slider))

#session = push_session(curdoc())
#session.show(widgetbox(text, slider)) # open the document in a browser
#session.loop_until_closed() # run forever

[bryevdv]

@zoohair this code seems to work as expected for me on master (or 0.12.7dev10)

from bokeh.models import TextInput, RangeSlider
from bokeh.layouts import widgetbox
from bokeh.io import curdoc
from bokeh.client import push_session

def on_slider_change(attr, old, new):
    text.value = str(new[1])

def on_text_change(attr, old, new):
    try:
        n = int(new)
        n = max(n, -10)
        n = min(n, 10)

        slider.value = (slider.value[0], n)

    except ValueError:
        return

text = TextInput(
        value=str(2),
        title='Population Beta:')
text.on_change('value', on_text_change)

slider = RangeSlider(
        start=0,
        end=10,
        value=(0, 2),
        step=1,
        title='Population Beta')

slider.on_change('value', on_slider_change)

curdoc().add_root(widgetbox(text, slider))

@mattpap it looks like the new slider has value instead of range? I’m OK with that but we need a migration note

I also still think it is a missing feature to not easily be able to update one or other of the values without updating both.

[bryevdv]

OK the original issue was resolved (I just checked again with 1.1.1dev1) but we should add an integration test now that is is easier to do