summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2012-09-18 09:17:55 +0300
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2012-09-18 09:17:55 +0300
commit64e76a87aaddf76c3b3a487c5eb39e4490e4724f (patch)
tree89b2390333b62201133aef23cf503052192dfb08
parent2aafd7cf4e5dc28c0d4692d4a0b35a714186615b (diff)
downloadgitlab-ce-64e76a87aaddf76c3b3a487c5eb39e4490e4724f.tar.gz
Notes votes: use icons instead of borders. Removed unnecessary padding
-rw-r--r--app/assets/stylesheets/common.scss12
-rw-r--r--app/assets/stylesheets/sections/notes.scss10
-rw-r--r--app/views/notes/_note.html.haml10
3 files changed, 22 insertions, 10 deletions
diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index fda8d54cec8..ffabdf8bfb7 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -158,6 +158,18 @@ span.update-author {
padding: 6px;
}
}
+
+ &.label-success {
+ background-color: #8D8;
+ color: #333;
+ text-shadow: 0 1px 1px white;
+ }
+
+ &.label-error {
+ background-color: #D88;
+ color: #333;
+ text-shadow: 0 1px 1px white;
+ }
}
.event_label {
diff --git a/app/assets/stylesheets/sections/notes.scss b/app/assets/stylesheets/sections/notes.scss
index e14a0828fe9..267a9b4356c 100644
--- a/app/assets/stylesheets/sections/notes.scss
+++ b/app/assets/stylesheets/sections/notes.scss
@@ -73,15 +73,7 @@
/* mark vote notes */
.voting_notes .note {
- padding: 8px 0 8px 12px;
- &.upvote {
- padding-left: 8px;
- border-left: 4px solid #468847;
- }
- &.downvote {
- padding-left: 8px;
- border-left: 4px solid #B94A48;
- }
+ padding: 8px 0;
}
.notes-status {
diff --git a/app/views/notes/_note.html.haml b/app/views/notes/_note.html.haml
index 23145f128a8..5234e55dcd0 100644
--- a/app/views/notes/_note.html.haml
+++ b/app/views/notes/_note.html.haml
@@ -6,8 +6,16 @@
%cite.cgray
= time_ago_in_words(note.updated_at)
ago
+ - if note.upvote?
+ %span.label.label-success
+ %i.icon-thumbs-up
+ \+1
+ - if note.downvote?
+ %span.label.label-error
+ %i.icon-thumbs-down
+ \-1
- if(note.author_id == current_user.id) || can?(current_user, :admin_note, @project)
- = link_to [@project, note], confirm: 'Are you sure?', method: :delete, remote: true, class: "cred delete-note btn very_small" do
+ = link_to [@project, note], confirm: 'Are you sure?', method: :delete, remote: true, class: "cred delete-note btn very_small" do
%i.icon-trash
Remove