Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Fixed #3143: fixed functionality of 'show-weeks' attribute on datepicker popup #3149

Closed
ankit5990 wants to merge 7 commits into from
Closed

Fixed #3143: fixed functionality of 'show-weeks' attribute on datepicker popup #3149

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
0f8c8b4
fix(datepicker): memory leak fix for datepicker.
Sep 10, 2014
0659a5e
Merge remote-tracking branch 'upstream/master'
ankit5990 Jan 2, 2015
9f03d40
fix(datepicker): fixed working of 'show-weeks' on datepickerPopup
ankit5990 Jan 2, 2015
c645db3
fix(datepicker): fixed 'show-weeks' datepickerPopup. resolved conflict
ankit5990 Mar 31, 2015
e8c0d39
Merge remote-tracking branch 'upstream/master'
Apr 1, 2015
9b00506
Merge remote-tracking branch 'upstream/master'
Apr 2, 2015
ec04cb4
fix(datepicker): fixed 'show-weeks' datepickerPopup. added tests.
Apr 3, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
fix(datepicker): memory leak fix for datepicker.
  • Loading branch information
Ankit Kumar committed Sep 10, 2014
commit 0f8c8b44a736537220c69fea45b36ebfa55441ab
2 changes: 2 additions & 0 deletions src/datepicker/datepicker.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -605,6 +605,8 @@ function ($compile, $parse, $document, $position, dateFilter, dateParser, datepi
};

var $popup = $compile(popupEl)(scope);
popupEl.remove();

if ( appendToBody ) {
$document.find('body').append($popup);
} else {
Expand Down