Merge branch 'master' of github.com:gitlabhq/gitlabhq

app/assets/javascripts/main.js.coffee

@@ -33,6 +33,11 @@ $ ->
   # Bottom tooltip
   $('.has_bottom_tooltip').tooltip(placement: 'bottom')
 
+  # Flash
+  if (flash = $("#flash-container")).length > 0
+    flash.click -> $(@).slideUp("slow")
+    flash.slideDown "slow"
+    setTimeout (-> flash.slideUp("slow")), 3000
 
   # Disable form buttons while a form is submitting
   $('body').on 'ajax:complete, ajax:beforeSend, submit', 'form', (e) ->

app/assets/stylesheets/common.scss

@@ -68,7 +68,7 @@ table a code {
 }
 
 /** FLASH message **/
-#flash_container {
+#flash-container {
   height: 50px;
   position: fixed;
   z-index: 10001;
@@ -79,6 +79,8 @@ table a code {
   background: white;
   cursor: pointer;
   border-bottom: 1px solid #ccc;
+  text-align: center;
+  display: none;
 
   h4 {
     color: #666;

app/models/note.rb

@@ -32,11 +32,10 @@ class Note < ActiveRecord::Base
   delegate :name, to: :project, prefix: true
   delegate :name, :email, to: :author, prefix: true
 
-  validates :project, presence: true
+  validates :note, :project, presence: true
-  validates :note, presence: true, length: { within: 0..5000 }
   validates :attachment, file_size: { maximum: 10.megabytes.to_i }
 
-  mount_uploader  :attachment, AttachmentUploader
+  mount_uploader :attachment, AttachmentUploader
 
   # Scopes
   scope :common, ->{ where(noteable_id: nil) }

app/views/commits/_diffs.html.haml

@@ -43,7 +43,7 @@
         - if file.text?
           = render "commits/text_file", diff: diff, index: i
         - elsif file.image?
-          - old_file = (@commit.prev_commit.tree / diff.old_path)
+          - old_file = (@commit.prev_commit.tree / diff.old_path) if !@commit.prev_commit.nil?
           - if diff.renamed_file || diff.new_file || diff.deleted_file
             .diff_file_content_image
               .image{class: image_diff_class(diff)}

app/views/layouts/_flash.html.haml

@@ -1,17 +1,3 @@
-- if alert || notice
+- if text = alert || notice
-  - text = alert || notice
+  #flash-container
-  %div{style: "display:none", id: "flash_container"}
+    %h4= text
-    %center
-      %h4= text
-  :javascript
-    $(function(){
-      $("#flash_container").slideDown("slow");
-      $("#flash_container").click(function(){
-        $(this).slideUp("slow");
-      });
-      setTimeout("hideFlash()",3000);
-    });
-
-    function hideFlash(){
-      $("#flash_container").slideUp("slow");
-    }

app/views/layouts/devise.html.haml

@@ -2,5 +2,5 @@
 %html{ lang: "en"}
   = render "layouts/head"
   %body.ui_basic.login-page
-    = render partial: "layouts/flash"
+    = render "layouts/flash"
     .container= yield