Datepicker: removed the mozilla check when building the year select box. Fixed #7176 - Datepicker shows only single year in year dropdown if numberOfMonths > 1

Chris Rohr · Chris Rohr · commit 16de24b54a03 · 2011-03-31T23:56:56.000-04:00
diff --git a/tests/visual/datepicker/datepicker_ticket_7176.html b/tests/visual/datepicker/datepicker_ticket_7176.html
@@ -0,0 +1,23 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+	<meta charset="UTF-8" />
+	<title>Datepicker Visual Test : Default</title>
+	<link rel="stylesheet" href="../visual.css" type="text/css" />
+	<link rel="stylesheet" href="../../../themes/base/jquery.ui.all.css" type="text/css">
+	<script type="text/javascript" src="../../../jquery-1.5.1.js"></script>
+	<script type="text/javascript" src="../../../ui/jquery.ui.core.js"></script>
+	<script type="text/javascript" src="../../../ui/jquery.ui.widget.js"></script>
+	<script type="text/javascript" src="../../../ui/jquery.ui.datepicker.js"></script>
+	<script type="text/javascript">
+	$(function() {
+		$('#datepicker').datepicker({numberOfMonths: 3, changeMonth: true, changeYear: true, yearRange: '2007:2015'});
+	});
+	</script>
+</head>
+<body>
+
+<div id="datepicker"></div>
+
+</body>
+</html>
diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js
@@ -1606,14 +1606,8 @@ $.extend(Datepicker.prototype, {
 					'>' + year + '</option>';
 			}
 			inst.yearshtml += '</select>';
-			//when showing there is no need for later update
-			if( ! $.browser.mozilla ){
-				html += inst.yearshtml;
-				inst.yearshtml = null;
-			} else {
-				// will be replaced later with inst.yearshtml
-				html += '<select class="ui-datepicker-year"><option value="' + drawYear + '" selected="selected">' + drawYear + '</option></select>';
-			}
+			html += inst.yearshtml;
+			inst.yearshtml = null;
 		}
 		html += this._get(inst, 'yearSuffix');
 		if (showMonthAfterYear)
