Id converted to class

internetarchive · Feb 18, 2021 · 0f88df0 · 0f88df0
1 parent fb25af9
commit 0f88df0
Show file tree

Hide file tree

Showing 8 changed files with 14 additions and 15 deletions.
diff --git a/openlibrary/macros/FulltextResults.html b/openlibrary/macros/FulltextResults.html
@@ -4,7 +4,7 @@
   $ hits = results['hits'].get('hits', [])
   $ num_found = results['hits'].get('total', 0)
   <div id="searchResults">
-    <ul id="siteSearch" class="list-books">
+    <ul class="list-books">
       $for doc in hits:                
         $if doc.get('edition'):
           $ snippet = macros.FulltextSnippet(query, doc=doc)

diff --git a/openlibrary/macros/SearchResults.html b/openlibrary/macros/SearchResults.html
@@ -2,7 +2,7 @@
 
 $ _ = i18n.get_namespace('/search')
 
-<ul id="siteSearch">
+<ul class="list-books">
 
 $for b in results:
     $if fulltext:

diff --git a/openlibrary/templates/account/books.html b/openlibrary/templates/account/books.html
@@ -103,7 +103,7 @@ <h3>
       </div>
       <div class="mybooks-list">
         $:macros.Pager(current_page, total_items, results_per_page=25)
-        <ul class="list-books" id="siteSearch">
+        <ul class="list-books">
           $if items:
             $ bookshelf_id = {'want-to-read': 1, 'currently-reading': 2, 'already-read': 3}.get(key, None)
             $for item in items:

diff --git a/openlibrary/templates/books/check.html b/openlibrary/templates/books/check.html
@@ -16,7 +16,7 @@ <h1>$_("Add a Book")</h1>
 
 <div id="contentBody">
     <div id="searchResults">
-        <ul id="siteSearch" class="list-books">
+        <ul class="list-books">
         $for work in matches:
             <li class="workResult" id="$work.key.split('/')[-1]">
                 <span class="bookcover">
@@ -55,7 +55,7 @@ <h3 class="booktitle"><a href="javascript:;" title="$_('Select this book')">$wor
 <script type="text/javascript">
 <!--
 window.q.push(function() {
-    \$("#siteSearch a").click(function() {
+    \$(".list-books a").click(function() {
         var li = \$(this).parents('li:first');
         \$("input#work").val("/works/" + li.attr("id"));
         \$("form#addbook").submit();

diff --git a/openlibrary/templates/stats/readinglog.html b/openlibrary/templates/stats/readinglog.html
@@ -61,15 +61,15 @@ <h2>$_('# Books Starred')</h2>
   <div class="admin-section">
     <h3>$_('Most Wanted Books (All Time)')</h3>
     <div class="mybooks-list">
-      <ul class="list-books" id="siteSearch">
+      <ul class="list-books">
         $for book in stats['leaderboard']['most_wanted_all']:
             $:macros.SearchResultsWork(book['work'], decorations=ungettext("Added 1 time", "Added %(count)d times", book['cnt'], count=book['cnt']))
       </ul>
     </div>
 
     <h3>$_('Most Wanted Books (This Month)')</h3>
     <div class="mybooks-list">
-      <ul class="list-books" id="siteSearch">
+      <ul class="list-books">
         $for book in stats['leaderboard']['most_wanted_month']:
             $:macros.SearchResultsWork(book['work'], decorations=ungettext("Added 1 time", "Added %(count)d times", book['cnt'], count=book['cnt']))
       </ul>
@@ -78,7 +78,7 @@ <h3>$_('Most Wanted Books (This Month)')</h3>
     <h2>$_('Most Logged Books')</h2>
     <h3>$_('Most Read Books (All Time)')</h3>
     <div class="mybooks-list">
-      <ul class="list-books" id="siteSearch">
+      <ul class="list-books">
         $for book in stats['leaderboard']['most_read']:
             $:macros.SearchResultsWork(book['work'], decorations=ungettext("Added 1 time", "Added %(count)d times", book['cnt'], count=book['cnt']))
       </ul>
@@ -87,7 +87,7 @@ <h3>$_('Most Read Books (All Time)')</h3>
     <h2>$_('Most Rated Books')</h2>
     <h3>$_('Most Rated Books (All Time)')</h3>
     <div class="mybooks-list">
-      <ul class="list-books" id="siteSearch">
+      <ul class="list-books">
         $for book in stats['leaderboard']['most_rated_all']:
             $:macros.SearchResultsWork(book['work'], decorations=ungettext("Added 1 time", "Added %(count)d times", book['cnt'], count=book['cnt']))
       </ul>

diff --git a/openlibrary/templates/type/author/view.html b/openlibrary/templates/type/author/view.html
@@ -158,7 +158,7 @@ <h2 class="collapse">
                 $:macros.Pager(page=safeint(query_param('page'), default=1), num_found=books.num_found)
 
                 <div id="searchResults">
-                    <ul id="siteSearch" class="list-books">
+                    <ul class="list-books">
                       $for doc in books.works:
                          $:macros.SearchResultsWork(doc)
                     </ul>

diff --git a/openlibrary/templates/work_search.html b/openlibrary/templates/work_search.html
@@ -212,7 +212,7 @@
         <div class="shift">$ungettext('1 hit', '%(count)s hits', num_found, count=commify(num_found))</div>
         <div class="resultsContainer search-results-container">
         <div id="searchResults">
-          <ul class="list-books" id="siteSearch">
+          <ul class="list-books">
             $ works = add_availability([get_doc(d) for d in docs])
 
             $for work in works:

diff --git a/static/css/legacy.less b/static/css/legacy.less
@@ -1662,8 +1662,7 @@ span.actions {
 // openlibrary/templates/account/books.html
 // openlibrary/macros/SearchResults.html
 // openlibrary/macros/FulltextResults.html
-// why not form#siteSearch ?
-ul#siteSearch {
+ul.list-books {
   > li {
     .display-flex();
   }
@@ -1715,7 +1714,7 @@ ul#siteSearch {
 // openlibrary/templates/search/lists.html
 // openlibrary/templates/lists/lists.html
 div.wide {
-  ul#siteSearch {
+  ul.list-books {
     span.resultTitle {
       max-width: 850px;
     }
@@ -1726,7 +1725,7 @@ div.wide {
 // openlibrary/templates/type/list/view.html
 // openlibrary/templates/type/work/editions.html
 div.narrow {
-  ul#siteSearch {
+  ul.list-books {
     span.resultTitle {
       max-width: 500px !important;
     }