Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Add data to thumbnail element, return data on thumbnail_getSelectedItems #1685

Merged
merged 2 commits into from
Jan 22, 2016
Merged

Conversation

nmerchant
Copy link
Contributor

Fixes #1686

@@ -36,7 +36,10 @@
$.fn.repeater.Constructor.prototype.thumbnail_getSelectedItems = function () {
var selected = [];
this.$canvas.find('.repeater-thumbnail-cont .selectable.selected').each(function () {
selected.push($(this));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leave this line as-is, since data can be appended to the thumbnail item and accessed in that way

@kevinparkerson
Copy link
Contributor

looks good!

kevinparkerson pushed a commit that referenced this pull request Jan 22, 2016
Add data to thumbnail element, return data on thumbnail_getSelectedItems
@kevinparkerson kevinparkerson merged commit e008203 into ExactTarget:master Jan 22, 2016
@swilliamset
Copy link
Contributor

removing "enhancement". bug fix to achieve parity with other _getSelectedItems functions in repeater

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants