Fix some issues with the new tab styling
authorTom MacWright <tom@macwright.org>
Tue, 28 Feb 2012 23:25:23 +0000 (18:25 -0500)
committerTom Hughes <tom@compton.nu>
Sun, 4 Mar 2012 10:37:46 +0000 (10:37 +0000)
Uses the parent of the tabnav element to do its positioning, and fixes
this style in RTL and small.

Also move the greetings block into the top-bar div in order to cleanly
have it share a border with tabs.

app/assets/stylesheets/common.css.scss
app/assets/stylesheets/ltr.css.scss
app/assets/stylesheets/rtl.css.scss
app/assets/stylesheets/small.css.scss
app/views/layouts/site.html.erb

index d62be0e..12a5a80 100644 (file)
@@ -248,9 +248,12 @@ h2 {
 
 /* Rules for tabbed navigation bar */
 
-#tabnav {
-  height: 29px;
+#top-bar {
   border-bottom: 1px solid #ccc;
+  height: 29px;
+}
+
+#tabnav {
   padding: 0;
   margin: 0;
   overflow:hidden;
@@ -293,10 +296,10 @@ h2 {
 /* Rules for greeting bar in the top right corner */
 
 #greeting {
-  height: 30px;
+  height: 28px;
   margin: 0px;
   font-size: 13px;
-  line-height: 30px;
+  line-height: 28px;
 }
 
 .greeting-bar-unread {
index 039d901..5eba8dd 100644 (file)
@@ -68,7 +68,7 @@ html body {
 
 /* Rules for tabbed navigation bar */
 
-#tabnav {
+#top-bar {
   margin-left: 185px;
 }
 
index 669889c..b9223e5 100644 (file)
@@ -68,15 +68,15 @@ html body {
 
 /* Rules for tabbed navigation bar */
 
-#tabnav {
-  margin-right: 185px;
+#top-bar {
+  margin-right: 185px
 }
 
 #tabnav a,
 #tabnav a:link,
 #tabnav a:visited {
   float: right;
-  margin-left: 4px;
+  margin-left: 1px;
 }
 
 /* Rules for greeting bar in the top right corner */
index 59a6469..412cf72 100644 (file)
@@ -17,9 +17,11 @@ h1 {
 }
 
 /* Rules for tabbed navigation bar */
+#top-bar {
+  margin: 0;
+}
 
-#tabnav
-{
+#tabnav {
   height: 14px;
   margin: 0px;
   padding-top: 5px;
@@ -28,8 +30,7 @@ h1 {
   line-height: 10px;
 }
 
-#tabnav a, #tabnav a:link, #tabnav a:visited 
-{
+#tabnav a, #tabnav a:link, #tabnav a:visited {
   font-size: 10px;
   line-height: 10px;
   padding: 2px 5px;
index f43d3b3..b6fb089 100644 (file)
       <%= yield %>
     </div>
 
-    <span id="greeting">
-      <% if @user and @user.id %>
-        <span id="full-greeting"><%= raw(t 'layouts.welcome_user', :user_link => (link_to h(@user.display_name), {:controller => 'user', :action => 'view', :display_name => @user.display_name}, :title => t('layouts.welcome_user_link_tooltip'))) %></span> 
-        <span id="small-greeting"><%= link_to t('layouts.welcome_user_link_tooltip'), {:controller => 'user', :action => 'view', :display_name => @user.display_name} %></span> | 
-        <%= yield :greeting %>
-        <%= render :partial => "layouts/inbox" %> |
-        <%= link_to t('layouts.logout'), {:controller => 'user', :action => 'logout', :session => request.session_options[:id], :referer => request.fullpath}, {:id => 'logoutanchor', :title => t('layouts.logout_tooltip')}%>
-      <% else %>
-        <%= link_to t('layouts.log_in'), {:controller => 'user', :action => 'login', :referer => request.fullpath}, {:id => 'loginanchor', :title => t('layouts.log_in_tooltip')} %> |
-        <%= link_to t('layouts.sign_up'), {:controller => 'user', :action => 'new'}, {:id => 'registeranchor', :title => t('layouts.sign_up_tooltip')} %>
-      <% end %>
-    </span>
-
-    <div>
+    <div id='top-bar'>
+      <span id="greeting">
+         <% if @user and @user.id %>
+           <span id="full-greeting"><%= raw(t 'layouts.welcome_user', :user_link => (link_to h(@user.display_name), {:controller => 'user', :action => 'view', :display_name => @user.display_name}, :title => t('layouts.welcome_user_link_tooltip'))) %></span> 
+           <span id="small-greeting"><%= link_to t('layouts.welcome_user_link_tooltip'), {:controller => 'user', :action => 'view', :display_name => @user.display_name} %></span> | 
+           <%= yield :greeting %>
+           <%= render :partial => "layouts/inbox" %> |
+           <%= link_to t('layouts.logout'), {:controller => 'user', :action => 'logout', :session => request.session_options[:id], :referer => request.fullpath}, {:id => 'logoutanchor', :title => t('layouts.logout_tooltip')}%>
+         <% else %>
+           <%= link_to t('layouts.log_in'), {:controller => 'user', :action => 'login', :referer => request.fullpath}, {:id => 'loginanchor', :title => t('layouts.log_in_tooltip')} %> |
+           <%= link_to t('layouts.sign_up'), {:controller => 'user', :action => 'new'}, {:id => 'registeranchor', :title => t('layouts.sign_up_tooltip')} %>
+         <% end %>
+      </span>
       <ul id="tabnav">
         <%
         viewclass = 'geolink llz layers'