Browse Source

#2758 #2760 fix this context for transition callbacks

pull/2755/merge
jlukic 9 years ago
parent
commit
0d23f97335
2 changed files with 8 additions and 6 deletions
  1. 4
      RELEASE-NOTES.md
  2. 10
      src/definitions/modules/transition.js

4
RELEASE-NOTES.md

@ -9,7 +9,6 @@
- **Label** - Added `basic` label variation, useful for item counts - **Label** - Added `basic` label variation, useful for item counts
- **Button** - Added `labeled button` variation for display a count next to a button. - **Button** - Added `labeled button` variation for display a count next to a button.
- **Divider** - Vertical divider can now be used multiple times in a single column row (not just 50/50 split). #2808 - **Divider** - Vertical divider can now be used multiple times in a single column row (not just 50/50 split). #2808
- **Icon** - Fixed typo in cube icon alias caused by bad grep #2765
**Additional Enhancements** **Additional Enhancements**
- **Menu** - `text menu` now uses padding for hitboxes to make target area for links larger - **Menu** - `text menu` now uses padding for hitboxes to make target area for links larger
@ -22,6 +21,9 @@
- **Modal** - Fix autofocus setting in modal not working due to improper selector #2737 - **Modal** - Fix autofocus setting in modal not working due to improper selector #2737
- **Modal** - Increased `close` specificity, modal will now only close on `> .close` #2736 - **Modal** - Increased `close` specificity, modal will now only close on `> .close` #2736
- **Accordion** - Added missing notation for accordion docs #2812 - **Accordion** - Added missing notation for accordion docs #2812
- **Icon** - Fixed typo in cube icon alias caused by bad grep #2765
- **Embed** - Remove accidental `console.log` statements in js #2760
- **Transition** - Transition callbacks now all have the correct `this` set. #2758
**Additional Bugs** **Additional Bugs**
- **Build Tools** - Fixes issue on `win` platform where packaged theme would not correctly update when using watch due to regExp not matching windows path separators. - **Build Tools** - Fixes issue on `win` platform where packaged theme would not correctly update when using watch due to regExp not matching windows path separators.

10
src/definitions/modules/transition.js

@ -392,7 +392,7 @@ $.fn.transition = function() {
module.add.failSafe(); module.add.failSafe();
} }
module.set.duration(settings.duration); module.set.duration(settings.duration);
settings.onStart.call(this);
settings.onStart.call(element);
} }
}, },
@ -768,8 +768,8 @@ $.fn.transition = function() {
module.remove.visible(); module.remove.visible();
module.set.hidden(); module.set.hidden();
module.force.hidden(); module.force.hidden();
settings.onHide.call(this);
settings.onComplete.call(this);
settings.onHide.call(element);
settings.onComplete.call(element);
// module.repaint(); // module.repaint();
}, },
@ -778,8 +778,8 @@ $.fn.transition = function() {
module.remove.hidden(); module.remove.hidden();
module.set.visible(); module.set.visible();
module.force.visible(); module.force.visible();
settings.onShow.call(this);
settings.onComplete.call(this);
settings.onShow.call(element);
settings.onComplete.call(element);
// module.repaint(); // module.repaint();
}, },

Loading…
Cancel
Save