Commit fbd345ea20eaabc58fe84a2cdf3ba960f3a10c24
1 parent
c4a7824a
Exists in
master
and in
4 other branches
Make attachments more obvious
Showing
2 changed files
with
20 additions
and
7 deletions
Show diff stats
app/assets/stylesheets/sections/notes.scss
| @@ -76,10 +76,22 @@ ul.notes { | @@ -76,10 +76,22 @@ ul.notes { | ||
| 76 | p { color: $style_color; } | 76 | p { color: $style_color; } |
| 77 | 77 | ||
| 78 | .avatar { | 78 | .avatar { |
| 79 | - margin-top:3px; | 79 | + margin-top: 3px; |
| 80 | + } | ||
| 81 | + .attachment { | ||
| 82 | + font-size: 16px; | ||
| 83 | + margin-top: -20px; | ||
| 84 | + | ||
| 85 | + .icon-attachment { | ||
| 86 | + @extend .icon-paper-clip; | ||
| 87 | + font-size: 24px; | ||
| 88 | + position: relative; | ||
| 89 | + text-align: right; | ||
| 90 | + top: 6px; | ||
| 91 | + } | ||
| 80 | } | 92 | } |
| 81 | .note-body { | 93 | .note-body { |
| 82 | - margin-left:45px; | 94 | + margin-left: 45px; |
| 83 | padding-top: 5px; | 95 | padding-top: 5px; |
| 84 | } | 96 | } |
| 85 | .note-header { | 97 | .note-header { |
| @@ -187,7 +199,7 @@ p.notify_controls span{ | @@ -187,7 +199,7 @@ p.notify_controls span{ | ||
| 187 | // TODO: end cleaup | 199 | // TODO: end cleaup |
| 188 | 200 | ||
| 189 | /** | 201 | /** |
| 190 | - * add line note button on the side of diffs | 202 | + * line note button on the side of diffs |
| 191 | */ | 203 | */ |
| 192 | .diff_file tr.line_holder { | 204 | .diff_file tr.line_holder { |
| 193 | .add-diff-note { | 205 | .add-diff-note { |
app/views/notes/_note.html.haml
| @@ -27,8 +27,9 @@ | @@ -27,8 +27,9 @@ | ||
| 27 | .note-body | 27 | .note-body |
| 28 | = preserve do | 28 | = preserve do |
| 29 | = markdown(note.note) | 29 | = markdown(note.note) |
| 30 | - - if note.attachment.url | ||
| 31 | - .right | ||
| 32 | - %div.file | ||
| 33 | - = link_to note.attachment_identifier, note.attachment.url, target: "_blank" | 30 | + - if note.attachment.url |
| 31 | + .attachment.right | ||
| 32 | + = link_to note.attachment.url, target: "_blank" do | ||
| 33 | + %i.icon-attachment | ||
| 34 | + = note.attachment_identifier | ||
| 34 | .clear | 35 | .clear |