add todos, docs

Daniel15 · Feb 10, 2011 · daa364b · daa364b
1 parent cd03b10
commit daa364b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/core/AbstractManager.js b/core/AbstractManager.js
@@ -32,7 +32,8 @@ AjaxSolr.AbstractManager = AjaxSolr.Class.extend(
   proxyUrl: null,
 
   /**
-   * The default Solr servlet.
+   * The default Solr servlet. You may prepend the servlet with a core if using
+   * multiple cores.
    *
    * @field
    * @public

diff --git a/helpers/jquery/ajaxsolr.theme.js b/helpers/jquery/ajaxsolr.theme.js
@@ -7,6 +7,7 @@
  * @param {String} list The list to append items to.
  * @param {Array} items The list of items to append to the list.
  * @param {String} [separator] A string to add between the items.
+ * @todo Return HTML rather than modify the DOM directly.
  */
 AjaxSolr.theme.prototype.list_items = function (list, items, separator) {
   jQuery(list).empty();

diff --git a/widgets/jquery/PagerWidget.js b/widgets/jquery/PagerWidget.js
@@ -10,6 +10,8 @@
  * @expects this.target to be a list.
  * @class PagerWidget
  * @augments AjaxSolr.AbstractWidget
+ * @todo Don't use the manager to send the request. Request only the results,
+ * not the facets. Update only itself and the results widget.
  */
 AjaxSolr.PagerWidget = AjaxSolr.AbstractWidget.extend(
   /** @lends AjaxSolr.PagerWidget.prototype */