r3387 committed - autocomplete: cleanup, todos

r3387 committed - autocomplete: cleanup, todos


Revision: 3387
Author: joern.zaefferer
Date: Fri Oct 23 09:16:15 2009
Log: autocomplete: cleanup, todos
http://code.google.com/p/jquery-ui/source/detail?r=3387
Modified:
/branches/dev/tests/visual/autocomplete/combobox.html
/branches/dev/tests/visual/autocomplete/remote-jsonp.html
/branches/dev/tests/visual/autocomplete/remote.html
/branches/dev/tests/visual/autocomplete/search.php
=======================================
--- /branches/dev/tests/visual/autocomplete/combobox.html    Thu Oct 22
12:08:24 2009
+++ /branches/dev/tests/visual/autocomplete/combobox.html    Fri Oct 23
09:16:15 2009
@@ -64,6 +64,7 @@
                }).css("top", "")
                .click(function() {
                    // close if already visible
+                    // TODO but only if shown by button click
                    if (input.autocomplete("widget").is(":visible")) {
                        input.autocomplete("close");
                        return;
@@ -80,7 +81,7 @@
    })(jQuery);
    $(function() {
-        $('<div/>').css({
+        $.fn.themeswitcher && $('<div/>').css({
            position: "absolute",
            right: 10,
            top: 10
@@ -101,7 +102,7 @@
    </script>
    <style>
        body { font-size:62.5%; }
-        /* shouldn't be necessary */
+        /* TODO shouldn't be necessary */
        .ui-button-icon-only .ui-button-text { padding: 0; }
    </style>
</head>
=======================================
--- /branches/dev/tests/visual/autocomplete/remote-jsonp.html    Sat Oct 17
05:09:52 2009
+++ /branches/dev/tests/visual/autocomplete/remote-jsonp.html    Fri Oct 23
09:16:15 2009
@@ -13,7 +13,7 @@
    <script type="text/javascript"
src="http://jqueryui.com/themeroller/themeswitchertool/"></script>
    <script type="text/javascript">
    $(function() {
-        $('<div/>').css({
+        $.fn.themeswitcher && $('<div/>').css({
            position: "absolute",
            right: 10,
            top: 10
=======================================
--- /branches/dev/tests/visual/autocomplete/remote.html    Sat Oct 17 04:40:40
2009
+++ /branches/dev/tests/visual/autocomplete/remote.html    Fri Oct 23 09:16:15
2009
@@ -13,7 +13,7 @@
    <script type="text/javascript"
src="http://jqueryui.com/themeroller/themeswitchertool/"></script>
    <script type="text/javascript">
    $(function() {
-        $('<div/>').css({
+        $.fn.themeswitcher && $('<div/>').css({
            position: "absolute",
            right: 10,
            top: 10
=======================================
--- /branches/dev/tests/visual/autocomplete/search.php    Tue Sep 29 15:29:20
2009
+++ /branches/dev/tests/visual/autocomplete/search.php    Fri Oct 23 09:16:15
2009
@@ -630,7 +630,7 @@
$result = array();
foreach ($items as $key=>$value) {
    if (strpos(strtolower($key), $q) !== false) {
-        array_push($result, array("id"=>$value, "label"=>$key));
+        array_push($result, array("id"=>$value, "label"=>$key, "result" =>
strip_tags($key)));
    }
    if (count($result) > 12)
        break;