glide icon indicating copy to clipboard operation
glide copied to clipboard

3.5.1 Appears to have broken the slider

Open richayles opened this issue 4 years ago • 10 comments

3.5.1 Appears to have broken the slider on one of our sites, specifically on mobile. Could you document what changes have been made - images below show the difference that is happening with the images on the site specifically when viewing on mobile

3.5.1 image

3.4.1 image

richayles avatar Nov 28 '21 11:11 richayles

What error do you see in the console? Can you share your initiation script and website or reproduced sandbox?

jedrzejchalubek avatar Nov 28 '21 14:11 jedrzejchalubek

Not seeing any console errors, but mobile definitely is not working.

const config = `{
                    "type": "slider",
                    "hoverpause": false,
                    "perView": 3,
                    "gap": 0,
                    "rewind": false,
                    "bound": true,
                    "breakpoints": {
                      "1100": {
                        "perView": "3",
                        "peek": {
                          "before": 0,
                          "after": -74
                        }
                      },
                      "500": {
                        "perView": "2",
                        "peek": {
                          "before": 0,
                          "after": -74
                        }
                      }
                    }}`;
new Glide(glide, JSON.parse(config))

half way down the page on the homepage https://pipersfarm.com/?preview_theme_id=120868864074

an additional class was added on that page to make it appear better, before reverting back

.glide img {
    width: auto !important;
}

Reverted back to the version 3.4.1 on the live site now - https://pipersfarm.com/

richayles avatar Nov 28 '21 21:11 richayles

Links are down. Don't have much information to help you 😢

jedrzejchalubek avatar Nov 30 '21 10:11 jedrzejchalubek

Might be the issue mentioned in v3.5.2 release (about the dist directory not being used on NPM after its installation)? https://github.com/glidejs/glide/releases/tag/v3.5.2

iammati avatar Dec 01 '21 13:12 iammati

Have just had another issue on a separate live site - which has been upgraded to 3.5.2 - seeing the below console errors - not sure if that helps image image

richayles avatar Dec 03 '21 12:12 richayles

Hi, I have a similar issue. You can see the slider at https://strategus.vercel.app/ - on desktop it works; mobile is not working.

Source file of component (SvelteKit): https://www.dropbox.com/s/e7kvtnwawthxxjz/Slider.svelte?dl=0

seppegadeyne avatar Dec 03 '21 13:12 seppegadeyne

not sure this is related to 3.5.1 looks to be an empty

    element for me without slides 🤦

richayles avatar Dec 03 '21 13:12 richayles

Hi, I have a similar issue. You can see the slider at https://strategus.vercel.app/ - on desktop it works; mobile is not working.

Source file of component (SvelteKit): https://www.dropbox.com/s/e7kvtnwawthxxjz/Slider.svelte?dl=0

I found a solution, give the parent node, the one wrapping <div class="glide"></div> a max-width style ... for example, <div style="max-width: 100%;"><div class="glide"> ... </div></div> ...

seppegadeyne avatar Dec 03 '21 14:12 seppegadeyne

When I performed this upgrade: image

My slider also broke: image

No JS errors, but it appears the "thing that calculates the widths" is getting the wrong answer; too much magic for me to understand how to help troubleshoot; I downgraded for now.. curious if anyone has any troubleshooting ideas..

coladarci avatar Mar 09 '22 15:03 coladarci

Same problem here after upgrading from 3.4.1 to 3.5.2 after which slide control items suddenly reduced its width and glide__slides <ul> element has added incorrect width.

Downgraded back to the version 3.4.1.

ivarzLV avatar Apr 04 '22 15:04 ivarzLV