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

Hairline gap in Portfolio widget (when Item Gap set to "0") #5077

Closed
5 tasks done
drawcard opened this issue Jul 16, 2018 · 2 comments
Closed
5 tasks done

Hairline gap in Portfolio widget (when Item Gap set to "0") #5077

drawcard opened this issue Jul 16, 2018 · 2 comments
Labels
status/merged Indicates when a Pull Request has been merged to a Release.

Comments

@drawcard
Copy link
Contributor

Prerequisites

  • I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
  • The issue still exists against the latest stable version of Elementor.

Description

For the Portfolio widget, setting Item Gap to "0" creates a hairline gap between repeater items.

image

image

(Magnified)

image

Steps to reproduce

  • Create a portfolio widget
  • Set a query that contains items with Featured Image thumbnails
  • Edit Portfolio > Style > Items > Set Item Gap to 0

Isolating the problem

  • This bug happens with only Elementor plugin active (and Elementor Pro).
  • This bug happens with a default WordPress theme active.
  • I can reproduce this bug consistently using the steps above.

Environment

http://whatsmybrowser.org/b/SQWT6JC

System Info ```

== Server Environment ==
Operating System: Linux
Software: LiteSpeed
MySQL version: 10.0.35
PHP Version: 7.1.19
PHP Max Input Vars: 1000
PHP Max Post Size: 32M
GD Installed: Yes
ZIP Installed: Yes
Write Permissions: All right
Elementor Library: Connected

== WordPress Environment ==
Version: 4.9.7
Site URL: https://drawcard.com.au
Home URL: https://drawcard.com.au
WP Multisite: No
Max Upload Size: 32 MB
Memory limit: 40M
Permalink Structure: /%category%/%postname%
Language: en-AU
Timezone: 0
Debug Mode: Active

== Theme ==
Name: Drawcard 2017
Version: 9.0.0-beta.2
Author: Roots
Child Theme: Yes

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

== Active Plugins ==
AdWords Conversion Tracking Code
Version: 1.0
Author: kcseopro

All-in-One WP Migration
	Version: 6.71
	Author: ServMask

dflip
	Version: 1.2.8
	Author: Deepak Ghimire

Duplicate Post
	Version: 3.2.2
	Author: Enrico Battocchi

Elementor
	Version: 2.1.2
	Author: Elementor.com

Elementor Pro
	Version: 2.1.0-beta2
	Author: Elementor.com

EWWW Image Optimizer
	Version: 4.2.3
	Author: Shane Bishop

Google Analytics Dashboard for WP (GADWP)
	Version: 5.3.5
	Author: ExactMetrics

Insert PHP Code Snippet
	Version: 1.2.5
	Author: xyzscripts.com

OnePress Image Elevator
	Version: 2.6.2
	Author: OnePress

Pods - Custom Content Types and Fields
	Version: 2.7.6
	Author: Pods Framework Team

Post Type Switcher
	Version: 3.1.0
	Author: John James Jacoby

Really Simple SSL
	Version: 3.0.2
	Author: Rogier Lankhorst

Redirection
	Version: 3.3.1
	Author: John Godley

Regenerate Thumbnails
	Version: 3.0.2
	Author: Alex Mills (Viper007Bond)

Review Wave - Google Places Reviews
	Version: 1.4.7
	Author: MessageMetric

Sunshine Photo Cart
	Version: 2.7.2
	Author: Sunshine Photo Cart

SVG Support
	Version: 2.3.15
	Author: Benbodhi

Theme Check
	Version: 20160523.1
	Author: Otto42, pross

TinyMCE Advanced
	Version: 4.7.11
	Author: Andrew Ozz

Wordfence Security
	Version: 7.1.9
	Author: Wordfence

WP-Optimize
	Version: 2.2.4
	Author: David Anderson, Ruhani Rabin, Team Updraft

Yoast SEO
	Version: 7.8
	Author: Team Yoast

== Must-Use Plugins ==
WP Sync DB Compatibility
Version: 1.0
Author: Sean Lang

== Debug ==
Errors:
TypeError: Cannot read property 'checked' of undefined
at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=2.1.2 - 2:43100
2018-07-16 01:08 +00:00

	TypeError: o.attributes[t.name].each is not a function
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=2.1.0-beta2 - 2:105550
		2018-07-02 07:04 +00:00

	TypeError: Cannot read property 'behavior' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=2.1.0-beta2 - 2:223542
		2018-07-02 04:48 +00:00

	TypeError: Cannot read property 'checked' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=2.0.12 - 2:43232
		2018-05-16 05:04 +00:00

	ViewDestroyedError: View (cid: "view1403") has already been destroyed and cannot be used.
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/lib/backbone/backbone.marionette.min.js?ver=2.4.5 - 22:19954
		2018-04-09 01:53 +00:00

	TypeError: Cannot read property 'top' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/lib/select2/js/select2.min.js?ver=4.0.5 - 1:10581
		2018-04-09 01:37 +00:00
		x 2 times

	TypeError: Cannot read property 'removeClass' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=1.7.11 - 2:104207
		2017-10-06 01:09 +00:00
		x 2 times

	TypeError: Cannot read property 'removeClass' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=1.7.5 - 2:104205
		2017-09-26 05:36 +00:00
		x 9 times

	TypeError: Cannot read property 'removeClass' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=1.6.4 - 2:111050
		2017-08-14 04:17 +00:00
		x 5 times

	TypeError: Cannot read property 'removeClass' of undefined
		at https://drawcard.com.au/wp-content/plugins/elementor/assets/js/editor.min.js?ver=1.6.0-RC3 - 2:110978
		2017-07-31 05:43 +00:00
		x 5 times
</details>
@bainternet bainternet added the status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. label Jul 16, 2018
@bainternet bainternet added this to the 2.1.0 Pro milestone Jul 16, 2018
@bainternet
Copy link
Contributor

@drawcard

thanks for reporting, we can replicate and it should be fixed in next version.

@arielk arielk closed this as completed Jul 16, 2018
@arielk arielk removed this from the 2.1.0 Pro milestone Jul 16, 2018
@arielk arielk added status/merged Indicates when a Pull Request has been merged to a Release. and removed status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. labels Jul 16, 2018
@drawcard
Copy link
Contributor Author

Nice one, cheers :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/merged Indicates when a Pull Request has been merged to a Release.
Projects
None yet
Development

No branches or pull requests

3 participants