Merge pull request #405 from Gluejar/ui_nits

Ui nits

core/models.py

@@ -1121,6 +1121,12 @@ class Work(models.Model):
     def openlibrary_url(self):
         return "http://openlibrary.org" + self.openlibrary_id
     
+    def uses_google_cover(self):
+        if self.preferred_edition and self.preferred_edition.cover_image:
+            return False
+        else: 
+            return self.googlebooks_id
+    
     def cover_image_small(self):
         if self.preferred_edition and self.preferred_edition.cover_image_small():
             return self.preferred_edition.cover_image_small()

frontend/templates/work.html

@@ -71,7 +71,7 @@
                 <div class="js-maincol-inner">
                     <div id="content-block">
                         <div class="book-detail">
-                            {% if work.googlebooks_id %}
+                            {% if work.uses_google_cover %}
                                 <div id="book-detail-img">
                                     <a href="{{ work.googlebooks_url }}">
                                     <img src="{{ work.cover_image_thumbnail }}" alt="Find {{ work.title }} at Google Books" title="Find {{ work.title }} at Google Books" width="131" height="192" /></a>
@@ -210,14 +210,8 @@
                                             <a rel="nofollow" id="find-oclc" href="http://www.worldcat.org/oclc/{{ work.first_oclc }}"><img src="/static/images/supporter_icons/worldcat_square.png" title="Find on Worldcat" alt="Find on Worldcat" /></a>
                                         {% endif %}
                                         <a rel="nofollow" class="find-openlibrary" href="{% url work_openlibrary work_id %}"><img src="/static/images/supporter_icons/openlibrary_square.png" title="Find on OpenLibrary" alt="Find on OpenLibrary" /></a>
-                                        {% if not request.user.is_anonymous %}
-                                            {% if request.user.profile.goodreads_user_link %}
                                         <a rel="nofollow" class="find-goodreads" href="{% url work_goodreads work_id %}"><img src="/static/images/supporter_icons/goodreads_square.png" title="Find on GoodReads"  alt="Find on GoodReads" /></a>
-                                            {% endif %}
-                                            {% if request.user.profile.librarything_id %}
                                         <a rel="nofollow" class="find-librarything" href="{% url work_librarything work_id %}"><img src="/static/images/supporter_icons/librarything_square.png" title="Find on LibraryThing" alt="Find on LibraryThing" /></a>
-                                            {% endif %}
-                                        {% endif %}
                                     </div>
                                 </div>                            
                                 <div class="btn_wishlist" id="wishlist_actions">
@@ -278,7 +272,7 @@
                                 </div>
                             </div>
                             <div id="tabs-2" class="tabs {% if activetab == '2' %}active{% endif %}">
-                                <h3>Why unglue this?  Have your say.</h3>
+                                <h3>Why {% if work.ebooks.all %}read{% else %}unglue{% endif %} this book? Have your say.</h3>
                                 <div class="tabs-content">
                                    {% render_comment_list for work %}
                                    {% if user.is_authenticated %}

static/less/book_detail.less

@@ -40,7 +40,7 @@
 	h3.book-author, h3.book-year {
 		font-size: @font-size-default;
 		font-weight:normal;
-		color:@medium-blue;
+		color:@text-blue;
 	}
 	
 	> div {