Disable autocomplete on datepicker and datetimepicker inputs

[Ryan J Ollos]

It would be nice to disable autocomplete on the datepicker and datetimepicker inputs.

Proposed change:

trac/templates/layout.html

@@ -62 +62 @@
         $(".trac-autofocus").focus();
         $(".trac-target-new").attr("target", "_blank");
         if ($.ui) { /* is jquery-ui added? */
-          $(".trac-datepicker:not([readonly])").datepicker();
-          $(".trac-datetimepicker:not([readonly])").datetimepicker();
+          $(".trac-datepicker:not([readonly])").datepicker()
+            .focus(function() {
+              $(this).prop("autocomplete", "off");
+//              return false;
+          });
+          $(".trac-datetimepicker:not([readonly])").datetimepicker()
+            .focus(function() {
+              $(this).prop("autocomplete", "off");
+//              return false;
+          });
         }
         $(".trac-disable").disableSubmit(".trac-disable-determinant");
         setTimeout(function() { $(".trac-scroll").scrollToTop() }, 1);

Should I return false to avoid the default behavior and prevent bubbling? So far I can't see any difference in behavior when returning false vs no return value.

[Jun Omae]

Good idea.

I think we could simply set autocomplete="off" without focus() listener.

-          $(".trac-datepicker:not([readonly])").datepicker();
-          $(".trac-datetimepicker:not([readonly])").datetimepicker();
+          $(".trac-datepicker:not([readonly])").prop("autocomplete", "off").datepicker();
+          $(".trac-datetimepicker:not([readonly])").prop("autocomplete", "off").datetimepicker();

[Ryan J Ollos]

Thanks, that's much simpler. Committed to trunk in [14344].

[jnanendra veer]

have you get any solution for this?