Merge remote branch 'remotes/schendje/master'

mediagoblin/static/css/base.css

@@ -133,15 +133,6 @@ a.mediagoblin_logo:hover {
 
 /* common website elements */
 
-.dotted_line {
-  width:100%;
-  height:0px;
-  border-bottom: dotted 1px #5f5f5f;
-  position:absolute;
-  left:0px;
-  margin-top:-20px;
-}
-
 .button {
   font-family:'Carter One', arial, serif;
   height:32px;
@@ -164,6 +155,10 @@ a.mediagoblin_logo:hover {
   padding-right:11px;
 }
 
+.pagination{
+text-align:center;
+}
+
 /* forms */
 
 .form_box {
@@ -223,6 +218,17 @@ a.mediagoblin_logo:hover {
   width:280px;
 }
 
+/* comments */
+
+.comment_author {
+  margin-bottom:40px;
+  padding-top:4px;
+}
+
+.comment_content p {
+  margin-bottom:4px;
+}
+
 /* media galleries */
 
 ul.media_thumbnail {
@@ -240,3 +246,10 @@ li.media_thumbnail {
   zoom:1;
 . *display:inline;
 }
+
+/* icons */
+
+img.media_icon{
+  margin:0 4px;
+  vertical-align:sub;
+}

mediagoblin/templates/mediagoblin/user_pages/media.html

@@ -23,13 +23,16 @@
   {# temporarily, an "image gallery" that isn't one really ;) #}
   {% if media %}
     <div class="grid_8 alpha media_image">
-      <h1>
-        {{media.title}}
-      </h1>
-      <img class="media_image" src="{{ request.app.public_store.file_url(
+      <img src="{{ request.app.public_store.file_url(
                     media.media_files.main) }}" />
+      <h2>
+        {{media.title}}
+      </h2>
+      {% autoescape False %}
+        <p>{{ media.description_html }}</p>
+      {% endautoescape %}
       <p>
-        Uploaded on
+        &mdash;&nbsp;uploaded on
         {{ "%4d-%02d-%02d"|format(media.created.year,
                                   media.created.month, media.created.day) }}
         by
@@ -37,44 +40,36 @@
                                    user= media.uploader().username) }}">
           {{- media.uploader().username }}</a>
       </p>
-
-      {% autoescape False %}
-        <p>{{ media.description_html }}</p>
-      {% endautoescape %}
-
-      {% if media['uploader'] == request.user['_id'] %}
-        <p><a href="{{ request.urlgen('mediagoblin.edit.edit_media',
-                                      user= media.uploader().username,
-                                      media= media._id) }}">Edit</a></p>
-      {% endif %}
+      <br /><br />
+      <h3>Comments</h3>
       {% if request.user %}
         <form action="{{ request.urlgen('mediagoblin.user_pages.media_post_comment', 
                                          user= media.uploader().username,
                                          media=media._id) }}" method="POST">
-          <h3>Post a comment!</h3>
           {{ wtforms_util.render_field_div(comment_form.comment) }}
           <div class="form_submit_buttons">
-            <input type="submit" value="Submit" class="button" />
+            <input type="submit" value="Post comment!" class="button" />
           </div>
         </form>
       {% endif %}
-
       {#
       {{ wtforms_util.render_textarea_div(submit_form.description) }}
       {{ wtforms_util.render_field_div(submit_form.file) }}
       #}
-
       {% if comments %}
-        <h3>Comments</h3>
         {% for comment in comments %}
           <div class="comment_wrapper" id="comment-{{ comment['_id'] }}">
-  	  <div class="comment_author">By: 
+            <div class="comment_content">
+              {% autoescape False %}
+                {{ comment.content_html }}
+              {% endautoescape %}
+            </div>
+            <div class="comment_author">&mdash; 
               <a href="{{ request.urlgen('mediagoblin.user_pages.user_home',
                                 user = comment['author']['username']) }}">
-  	      {{ comment['author']['username'] }}
-  	    </a>
-  	  </div>
-  	  <div class="comment_datetime">
+              {{ comment['author']['username'] }}</a> at 
+          <!--</div>
+          <div class="comment_datetime">-->
               <a href="#comment-{{ comment['_id'] }}">
                 {{ "%4d-%02d-%02d %02d:%02d"|format(comment.created.year,
                                          comment.created.month,
@@ -83,19 +78,21 @@
                                          comment.created.minute) }}
               </a>
             </div>
-            <div class="comment_content">
-              {% autoescape False %}
-                {{ comment.content_html }}
-              {% endautoescape %}
-  	  </div>
          </div>
         {% endfor %}
         {% include "mediagoblin/utils/pagination.html" %}
       </div>
     {% endif %}
-
     <div class="grid_4 omega media_sidebar">
-      <p>This is a sidebar! Yay!</p>
+      <h3>Sidebar content here!</h3>
+      <p>
+       {% if media['uploader'] == request.user['_id'] %}
+        <p><a href="{{ request.urlgen('mediagoblin.edit.edit_media',
+                                      user= media.uploader().username,
+                                      media= media._id) }}"><img src="{{ request.staticdirect('/images/iconEdit.png') }}" class="media_icon" />edit</a></p>
+        <p><img src="{{ request.staticdirect('/images/iconDelete.png') }}" class="media_icon" />delete</p>
+        {% endif %}
+      </p>
     </div>
   {% else %}
   <p>Sorry, no such media found.<p/>

mediagoblin/templates/mediagoblin/utils/pagination.html

@@ -19,9 +19,9 @@
 
 {% if pagination %}
   <div class="pagination">
-
+  <p>
       {% if pagination.has_prev %}
-        <a href="{{ pagination.get_page_url(request, pagination.page-1) }}">&laquo; Prev</>
+        <a href="{{ pagination.get_page_url(request, pagination.page-1) }}">&laquo; Prev</a>
       {% endif %}
 
       {%- for page in pagination.iter_pages() %}
@@ -29,7 +29,7 @@
           {% if page != pagination.page %}
       <a href="{{ pagination.get_page_url(request, page) }}">{{ page }}</a>
           {% else %}
-            <strong>{{ page }}</strong>
+            {{ page }}
           {% endif %}
         {% else %}
           <span class="ellipsis">…</span>
@@ -39,6 +39,7 @@
       {% if pagination.has_next %}
         <a href="{{ pagination.get_page_url(request, pagination.page + 1) }}">Next &raquo;</a>
       {% endif %}
+     </p>
    </div>
 {% endif %}