SONAR-1492 Apply the "bleu" color on actions links in admin section

For more consistency.
This "link-action" style is used in places where the text is dense
(like rule descriptions in profile or tables) and where links must
be clearly visible.

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/admin_dashboards/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/admin_dashboards/index.html.erb
index 3ac4e156952590173cdf33e9e474b4020076496f..9590cd4ef353b01a7fef51c2941ec292da05e329 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/admin_dashboards/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/admin_dashboards/index.html.erb
@@ -35,7 +35,7 @@
           <% end %>
         </td>
         <td class="thin nowrap right">
-          <%= link_to 'Remove from defaults', {:action => 'remove', :id => active.id}, :confirm => 'Are you sure to remove it from default dashboards ?', :method => :post, :id => "remove-#{u active.name}" %>
+          <%= link_to 'Remove from defaults', {:action => 'remove', :id => active.id}, {:confirm => 'Are you sure to remove it from default dashboards ?', :method => :post, :id => "remove-#{u active.name}", :class => 'link-action'} %>
         </td>
       </tr>
     <% end %>
@@ -70,11 +70,11 @@
         </td>
         <td><%= h(dashboard.user.name) if dashboard.user %></td>
         <td class="thin nowrap right">
-          <%= link_to 'Add to defaults', {:action => 'add', :id => dashboard.id}, :method => :post, :id => "add-#{u dashboard.name}" %>
+          <%= link_to 'Add to defaults', {:action => 'add', :id => dashboard.id}, {:method => :post, :id => "add-#{u dashboard.name}", :class => 'link-action'} %>
         </td>
         <td class="thin nowrap right">
           <% if dashboard.provided_programmatically? %>
-            <%= link_to 'Delete', {:action => 'delete', :id => dashboard.id}, :confirm => "Do you want to delete the dashboard #{dashboard.name(true)}?", :method => :post, :id => "delete-#{u dashboard.name}" %>
+            <%= link_to 'Delete', {:action => 'delete', :id => dashboard.id}, {:confirm => "Do you want to delete the dashboard #{dashboard.name(true)}?", :method => :post, :id => "delete-#{u dashboard.name}", :class => 'link-action'} %>
           <% end %>
         </td>
       </tr>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/admin_filters/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/admin_filters/index.html.erb
index 9569c7005557acc4f58c6f841943e3da42bfb8f7..22002452b2ce72fe62bad920a94020377366d6cc 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/admin_filters/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/admin_filters/index.html.erb
@@ -31,7 +31,7 @@
           <% end %>
         </td>
         <td>
-          <%= link_to 'Remove', {:action => 'remove', :id => active.id}, :confirm => 'Are you sure to remove it from default filters ?', :method => :post, :id => "remove-#{u active.name}" %>
+          <%= link_to 'Remove', {:action => 'remove', :id => active.id}, {:confirm => 'Are you sure to remove it from default filters ?', :method => :post, :id => "remove-#{u active.name}", :class => 'link-action'} %>
         </td>
       </tr>
     <% end %>
@@ -62,7 +62,7 @@
       <tr class="<%= cycle('even', 'odd') -%>">
         <td><%= h(filter.name) -%></td>
         <td><%= h(filter.user.name) if filter.user -%></td>
-        <td><%= link_to 'Add to defaults', {:action => 'add', :id => filter.id}, :method => :post, :id => "add-#{u filter.name}" %></td>
+        <td><%= link_to 'Add to defaults', {:action => 'add', :id => filter.id}, {:method => :post, :id => "add-#{u filter.name}", :class => 'link-action'} %></td>
       </tr>
     <% end %>
   <% end %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/groups/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/groups/index.html.erb
index 9515cb7471ca40dce1c269c75151b3776e6b49ed..bfa438fdc153f3036fd8254c0f83135eb5a6406e 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/groups/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/groups/index.html.erb
@@ -19,12 +19,14 @@
             <td class="left"><%= group.name %></td>
             <td class="left" style="word-break:break-all"><%=group.description%></td>
             <td class="left">
-              <span id="count-<%= u group.name -%>"><%= group.users.count %></span> (<%= link_to "select", { :action => 'select_user', :id => group.id}, :id => "select-#{u group.name}" %>)
+              <span id="count-<%= u group.name -%>"><%= group.users.count %></span> (<%= link_to "select", { :action => 'select_user', :id => group.id}, {:id => "select-#{u group.name}", :class => 'link-action'} %>)
             </td>
             <td class="left">
-              <%= link_to "edit", { :action => 'index', :id => group.id}, :class => 'action',
-              :method => :get, :id => "edit-#{u group.name}" %> | 
-              <%= link_to "delete", { :action => 'destroy', :id => group.id}, {:confirm => "Are you sure to delete this group ? Members will not be deleted.", :class => 'action', :method => 'get', :id => "delete-#{u group.name}"} %>
+              <%= link_to "edit", { :action => 'index', :id => group.id}, 
+                                  { :method => :get, :id => "edit-#{u group.name}", :class => 'link-action'} %>
+               
+              <%= link_to "delete", { :action => 'destroy', :id => group.id}, 
+                                    {:confirm => "Are you sure to delete this group ? Members will not be deleted.", :class => 'link-action', :method => 'get', :id => "delete-#{u group.name}"} %>
             </td>
           </tr>
         <% end %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/manual_rules/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/manual_rules/index.html.erb
index 79b2e2fe174acabe3c5c10774e6b3f3743ddd004..21bfa98df6c6b311b1b37bd2e72f1fcb9dff4a0c 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/manual_rules/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/manual_rules/index.html.erb
@@ -22,8 +22,8 @@
               <span class="note"><%= h rule.description -%></span>
             </td>
             <td class="right thin nowrap">
-              <%= link_to 'Edit', {:action => 'edit', :id => rule.id}, {:id => "edit_#{u(rule.key)}"} %>
-              <%= link_to 'Delete', {:action => 'delete', :id => rule.id}, {:confirm => message('are_you_sure'), :id => "delete_#{u(rule.key)}", :method => 'delete'} %>
+              <%= link_to 'Edit', {:action => 'edit', :id => rule.id}, {:id => "edit_#{u(rule.key)}", :class => 'link-action'} %>
+              <%= link_to 'Delete', {:action => 'delete', :id => rule.id}, {:confirm => message('are_you_sure'), :id => "delete_#{u(rule.key)}", :class => 'link-action', :method => 'delete'} %>
             </td>
           </tr>
         <% end %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/metrics/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/metrics/index.html.erb
index 9de32a8aa63ebdb7eb12f29089eeac888bc4b955..0484469e3b2f2f5f7f7fe68c1895404dc808aaf6 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/metrics/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/metrics/index.html.erb
@@ -22,10 +22,10 @@
           <td class="left" ><%= metric.value_type_name -%></td>
           <td class="right thin nowrap">
             <% if is_admin? && metric.updatable_online? %>
-              <%= link_to 'Edit', {:action => 'index', :id => metric.id}, {:class => 'action', :id => "edit_#{h(metric.short_name)}", :method => 'get'} %>
+              <%= link_to 'Edit', {:action => 'index', :id => metric.id}, {:class => 'link-action', :id => "edit_#{h(metric.short_name)}", :method => 'get'} %>
             <% end %>
             <% if is_admin? && metric.updatable_online? %>
-              <%= link_to 'Delete', {:action => 'delete_from_web', :id => metric.id}, {:confirm => "Warning : all the measures will be deleted.", :id => "delete_#{h(metric.short_name)}"} %>
+              <%= link_to 'Delete', {:action => 'delete_from_web', :id => metric.id}, {:confirm => "Warning : all the measures will be deleted.", :class => 'link-action', :id => "delete_#{h(metric.short_name)}"} %>
             <% end %>
           </td>
         </tr>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/global.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/global.html.erb
index 06498d1c12af1dfaff4819f86e978702adeb987d..08c25d5b4cbffb6caf687c7ea364d49975edb68f 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/global.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/global.html.erb
@@ -13,11 +13,11 @@
     <td valign="top"><b>Administrators</b><br/><span class="small gray">Ability to perform all administration functions for the instance: global configuration, personalization of Time Machine and homepage.</span></td>
     <td valign="top" style="word-break:break-all;width:30%;">
       <span><%= users('admin').map(&:login).join(', ') %></span>
-      (<%= link_to "select", {:action => 'edit_users', :role => 'admin', :redirect => 'global'}, :class => 'action' %>)
+      (<%= link_to "select", {:action => 'edit_users', :role => 'admin', :redirect => 'global'}, :class => 'link-action' %>)
     </td>
     <td valign="top" style="word-break:break-all;width:30%;">
       <span><%= groups('admin').map{|g| group_name(g)}.join(', ') %></span>
-      (<%= link_to "select", {:action => 'edit_groups', :role => 'admin', :redirect => 'global'}, :class => 'action' %>)
+      (<%= link_to "select", {:action => 'edit_groups', :role => 'admin', :redirect => 'global'}, :class => 'link-action' %>)
     </td>
   </tr>
   </tbody>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
index 05066f0b443fb5bc2f27c82270beeeedeb85e998..3331b8dadca87af2ab7336808e812760f6c35a7f 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
@@ -14,34 +14,34 @@
       <td valign="top" style="word-break:break-all;width:35%;">
       <div style="vertical-align:top">
         <span><%= users('default-admin').map(&:login).join(', ') %></span>
-        (<%= link_to "select", {:action => 'edit_users', :role => 'default-admin', :redirect => 'projects'}, :class => 'action' %>)
+        (<%= link_to "select", {:action => 'edit_users', :role => 'default-admin', :redirect => 'projects'}, :class => 'link-action' %>)
         </div>
       </td>
       <td valign="top" style="word-break:break-all;width:35%;">
         <span><%= groups('default-admin').map{|g| group_name(g)}.join(', ') %></span>
-      (<%= link_to "select", {:action => 'edit_groups', :role => 'default-admin', :redirect => 'projects'}, :class => 'action' %>)
+      (<%= link_to "select", {:action => 'edit_groups', :role => 'default-admin', :redirect => 'projects'}, :class => 'link-action' %>)
       </td>
     </tr>
     <tr class="odd" >
       <td valign="top"><b>Users</b><br/><span class="small gray">Ability to navigate through every service of a project, except viewing source code and settings.</span></td>
       <td valign="top" style="word-break:break-all;width:35%;">
         <span><%= users('default-user').map(&:login).join(', ') %></span>
-        (<%= link_to "select", {:action => 'edit_users', :role => 'default-user', :redirect => 'projects'}, :class => 'action' %>)
+        (<%= link_to "select", {:action => 'edit_users', :role => 'default-user', :redirect => 'projects'}, :class => 'link-action' %>)
       </td>
       <td valign="top" style="word-break:break-all;width:35%;">
         <span><%= groups('default-user').map{|g| group_name(g)}.join(', ') %></span>
-        (<%= link_to "select", {:action => 'edit_groups', :role => 'default-user', :redirect => 'projects'}, :class => 'action' %>)
+        (<%= link_to "select", {:action => 'edit_groups', :role => 'default-user', :redirect => 'projects'}, :class => 'link-action' %>)
       </td>
     </tr>
     <tr class="even" >
       <td valign="top"><b>Code viewers</b><br/><span class="small gray">Ability to view source code of a project.</span></td>
       <td valign="top" style="word-break:break-all;width:35%;">
         <span><%= users('default-codeviewer').map(&:login).join(', ') %></span>
-        (<%= link_to "select", {:action => 'edit_users', :role => 'default-codeviewer', :redirect => 'projects'}, :class => 'action' %>)
+        (<%= link_to "select", {:action => 'edit_users', :role => 'default-codeviewer', :redirect => 'projects'}, :class => 'link-action' %>)
       </td>
       <td valign="top" style="word-break:break-all;width:35%;">
         <span><%= groups('default-codeviewer').map{|g| group_name(g)}.join(', ') %></span>
-      (<%= link_to "select", {:action => 'edit_groups', :role => 'default-codeviewer', :redirect => 'projects'}, :class => 'action' %>)
+      (<%= link_to "select", {:action => 'edit_groups', :role => 'default-codeviewer', :redirect => 'projects'}, :class => 'link-action' %>)
       </td>
     </tr>
   </tbody>
@@ -73,24 +73,24 @@
             users=project.user_roles.select{|ur| ur.role=='admin'}.map{|ur| ur.user.login}
             groups=project.group_roles.select{|gr| gr.role=='admin'}.map{|gr| group_name(gr.group) }
           %>
-          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'admin', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)<br/>
-          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'admin', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)
+          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'admin', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)<br/>
+          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'admin', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)
         </td>
         <td valign="top">
           <%
             users=project.user_roles.select{|ur| ur.role=='user'}.map{|ur| ur.user.login}
             groups=project.group_roles.select{|gr| gr.role=='user'}.map{|gr| group_name(gr.group) }
           %>
-          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'user', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)<br/>
-          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'user', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)
+          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'user', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)<br/>
+          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'user', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)
         </td>
         <td valign="top">
           <%
             users=project.user_roles.select{|ur| ur.role=='codeviewer'}.map{|ur| ur.user.login}
             groups=project.group_roles.select{|gr| gr.role=='codeviewer'}.map{|gr| group_name(gr.group) }
           %>
-          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'codeviewer', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)<br/>
-          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'codeviewer', :resource => project.id, :redirect => 'projects'}, :class => 'action' %>)
+          <%= users.join(', ') %> (<%= link_to "select users", {:action => 'edit_users', :role => 'codeviewer', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)<br/>
+          <%= groups.join(', ') %> (<%= link_to "select groups", {:action => 'edit_groups', :role => 'codeviewer', :resource => project.id, :redirect => 'projects'}, :class => 'link-action' %>)
         </td>
       </tr>
     <% end

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/users/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/users/index.html.erb
index 18f32f3636506d5111ab7ab0e74f39b9a673b14c..81eb2de4f9053e44cadccef34857bcade7e6a0e8 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/users/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/users/index.html.erb
@@ -20,12 +20,14 @@
               <td class="left" valign="top"><%=user.name %></td>
               <td class="left" valign="top"><%=user.email %></td>
               <td class="left" valign="top">
-                <%= user.groups.sort.map(&:name).join(', ') %> (<%= link_to "select", {:action => 'select_group', :id => user.id}, :id => "select-#{u user.login}" %>)
+                <%= user.groups.sort.map(&:name).join(', ') %> (<%= link_to "select", {:action => 'select_group', :id => user.id}, {:id => "select-#{u user.login}", :class => 'link-action'} %>)
               </td>
               <td class="left" valign="top">
-                <%= link_to "edit", { :id => user.id, :action => 'edit'}, :id => "edit-#{u user.login}" %> |
-                <%= link_to "change password", { :id => user.id, :action => 'change_password'}, :id => "change-password-#{u user.login}" %> |
-                <%= link_to "delete", {:action => 'destroy', :id => user.id}, {:confirm => "Warning : are you sure to delete this user ?", :method => 'delete', :id => "delete-#{u user.login}"} %>
+                <%= link_to "edit", { :id => user.id, :action => 'edit'}, {:id => "edit-#{u user.login}", :class => 'link-action'} %>
+                
+                <%= link_to "change password", { :id => user.id, :action => 'change_password'}, {:id => "change-password-#{u user.login}", :class => 'link-action'} %>
+                
+                <%= link_to "delete", {:action => 'destroy', :id => user.id}, {:confirm => "Warning : are you sure to delete this user ?", :method => 'delete', :id => "delete-#{u user.login}", :class => 'link-action'} %>
               </td>
             </tr>
           <% end %>

diff --git a/sonar-server/src/main/webapp/stylesheets/style.css b/sonar-server/src/main/webapp/stylesheets/style.css
index 24104be5c8e1448f45c945dead78dcb4f319c507..bbcf71f9c097eccdba3095c940291d9d1cb8a849 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/style.css
+++ b/sonar-server/src/main/webapp/stylesheets/style.css
@@ -2363,7 +2363,8 @@ table.nowrap td, td.nowrap, th.nowrap {
 
 
 
-
+/* Used on links which are located inside a dense text place or in tables */
+/* in order to rapidly identify them                                      */
 .link-action {
   text-decoration: underline;
   color: #4183C4;