Select row(s) issues

Describe the bug Select rows is not working like the docs say: http://tabulator.info/docs/4.2/select

Tabulator Info Tabulator 4.2.0

var inv_tags_tbl = new Tabulator("#inv_tags_table", {
    clipboard:true,
    selectable:true,
    selectableRangeMode:"click",
    height: "750px",
    pagination: "local",
    paginationSize: 20,
    layout: "fitDataFill",
    addRowPos: "top",
    columns:[
        {title: "Common", field: "common", dir: "asc", frozen: true, headerFilter: true},
        {title: "Plant#", field: "p_item_no", frozen: true, headerFilter: true, headerFilterFunc: "="},
        {title: "Item Key", field: "item_key", headerFilter: true,
            headerFilterFunc: startWithFilter},
        {title: "Tag#", field: "tag_number", headerFilter: true, headerFilterFunc: "="},
        {title: "UPC", field: "upc_prod", headerFilter: true, headerFilterFunc: startWithFilter},
        {title: "On Hand", field: "tag_oh"},
        {title: "Loc 1", field: "loc_1"},
        {title: "Loc 2", field: "loc_2"}

    ]
});

To Reproduce Steps to reproduce the behavior:

1. With selectableRangeMode commented out Shift click/drag a range of cells.
2. Repeat with selectableRangeMode uncommented.

Expected behavior

1. That a range of rows is selected. Instead only the first and low row in range are selected.
2. A range of rows is selected. The issue is that you can only untoggle the row selection down to the number of rows -1. In other words there is always one row that cannot be toggled off.

Screenshots 1)

Desktop (please complete the following information): openSUSE 42.3 Firefox 60.5.0esr (64-bit)

Additional context Add any other context about the problem here.

[aklaver]

Done: https://bugzilla.mozilla.org/show_bug.cgi?id=1529753

[olifolkerd]

@aklaver no problem, it wasnt you specifically, just getting bombarded by a lot of people all treating their problems as the end of the world and not understanding that this is a very big library and i have to prioritise things.

There are a lot of issues with mozilla and mouse events. it is likely the same reason as the movable rows issue. Their events don’t fire in the same sequences with the same targets as the other browsers. Its why the drag rows between tables system is still bugged on firefox.

Good idea on the support ticket with them, i will add some detail to it at some point from my investigations.

Not to worry though, i implemented a new system to handle dragging for touch events (they work very differently to mouse events) so im gonna see if i can re-purpose that to handle firefoxes issues in the next release

Cheers

Oli 😃