BUG: input field inside slide doesnt work on android

[Overpowered]

Hi, thx for your great script! I chose your one, because its very lightweight (not full of unnecessary stuff) and good to handle ... BUT ;) there is an annoying bug.

Input fields (inside a slide) will not work on android (I use android 8). If I click inside the input field, the keyboard will slide in and in the same step also slide out. So the keyboard will not stay (input isnt possible).

Here is my example fiddle:
https://jsfiddle.net/cjobq2o5/

Hope you can fix it?

[jisi724]

Have the same issue here.

The reason is actually the resizeHandler method, which caused lots of problems I would say. This method would be triggered in many situations that window size doesn't change, like scrolling or clicking input fields.

Here is a quick fix you can try @Overpowered : extends a new class from Siema, and overwrite the resizeHandler method then you will see the keyboard stay there as usual.

[craigpryde]

Incase anyone hits this issue. I have addressed this in my code by extending from siema and overriding the resizeHandler();

I have added the following to my code:

    class Carousel extends Siema {
        // Store width to perform check
        _currrentViewWidth = window.innerWidth;

        resizeHandler() {
            // This is used to prevent the resize handler firing when the 
            // Android keyboard pops up.
            if(Math.abs(window.innerWidth - this._currrentViewWidth) === 0) {
                return; 
            }
        
            this._currrentViewWidth = window.innerWidth;
            ... rest of the resize handler code.
        }
    }

What i am doing is storing the current window innerwidth with:

    _currrentViewWidth = window.innerWidth;

And then using that value as a reference in the original resize handler to check if the window width has changed. i.e > 0px difference. And if the viewport size does change then update the stored reference.

    if(Math.abs(window.innerWidth - this._currrentViewWidth) === 0) {
        return; 
    }
        
    this._currrentViewWidth = window.innerWidth;

Hopefully this helps.

[andsat]

Thanks for the fix! This is also a bug on Firefox mobile. Strangely, iOS Safari works fine.