Project

General

Profile

Actions

Defect #2641

closed

other-formats paragraph location is inconsistent

Added by Brad Beattie almost 16 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
UI
Target version:
-
Start date:
2009-02-02
Due date:
% Done:

50%

Estimated time:
0.20 h
Resolution:
Cant reproduce
Affected version:

Description

On an issue view page, the other-formats paragraph appears at the bottom of the page, just above the footer through use of

<div style="clear: both;"/>

On other pages, the other-formats paragraph appears simply as the last element in the #content div.

It doesn't really matter to me which one is used, but I figured consistency would be good, yeah? The following patch puts all at the bottom through use of relative absolute positioning with the #content div and, consequently, removes the single instance of the clear div.

### Eclipse Workspace Patch 1.0
#P Redmine
Index: app/views/issues/show.rhtml
===================================================================
--- app/views/issues/show.rhtml    (revision 2338)
+++ app/views/issues/show.rhtml    (working copy)
@@ -99,7 +99,6 @@
 <%= render :partial => 'history', :locals => { :journals => @journals } %>
 </div>
 <% end %>
-<div style="clear: both;"></div>

 <% if authorize_for('issues', 'edit') %>
   <div id="update" style="display:none;">
Index: public/stylesheets/application.css
===================================================================
--- public/stylesheets/application.css    (revision 2352)
+++ public/stylesheets/application.css    (working copy)
@@ -55,7 +55,7 @@
 #sidebar hr{ width: 100%; margin: 0 auto; height: 1px; background: #ccc; border: 0; }
 * html #sidebar hr{ width: 95%; position: relative; left: -6px; color: #ccc; }

-#content { width: 80%; background-color: #fff; margin: 0px; border-right: 1px solid #ddd; padding: 6px 10px 10px 10px; z-index: 10; }
+#content { width: 80%; background-color: #fff; margin: 0px; border-right: 1px solid #ddd; padding: 6px 10px 10px 10px; z-index: 10; position: relative; }
 * html #content{ width: 80%; padding-left: 0; margin-top: 0px; padding: 6px 10px 10px 10px;}
 html>body #content { min-height: 600px; } 
 * html body #content { height: 600px; } /* IE */
@@ -308,7 +308,7 @@
 div.attachments img { vertical-align: middle; }
 div.attachments span.author { font-size: 0.9em; color: #888; }

-p.other-formats { text-align: right; font-size:0.9em; color: #666; }
+p.other-formats { text-align: right; font-size:0.9em; color: #666; position: absolute; bottom: 0; right: 1em; }
 .other-formats span + span:before { content: "| "; }

 a.atom { background: url(../images/feed.png) no-repeat 1px 50%; padding: 2px 0px 3px 16px; }
Actions #1

Updated by Brad Beattie almost 16 years ago

  • Assignee set to Jean-Philippe Lang
Actions #2

Updated by Anonymous almost 6 years ago

Maybe it's safe to close this issue now, because it's obsolete.
1. Seems like it's from the era when views used to be in .rhtml format
2. <div style="clear: both;"></div> line don't exist in show.html.erb already and

<% if authorize_for('issues', 'edit') %>
<div id="update" style="display:none;">

is now a part of a separate file, called _action_menu_edit.html.erb.

3. " The other-formats paragraph appears at the bottom of the page, just above the footer through use of <div style="clear: both;"/> " It doesn't appear above footer the way it probably used to anymore and <div style="clear:both;"></div> probably shouldn't actually exist anywhere, because it's another useless DIV block. It is in base.html.erb now though.

4. It's not a .patch or .diff file people could easily test.

Actions #3

Updated by Marius BÄ‚LTEANU almost 6 years ago

  • Tracker changed from Patch to Defect
  • Status changed from New to Closed
  • Assignee deleted (Jean-Philippe Lang)
  • Resolution set to Cant reproduce
Actions

Also available in: Atom PDF