Restyled error messages in common way. Added title to head partial

c873cf81 · Dmitriy Zaporozhets · d8f825ef · c873cf81 · c873cf81 · c873cf81
Commit c873cf81 authored Nov 28, 2012 by Dmitriy Zaporozhets
17 changed files
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -666,3 +666,10 @@ pre {
    float:left;
  }
 }
+h1.http_status_code {
+  font-size: 56px;
+  line-height: 100px;
+  font-weight: normal;
+  color: #456;
+}
--- a/app/views/errors/access_denied.html.haml
+++ b/app/views/errors/access_denied.html.haml
-%h1 Access Denied
+%h1 403
+%h3.page_title Access Denied
 %hr
-%h2 You are not allowed to access this page.
+%p You are not allowed to access this page.
 %p Read more about project permissions #{link_to "here", help_permissions_path, class: "vlink"}
--- a/app/views/errors/encoding.html.haml
+++ b/app/views/errors/encoding.html.haml
-%h1 Encoding Error
+%h1.http_status_code 500
+%h3.page_title Encoding Error
 %hr
 %p Page can't be loaded because of an encoding error.
--- a/app/views/errors/git_not_found.html.haml
+++ b/app/views/errors/git_not_found.html.haml
-%h1 404
+%h1.http_status_code 404
+%h3.page_title Git Resource Not found
 %hr
-%h2 Git Resource Not found
 %p
  Application can't get access to some branch or commit in your repository. It
  may have been moved.
--- a/app/views/errors/gitolite.html.haml
+++ b/app/views/errors/gitolite.html.haml
-%h1 Git Error
+%h1.http_status_code 500
+%h3.page_title GitLab was unable to access your Gitolite system.
 %hr
-%h2 GitLab was unable to access your Gitolite system.
 .git_error_tips
  %h4 Tips for Administrator:

--- a/app/views/errors/not_found.html.haml
+++ b/app/views/errors/not_found.html.haml
-%h1 404
+%h1.http_status_code 404
+%h3.page_title The resource you were looking for doesn't exist.
 %hr
-%h2 The resource you were looking for doesn't exist.
 %p You may have mistyped the address or the page may have moved.
--- a/app/views/layouts/_head.html.haml
+++ b/app/views/layouts/_head.html.haml
@@ -2,7 +2,7 @@
  %meta{charset: "utf-8"}
  %title
    GitLab
-    = " > #{@project.name}" if @project && !@project.new_record?
+    = " > #{title}" if defined?(title)
  = favicon_link_tag 'favicon.ico'
  = stylesheet_link_tag    "application"
  = javascript_include_tag "application"

--- a/app/views/layouts/admin.html.haml
+++ b/app/views/layouts/admin.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: "Admin area"
  %body{class: "#{app_theme} admin"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: "Admin area"

--- a/app/views/layouts/application.html.haml
+++ b/app/views/layouts/application.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: "Dashboard"
  %body{class: "#{app_theme} application"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: "Dashboard"

--- a/app/views/layouts/errors.html.haml
+++ b/app/views/layouts/errors.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: "Error"
  %body{class: "#{app_theme} application"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: ""

--- a/app/views/layouts/group.html.haml
+++ b/app/views/layouts/group.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: "#{@group.name}"
  %body{class: "#{app_theme} application"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: "#{@group.name}"

--- a/app/views/layouts/profile.html.haml
+++ b/app/views/layouts/profile.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: "Profile"
  %body{class: "#{app_theme} profile"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: "Profile"

--- a/app/views/layouts/project_resource.html.haml
+++ b/app/views/layouts/project_resource.html.haml
 !!! 5
 %html{ lang: "en"}
-  = render "layouts/head"
+  = render "layouts/head", title: @project.name
  %body{class: "#{app_theme} project"}
    = render "layouts/flash"
    = render "layouts/head_panel", title: @project.name

--- a/public/404.html
+++ b/public/404.html
@@ -7,9 +7,8 @@
 <body>
  <h1>404</h1>
-  <div>
+  <h3>The page you were looking for doesn't exist.</h3>
-    <h2>The page you were looking for doesn't exist.</h2>
+  <hr/>
-    <p>You may have mistyped the address or the page may have moved.</p>
+  <p>You may have mistyped the address or the page may have moved.</p>
-  </div>
 </body>
 </html>
--- a/public/500.html
+++ b/public/500.html
@@ -4,13 +4,10 @@
  <title>We're sorry, but something went wrong (500)</title>
  <link href="/static.css" media="screen" rel="stylesheet" type="text/css" />
 </head>
 <body>
-  <!-- This file lives in public/500.html -->
  <h1>500</h1>
-  <div>
+  <h3>We're sorry, but something went wrong.</h3>
-    <h2>We're sorry, but something went wrong.</h2>
+  <hr/>
-    <p>We've been notified about this issue and we'll take a look at it shortly.</p>
+  <p>We've been notified about this issue and we'll take a look at it shortly.</p>
-  </div>
 </body>
 </html>
--- a/public/githost_error.html
+++ b/public/githost_error.html
-<!DOCTYPE html>
-<html>
-<head>
-  <title>We're sorry, but we can't get access to your gitolite</title>
-  <style type="text/css">
-    body { background-color: #EAEAEA; color: #666; text-align: center; font-family: arial, sans-serif; }
-    div.dialog {
-      width: 600px;
-      padding: 0 4em;
-      margin: 4em auto 0 auto;
-    }
-    h1 { font-size: 48px; color: #444; line-height: 1.5em; }
-    h2 { font-size: 24px; color: #666; line-height: 1.5em; }
-    h3, code { text-align:left; }
-    code pre { margin-left:40px; }
-  </style>
-</head>
-<body>
-  <!-- This file lives in public/500.html -->
-  <div class="dialog">
-    <h1>Gitolite Error</h1>
-    <h2>Application can't get access to your gitolite system.</h2>
-    <hr>
-    <h3> 1. Check 'config/gitlab.yml' for correct settings.</h3>
-    <h3> 2. Make sure web server user has access to gitolite. <a href="https://github.com/gitlabhq/gitlabhq/wiki/Gitolite">Setup tutorial</a></h3>
-    <h3> 3. Try: </h3>
-    <code>
-        <pre>
-sudo chmod -R 770 /home/git/repositories/
-sudo chown -R git:git /home/git/repositories/
-        </pre>
-    </code>
-  </div>
-</body>
-</html>
--- a/public/static.css
+++ b/public/static.css
-body {  color: #666; text-align: center; font-family: arial, sans-serif; margin:0; padding:0; }
+body {
-h1 { font-size: 48px; color: #444; line-height: 1.5em; }
+  color: #666;
+  text-align: center;
+  font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;
+  sans-serif;
+  margin:0;
+  width: 800px;
+  margin: auto;
+  font-size: 14px;
+}
+h1 {
+  font-size: 56px;
+  line-height: 100px;
+  font-weight: normal;
+  color: #456;
+}
 h2 { font-size: 24px; color: #666; line-height: 1.5em; }
-.alert-message {
+h3 {
-  position: relative;
+  color: #456;
-  padding: 7px 15px;
+  font-size: 20px;
-  margin-bottom: 18px;
+  font-weight: normal;
-  color: #404040;
-  background-color: #eedc94;
-  background-repeat: repeat-x;
-  background-image: -khtml-gradient(linear, left top, left bottom, from(#fceec1), to(#eedc94));
-  background-image: -moz-linear-gradient(top, #fceec1, #eedc94);
-  background-image: -ms-linear-gradient(top, #fceec1, #eedc94);
-  background-image: -webkit-gradient(linear, left top, left bottom, color-stop(0%, #fceec1), color-stop(100%, #eedc94));
-  background-image: -webkit-linear-gradient(top, #fceec1, #eedc94);
-  background-image: -o-linear-gradient(top, #fceec1, #eedc94);
-  background-image: linear-gradient(top, #fceec1, #eedc94);
-  filter: progid:DXImageTransform.Microsoft.gradient(startColorstr='#FFFCEEC1', endColorstr='#FFEEDC94', GradientType=0);
-  text-shadow: 0 -1px 0 rgba(0, 0, 0, 0.25);
-  border-color: #eedc94 #eedc94 #e4c652;
-  border-color: rgba(0, 0, 0, 0.1) rgba(0, 0, 0, 0.1) fadein(rgba(0, 0, 0, 0.1), 15%);
-  text-shadow: 0 1px 0 rgba(255, 255, 255, 0.5);
-  border-width: 1px;
-  border-style: solid;
-  -webkit-box-shadow: inset 0 1px 0 rgba(255, 255, 255, 0.25);
-  -moz-box-shadow: inset 0 1px 0 rgba(255, 255, 255, 0.25);
-  box-shadow: inset 0 1px 0 rgba(255, 255, 255, 0.25);
-}
-.alert-message .close {
-  margin-top: 1px;
-  *margin-top: 0;
-}
-.alert-message a {
-  font-weight: bold;
-  color: #404040;
-}
-.alert-message.danger p a, .alert-message.error p a, .alert-message.success p a, .alert-message.info p a {
-  color: #404040;
-}
-.alert-message h5 {
-  line-height: 18px;
-}
-.alert-message p {
-  margin-bottom: 0;
-}
-.alert-message div {
-  margin-top: 5px;
-  margin-bottom: 2px;
  line-height: 28px;
 }
-.alert-message.block-message.error {
+hr {
-  background: #FDDFDE;
+  margin: 18px 0;
-  border-color: #FBC7C6;
+  border: 0;
+  border-top: 1px solid #EEE;
+  border-bottom: 1px solid white;
 }