Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Image Carousel - Slides to Show: responsive desktop setting overrides tablet setting #2341

Closed
humangeometry opened this issue Sep 6, 2017 · 1 comment
Labels
bug Indicates a bug with one or multiple components.

Comments

@humangeometry
Copy link

Description

Image Carousel's "Slides to Show" option is overriding the tablet setting with the number selected for desktop-sized screens.

Steps to reproduce

For the Slides to Show option in the Image Carousel element, set desktop screens to display 4 slides and tablet screens to display 3 slides.

Change the responsive size of the screen to Tablet. Notice that 4 slides are still shown, despite being set to display 3 slides.

See demonstration: http://d.pr/i/MuUd7M/2zJjRjXy

Environment

== Server Environment ==
Operating System: Linux
Software: nginx/1.11.9
MySQL version: 5.5.57
PHP Version: 5.5.9-1ubuntu4.22
PHP Max Input Vars: 1000
PHP Max Post Size: 12M
GD Installed: Yes
Elementor Library: Connected

== WordPress Environment ==
Version: 4.8.1
Site URL:
Home URL:
WP Multisite: No
Max Upload Size: 8 MB
Memory limit: 40M
Permalink Structure: /%postname%/
Language: en-US
Timezone: America/Chicago
Debug Mode: Inactive

== Theme ==
Name: GeneratePress
Version: 1.4
Author: Tom Usborne
Child Theme: No

== User ==
Role: administrator
WP Profile lang: en_US
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.113 Safari/537.36

== Active Plugins ==
Code Snippets
Version: 2.8.6
Author: Shea Bunge

Elementor
	Version: 1.7.1
	Author: Elementor.com

Elementor Extras
	Version: 1.2.1
	Author: Namogo

Elementor Pro
	Version: 1.6.1
	Author: Elementor.com

GP Premium
	Version: 1.4.3
	Author: Tom Usborne

MailOptin - Lite
	Version: 1.1.2.0
	Author: MailOptin Team

Simple CSS
	Version: 0.4
	Author: Tom Usborne

Two Factor
	Version: 0.1-dev-20170603
	Author: George Stephanis

W3 Total Cache
	Version: 0.9.5.4
	Author: Frederick Townes

WordPress Importer
	Version: 0.6.3
	Author: wordpressdotorg

Yoast SEO Premium
	Version: 5.3.3
	Author: Team Yoast

== Must-Use Plugins ==
mu-includes.php
Version:
Author:

== Debug ==
Errors:
TypeError: Cannot read property 'get' of undefined
at /wp-content/plugins/elementor/assets/js/editor.min.js?ver=1.7.0 - 2:164830
2017-08-29 16:20 +00:00

@humangeometry humangeometry changed the title [Bug] Image Carousel 'Slides to Show' Desktop overrides Tablet responsive setting [Bug] Image Carousel 'Slides to Show' responsive desktop setting overrides tablet setting Sep 6, 2017
@humangeometry humangeometry changed the title [Bug] Image Carousel 'Slides to Show' responsive desktop setting overrides tablet setting [Bug] Image Carousel --> Slides to Show responsive desktop setting overrides tablet setting Sep 6, 2017
@humangeometry humangeometry changed the title [Bug] Image Carousel --> Slides to Show responsive desktop setting overrides tablet setting [Bug] Image Carousel: Slides to Show - responsive desktop setting overrides tablet setting Sep 6, 2017
@humangeometry humangeometry changed the title [Bug] Image Carousel: Slides to Show - responsive desktop setting overrides tablet setting [Bug] Image Carousel - Slides to Show: responsive desktop setting overrides tablet setting Sep 6, 2017
@arielk arielk added the bug Indicates a bug with one or multiple components. label Sep 8, 2017
kobizz added a commit to kobizz/elementor that referenced this issue Sep 10, 2017
@arielk
Copy link
Member

arielk commented Sep 10, 2017

Fix for v1.7.3

Thanks for your feedback!

@arielk arielk closed this as completed Sep 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates a bug with one or multiple components.
Projects
None yet
Development

No branches or pull requests

2 participants