From 37acc35921c086c27be6ff72979733344ca9f958 Mon Sep 17 00:00:00 2001
From: Ilya Konanykhin <ilya.konanykhin@gmail.com>
Date: Thu, 22 Dec 2016 23:06:26 +0600
Subject: [PATCH] BS3: timesheet.

---
 app/views/time_entries/index.haml | 69 ++++++++++++++++---------------
 app/views/time_entries/new.haml   |  6 +--
 2 files changed, 39 insertions(+), 36 deletions(-)

diff --git a/app/views/time_entries/index.haml b/app/views/time_entries/index.haml
index 2587783..ed91afc 100644
--- a/app/views/time_entries/index.haml
+++ b/app/views/time_entries/index.haml
@@ -1,44 +1,47 @@
 = link_to_dashboard
 
-%h2 Your Timesheet
+%h1 Your Timesheet
 
-%dl.dl-horizontal
-  %dd
-    %span.badge #{@hours_worked}
-    Total Hours Worked
-  %dd
-    %span.badge #{@credits_available}
-    Total Credits Available
+.row
+  .col-xs-12.col-sm-6.col-lg-3
+    %p
+      %span.badge= @hours_worked
+      Total Hours Worked
+
+  .col-xs-12.col-sm-6.col-lg-3
+    %p
+      %span.badge= @credits_available
+      Total Credits Available
 
 %table.table
   %tbody
     - @user_time_entries.each do |entry|
-      %tr{ "data-id" => entry.id, "data-description" => entry.description,
-          "data-duration" => entry.duration_in_hours,
-          "data-start_date" => entry.start_date.to_date.to_formatted_s(:rfc822)}
-        %td #{entry.start_date.to_date.to_formatted_s(:rfc822)}
-        %td #{entry.duration_in_hours}
-        %td #{entry.type}
-        %td #{truncate(entry.description)}
+      -# data: {start_date: ..} gets transformed to data-start-date instead of data-start_date, we don't want this
+      %tr{data: {id: entry.id, description: entry.description, duration: entry.duration_in_hours},
+          "data-start_date": entry.start_date.to_date.to_formatted_s(:rfc822)}
+        %td= entry.start_date.to_date.to_formatted_s(:rfc822)
+        %td= entry.duration_in_hours
+        %td= entry.type
+        %td= link_to truncate(entry.description).presence || '<em>No title</em>'.html_safe, edit_task_list_path(entry)
         %td
-          %button{ class: "btn icon-remove btn-danger work_entry-delete-btn", role: "button", "data-toggle" => "modal", "data-target" => "#confirmation" }
-
-%a{class: "btn btn-lg btn-block btn-primary", href: new_time_entry_path } Add Time Entry
-
-.modal{ id: "confirmation", class: "hide", role: "dialog", "aria-labelledby" => "confirmation_title", "aria-hidden" => "true", tabindex: -1 }
-  .modal-header
-    %button{ class: "close", "data-dismiss" => "modal", "aria-hidden" => "true"}x
-    %h4{ id: "confirmation_title" } Are you sure you want to delete?
-  .modal-body
-    %div{ id: "work_entry_start_date" }
-      Start Date
-    %div{ id: "work_entry_duration" }
-      Duration
-    %div{ id: "work_entry_description" }
-      Description
-  .modal-footer
-    %button{ class: "btn", "data-dismiss" => "modal", "aria-hidden" => "true"} Close
-    %button{ id: "confirmation_delete", class: "btn btn-primary", "data-url" => "api/v1/time_entries/" } Delete
+          = link_to '#modal', class: 'text-danger work_entry-delete-btn', role: 'button', data: {toggle: 'modal', target: '#confirmation'} do
+            %i.glyphicon.glyphicon-remove
+
+= link_to 'Add Time Entry', new_time_entry_path, class: 'btn btn-primary'
+
+.modal#confirmation{role: 'dialog', "aria-labelledby": 'confirmation_title', "aria-hidden": 'true', tabindex: -1}
+  .modal-dialog
+    .modal-content
+      .modal-header
+        %button.close{"data-dismiss": 'modal', "aria-hidden": 'true'} &times;
+        %h4#confirmation_title Are you sure you want to delete?
+      .modal-body
+        #work_entry_start_date Start Date
+        #work_entry_duration Duration
+        #work_entry_description Description
+      .modal-footer
+        %button.btn.btn-default{"data-dismiss": 'modal', "aria-hidden": 'true'} Cancel
+        %button.btn.btn-danger#confirmation_delete{"data-url": "api/v1/time_entries/"} Delete
 
 
 
diff --git a/app/views/time_entries/new.haml b/app/views/time_entries/new.haml
index 4bf9623..bfe6350 100644
--- a/app/views/time_entries/new.haml
+++ b/app/views/time_entries/new.haml
@@ -27,13 +27,13 @@
 
       .form-group
         .btn-group{"data-toggle": 'buttons'}
-          %label{class: 'btn btn-default'}
+          %label.btn.btn-default
             = radio_button_tag 'action_id', 1, false, autocomplete: 'off'
             Volunteer
-          %label{class: 'btn btn-default'}
+          %label.btn.btn-default
             = radio_button_tag 'action_id', 2, false, autocomplete: 'off'
             Personal
-          %label{class: 'btn btn-default'}
+          %label.btn.btn-default
             = radio_button_tag 'action_id', 3, false, autocomplete: 'off'
             Staff
         .hidden#log_action_id