Clean up some todos/comments

Author: mikejritter

src/components/search/grid/SearchResultGrid.jsx

@@ -18,6 +18,7 @@ const GRID_COLUMN_SET = 'grid';
 const cardPropTypes = {
   result: PropTypes.instanceOf(Immutable.Map),
   index: PropTypes.number,
+  listType: PropTypes.string,
   titleFields: PropTypes.array,
   subtitleFields: PropTypes.array,
   searchDescriptor: PropTypes.object,
@@ -41,7 +42,6 @@ export function SearchResultCard({
     location = getItemLocationPath(result, { config, searchDescriptor });
   }
 
-  // todo: read fields from record config
   // todo: loading image
   return (
     <div style={{ paddingBottom: '10px' }}>

src/components/search/list/SearchList.jsx

@@ -19,7 +19,6 @@ const DETAIL_COLUMN_SET = 'list';
 
 const itemPropTypes = {
   item: PropTypes.instanceOf(Immutable.Map),
-  // exact shape tbd
   index: PropTypes.number,
   listItems: PropTypes.array,
   // render context like search table? or a better way to handle all this?
@@ -116,8 +115,6 @@ function SearchDetailList({ searchDescriptor, intl, listType = 'search' }) {
 
   // Note: The items returned is an Immutable.Map, so we need to use get
   // in order to retrieve the data
-  // Note x2: This is only available for 'new' searches, so we don't need a list type prop
-  // todo: read into the search results based on the list type
   // todo: why do we need to do !results AND !items?
   const items = readListItems(config, listType, results);
   if (!items) {

src/components/search/table/SearchTable.jsx

@@ -92,10 +92,8 @@ function getSortDir(searchDescriptor) {
  * to be displayed.
  *
  * todo: parity with other search result table
- *   - row data formatting
- *   - sorting
- *   - checkboxes actually working
  *   - aria-labels + general wcag compliance
+ *   - figure out hrefs
  *   - ???
  */
 function SearchResultTable({ searchDescriptor, listType = 'common', intl }) {
@@ -113,7 +111,6 @@ function SearchResultTable({ searchDescriptor, listType = 'common', intl }) {
 
   // Note: The items returned is an Immutable.Map, so we need to use get
   // in order to retrieve the data
-  // todo: read into the search results based on the list type
   // todo: why do we need to do !results AND !items?
   const items = readListItems(config, listType, results);
   if (!items) {
@@ -129,7 +126,6 @@ function SearchResultTable({ searchDescriptor, listType = 'common', intl }) {
   // read headers
   const columnConfig = getColumnConfig(config, searchDescriptor, 'default');
 
-  // console.log(`column config: ${JSON.stringify(columnConfig)}`);
   // todo: dataKey is for ucb support. basically it allows multiple fields to be used
   // in the event one is missing. See cspace-ui-plugin-profile-pahma.js
   const columns = Object.keys(columnConfig)