Skip to content

Fixed bug in _findBackgrounds() which caused to ignore elements with dat... - #25

Merged
markdalgleish merged 1 commit into
markdalgleish:masterfrom
kroleg:fix-process-els-w-data-bg-attrs
Feb 5, 2013
Merged

Fixed bug in _findBackgrounds() which caused to ignore elements with dat...#25
markdalgleish merged 1 commit into
markdalgleish:masterfrom
kroleg:fix-process-els-w-data-bg-attrs

Conversation

@kroleg

@kroleg kroleg commented Feb 5, 2013

Copy link
Copy Markdown
Contributor

...a-stellar-background if you pass them directly to stellar() function. Also replaced check for data property via "is([data-prop])" with "data('prop')".

  1. $stuff.add() doesn't change $stuff, so $backgroundElements remains unchanged
    http://api.jquery.com/add/

  2. I think check with "data('something')" is more appropriate than "is([data-something])"

Thank you for such a nice library! 👍

…data-stellar-background if you pass them directly to stellar() function. Also replaced check for data property via "is([data-prop])" with "data('prop')".
markdalgleish added a commit that referenced this pull request Feb 5, 2013
Fixed bug in _findBackgrounds() which caused to ignore elements with dat...
@markdalgleish
markdalgleish merged commit 680e2b1 into markdalgleish:master Feb 5, 2013
@markdalgleish

Copy link
Copy Markdown
Owner

Nice pickup, thanks for the pull request :)

@kroleg
kroleg deleted the fix-process-els-w-data-bg-attrs branch February 6, 2013 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants