hr_lite 0.53.0 → 0.55.0

data/app/views/hr_lite/admin/payroll_runs/show.html.erb

@@ -2,16 +2,16 @@
 <div class="hrl-page__head">
   <h1 class="hrl-page__title">Payroll — <%= @run.label %></h1>
   <div class="hrl-page__actions">
-    <a class="hrl-btn" href="<%= hr_lite.admin_payroll_line_items_path(month: @run.period_month.strftime("%Y-%m")) %>">One-off items</a>
+    <% if hr_can?("payroll.manage", scope: :all) %><a class="hrl-btn" href="<%= hr_lite.admin_payroll_line_items_path(month: @run.period_month.strftime("%Y-%m")) %>">One-off items</a><% end %>
     <a class="hrl-btn" href="<%= hr_lite.admin_payroll_runs_path %>">All runs</a>
   </div>
 </div>
 
-<div class="hrl-kpis">
+<div class="hrl-kpis hrl-kpis--money">
   <div class="hrl-kpi"><div class="hrl-kpi__value"><%= @slips.size %></div><div class="hrl-kpi__label">Slips</div></div>
   <div class="hrl-kpi"><div class="hrl-kpi__value"><%= hrl_money(@run.total_gross) %></div><div class="hrl-kpi__label">Gross</div></div>
   <div class="hrl-kpi"><div class="hrl-kpi__value"><%= hrl_money(@run.total_net) %></div><div class="hrl-kpi__label">Net payout</div></div>
-  <div class="hrl-kpi"><div class="hrl-kpi__value"><%= hrl_money(@run.total_employer_cost) %></div><div class="hrl-kpi__label">Employer cost</div></div>
+  <div class="hrl-kpi"><div class="hrl-kpi__value"><%= hrl_money(@run.total_employer_cost) %></div><div class="hrl-kpi__label">Employer contributions</div></div>
 </div>
 
 <% if @run.warnings.any? %>
@@ -29,12 +29,14 @@
     <% manager = hr_can?("payroll.manage", scope: :all) %>
     <% if manager && (@run.draft? || @run.review? || @run.processing?) %>
       <%= button_to @run.draft? ? "Compute" : "Recompute", hr_lite.compute_admin_payroll_run_path(@run),
-                    method: :post, class: "hrl-btn" %>
+                    method: :post, class: @run.draft? ? "hrl-btn hrl-btn--primary" : "hrl-btn",
+                    form: { data: { turbo_confirm: ("Recompute #{@run.label}? This clears #{hr_display_name(@run.first_approved_by)}'s approval." if @run.first_approved_by) } } %>
     <% end %>
     <% if @run.review? && hr_can?("payroll.approve", scope: :all) && @run.first_approved_by_id != hr_current_user.id %>
       <%= button_to @run.first_approved_by_id ? "Approve and publish" : "Approve", hr_lite.approve_admin_payroll_run_path(@run),
                     method: :post, class: "hrl-btn hrl-btn--primary",
-                    form: { data: { turbo_confirm: @run.first_approved_by_id ? "Approve #{@run.label}? This finalizes the run and publishes the slips." : "Approve #{@run.label}?" } } %>
+                    form: { data: { turbo_confirm: [ @run.first_approved_by_id ? "Approve #{@run.label}? This finalizes the run and publishes the slips." : "Approve #{@run.label}?",
+                                                     *("Blocking warnings: #{@run.blocking_warnings.join('; ')}" if @run.blocking_warnings.any?) ].join("\n\n") } } %>
     <% end %>
     <% if manager && @run.finalized? %>
       <%= button_to "Unlock", hr_lite.unlock_admin_payroll_run_path(@run), method: :post, class: "hrl-btn" %>
@@ -54,6 +56,9 @@
   <p class="hrl-small hrl-muted hrl-mt">
     <% if @run.review? %>
       <%= @run.first_approved_by ? "Approved by #{hr_display_name(@run.first_approved_by)} on #{@run.first_approved_at.strftime('%-d %b')} — one more approver (HR or Super Admin) publishes it." : "Needs two approvals (HR and Super Admin). If nobody approves by the 3rd, a run with no blocking warnings is approved automatically." %>
+    <% elsif @run.processing? %>
+      <% content_for(:head) { tag.meta("http-equiv": "refresh", content: "3") } %>
+      Computing… this page refreshes until the slips are ready.
     <% elsif @run.auto_approved_at %>
       Approved automatically on <%= @run.auto_approved_at.strftime("%-d %b") %> — nobody had approved it by the 3rd.
     <% end %>
@@ -64,20 +69,20 @@
   <% if @slips.any? %>
     <div class="hrl-table-wrap">
       <table class="hrl-table hrl-table--stack">
-        <thead><tr><th>Employee</th><th>Days</th><th>LOP</th><th>Gross</th><th>Deductions</th><th>Net</th><th></th></tr></thead>
+        <thead><tr><th>Employee</th><th>Paid days</th><th><abbr title="Loss of pay: unpaid days">LOP</abbr></th><th>Gross</th><th>Deductions</th><th>Net</th><th></th></tr></thead>
         <tbody>
           <% @slips.each do |slip| %>
             <tr>
               <td data-label="Employee"><%= hr_display_name(slip.user) %></td>
-              <td data-label="Days" class="hrl-num"><%= slip.payable_days %></td>
-              <td data-label="LOP" class="hrl-num">
-                <%= slip.effective_lop_days %>
+              <td data-label="Paid days" class="hrl-num"><%= hrl_days(slip.payable_days) %></td>
+              <td data-label="Unpaid (LOP)" class="hrl-num">
+                <%= hrl_days(slip.effective_lop_days) %>
                 <% if slip.lop_override %><span class="hrl-badge hrl-badge--warn">override</span><% end %>
               </td>
               <td data-label="Gross" class="hrl-num"><%= hrl_money(slip.gross_earnings) %></td>
               <td data-label="Deductions" class="hrl-num"><%= hrl_money(slip.total_deductions) %></td>
               <td data-label="Net" class="hrl-num"><strong><%= hrl_money(slip.net_pay) %></strong></td>
-              <td data-label=""><a class="hrl-small" href="<%= hr_lite.admin_salary_slip_path(slip) %>">Detail</a></td>
+              <td data-label=""><a class="hrl-btn" href="<%= hr_lite.admin_salary_slip_path(slip) %>">Detail</a></td>
             </tr>
           <% end %>
         </tbody>

data/app/views/hr_lite/admin/performance_plans/_form.html.erb

@@ -22,7 +22,8 @@
       <div class="hrl-field"><%= f.label :outcome_note, "Outcome note" %><%= f.text_area :outcome_note, rows: 3 %></div>
     <% end %>
     <div class="hrl-form-actions">
-      <%= f.submit plan.persisted? ? "Save" : "Start PIP", class: "hrl-btn hrl-btn--primary" %>
+      <%= f.submit plan.persisted? ? "Save" : "Start PIP", class: "hrl-btn hrl-btn--primary",
+                   data: { turbo_confirm: ("Save? Passed or failed closes the plan for good and tells the employee." if plan.persisted?) } %>
       <a class="hrl-btn" href="<%= hr_lite.admin_performance_plans_path %>">Cancel</a>
     </div>
   <% end %>

data/app/views/hr_lite/admin/policies/index.html.erb

@@ -14,10 +14,10 @@
         <tbody>
           <% @policies.each do |policy| %>
             <tr>
-              <td data-label="Title"><%= policy.title %> <span class="hrl-badge hrl-badge--muted">v<%= policy.version %></span></td>
+              <td data-label="Title"><a class="hrl-card__link" href="<%= hr_lite.policy_path(policy) %>"><%= policy.title %></a> <span class="hrl-badge hrl-badge--muted">v<%= policy.version %></span></td>
               <td data-label="Kind"><%= policy.announcement? ? "Announcement" : "Policy" %></td>
               <td data-label="In force"><%= policy.effective_from.strftime("%d %b %Y") %></td>
-              <td data-label="Still to acknowledge"><%= policy.announcement? ? "—" : policy.outstanding_for.count %></td>
+              <td data-label="Still to acknowledge"><% if policy.announcement? %>—<% else %><% waiting = policy.outstanding_for.to_a %><details><summary class="hrl-btn"><%= pluralize(waiting.size, "person") %></summary><%= waiting.map { |u| hr_display_name(u) }.to_sentence %></details><% end %></td>
             </tr>
           <% end %>
         </tbody>

data/app/views/hr_lite/admin/policies/new.html.erb

@@ -8,14 +8,15 @@
   <%= form_with model: @policy, url: hr_lite.admin_policies_path, method: :post, local: true do |f| %>
     <%= render "hr_lite/shared/form_errors", record: @policy %>
     <div class="hrl-field"><%= f.label :title %><%= f.text_field :title, required: true %></div>
-    <div class="hrl-field"><%= f.label :body, "Text" %><%= f.text_area :body, rows: 10, required: true %></div>
-    <div class="hrl-field"><%= f.label :effective_from, "In force from" %><%= f.date_field :effective_from %></div>
     <div class="hrl-field hrl-field--check">
       <%= f.check_box :acknowledgement_required %>
       <%= f.label :acknowledgement_required, "Policy — everyone must acknowledge it (untick for an announcement)" %>
     </div>
+    <div class="hrl-field"><%= f.label :body, "Text" %><%= f.text_area :body, rows: 10, required: true %></div>
+    <div class="hrl-field"><%= f.label :effective_from, "In force from" %><%= f.date_field :effective_from %></div>
     <div class="hrl-form-actions">
-      <%= f.submit "Publish", class: "hrl-btn hrl-btn--primary" %>
+      <%= f.submit "Publish", class: "hrl-btn hrl-btn--primary",
+                   data: { turbo_confirm: "Publish and notify #{pluralize(HrLite.active_employees.size, 'person')} by email?" } %>
       <a class="hrl-btn" href="<%= hr_lite.admin_policies_path %>">Cancel</a>
     </div>
   <% end %>

data/app/views/hr_lite/admin/regularization_requests/index.html.erb

@@ -1,13 +1,12 @@
-<% content_for(:page_title) { "Regularization tickets" } %>
+<% content_for(:page_title) { "Day fixes" } %>
 <div class="hrl-page__head">
-  <h1 class="hrl-page__title">Regularization tickets</h1>
+  <h1 class="hrl-page__title">Day fixes</h1>
 </div>
 
 <div class="hrl-row hrl-row--mb">
   <span class="hrl-small hrl-muted">Status</span>
   <% HrLite::RegularizationRequest::STATUSES.each do |status| %>
-    <a class="hrl-btn <%= 'hrl-btn--primary' if @status == status %>"
-       href="<%= hr_lite.admin_regularization_requests_path(status: status) %>"><%= status.humanize %></a>
+    <%= link_to status.humanize, hr_lite.admin_regularization_requests_path(status: status), class: "hrl-btn", aria: { current: ("true" if @status == status) } %>
   <% end %>
 </div>
 

data/app/views/hr_lite/admin/regularization_requests/show.html.erb

@@ -1,10 +1,7 @@
 <% content_for(:page_title) { "Ticket — #{hr_display_name(@request.user)}" } %>
+<a class="hrl-backlink" href="<%= hr_lite.admin_regularization_requests_path %>"><span aria-hidden="true">‹</span> Approvals</a>
 <div class="hrl-page__head">
   <h1 class="hrl-page__title">Regularization — <%= hr_display_name(@request.user) %></h1>
-  <div class="hrl-page__actions">
-    <a class="hrl-btn" href="<%= hr_lite.admin_regularization_requests_path %>">All tickets</a>
-    <a class="hrl-btn" href="<%= hr_lite.admin_attendance_path(@request.user_id, month: @request.date.strftime("%Y-%m")) %>">Their month</a>
-  </div>
 </div>
 
 <section class="hrl-card">
@@ -28,6 +25,7 @@
         <% if @request.decision_note.present? %> · <%= @request.decision_note %><% end %></dd>
     <% end %>
   </dl>
+  <a class="hrl-card__link" href="<%= hr_lite.admin_attendance_path(@request.user_id, month: @request.date.strftime("%Y-%m")) %>">Their month</a>
 </section>
 
 <% if @request.pending? %>
@@ -38,7 +36,8 @@
     <%# Approve and reject are two separate forms on purpose. Rails binds the
         CSRF token to the form's OWN action, so one form retargeted with
         `formaction:` can never pass verification on the second endpoint. %>
-    <%= form_with url: hr_lite.approve_admin_regularization_request_path(@request), method: :post, local: true do |f| %>
+    <%= form_with url: hr_lite.approve_admin_regularization_request_path(@request), method: :post, local: true,
+                  html: { data: { turbo_confirm: "Write the proposed times onto #{hr_display_name(@request.user)}'s attendance for #{@request.date.strftime('%d %b %Y')}?" } } do |f| %>
       <div class="hrl-field">
         <%= f.label :decision_note, "Note (optional)", for: "approve_note" %>
         <%= f.text_area :decision_note, id: "approve_note" %>
@@ -54,7 +53,7 @@
     <%= form_with url: hr_lite.reject_admin_regularization_request_path(@request), method: :post, local: true do |f| %>
       <div class="hrl-field">
         <%= f.label :decision_note, "Reason (required)", for: "reject_note" %>
-        <%= f.text_area :decision_note, id: "reject_note" %>
+        <%= f.text_area :decision_note, id: "reject_note", required: true %>
       </div>
       <div class="hrl-form-actions">
         <%= f.button "Reject", type: :submit, class: "hrl-btn hrl-btn--danger" %>

data/app/views/hr_lite/admin/reports/index.html.erb

@@ -9,7 +9,7 @@
   <%= form_with url: hr_lite.admin_reports_path, method: :get, local: true do %>
     <div class="hrl-field">
       <%= label_tag :month, "Month" %>
-      <%= text_field_tag :month, @month.strftime("%Y-%m"), placeholder: "YYYY-MM" %>
+      <%= month_field_tag :month, @month.strftime("%Y-%m"), max: Date.current.strftime("%Y-%m"), class: "hrl-input" %>
     </div>
     <div class="hrl-form-actions">
       <%= submit_tag "Change month", class: "hrl-btn" %>
@@ -26,11 +26,8 @@
           <tr>
             <td data-label="Report"><%= label %></td>
             <td data-label="">
-              <a class="hrl-small"
-                 href="<%= hr_lite.admin_report_path(name, month: @month.strftime("%Y-%m")) %>">Open</a>
-              &nbsp;
-              <a class="hrl-small"
-                 href="<%= hr_lite.admin_report_path(name, month: @month.strftime("%Y-%m"), format: :csv) %>">CSV</a>
+              <a class="hrl-btn" href="<%= hr_lite.admin_report_path(name, month: @month.strftime("%Y-%m")) %>">Open</a>
+              <a class="hrl-btn" href="<%= hr_lite.admin_report_path(name, month: @month.strftime("%Y-%m"), format: :csv) %>">CSV</a>
             </td>
           </tr>
         <% end %>

data/app/views/hr_lite/admin/reports/show.html.erb

@@ -1,7 +1,21 @@
 <% content_for(:page_title) { @reports_title = HrLite::Admin::ReportsController::REPORTS[@name] } %>
+<% content_for(:back) { "Reports" } %>
+<%# The screen reads like the rest of the app; the CSV keeps raw values for spreadsheets. %>
+<% heading = ->(h) { h == "LOP" ? "Unpaid days (LOP)" : h }
+   shown = lambda do |header, value|
+     if value.blank? then "—"
+     elsif header == "Amount" then hrl_money(value)
+     elsif header.in?([ "Joined", "Left", "Spent on" ]) then Date.parse(value.to_s).strftime("%d %b %Y")
+     elsif header == "Status" then value.to_s.humanize
+     elsif value.is_a?(Float) then hrl_days(value)
+     else value
+     end
+   end %>
 <div class="hrl-page__head">
   <h1 class="hrl-page__title"><%= HrLite::Admin::ReportsController::REPORTS[@name] %></h1>
   <div class="hrl-page__actions">
+    <a class="hrl-btn" href="<%= hr_lite.admin_report_path(@name, month: @month.prev_month.strftime("%Y-%m")) %>">‹ <%= @month.prev_month.strftime("%b %Y") %></a>
+    <% if @month.next_month <= Date.current %><a class="hrl-btn" href="<%= hr_lite.admin_report_path(@name, month: @month.next_month.strftime("%Y-%m")) %>"><%= @month.next_month.strftime("%b %Y") %> ›</a><% end %>
     <a class="hrl-btn"
        href="<%= hr_lite.admin_report_path(@name, month: @month.strftime("%Y-%m"), format: :csv) %>">Download CSV</a>
   </div>
@@ -15,13 +29,13 @@
     <div class="hrl-table-wrap">
       <table class="hrl-table hrl-table--stack">
         <thead>
-          <tr><% @rows.first.keys.each do |header| %><th><%= header %></th><% end %></tr>
+          <tr><% @rows.first.keys.each do |header| %><th><%= heading.(header) %></th><% end %></tr>
         </thead>
         <tbody>
           <% @rows.each do |row| %>
             <tr>
               <% row.each do |header, value| %>
-                <td data-label="<%= header %>"><%= value %></td>
+                <td data-label="<%= heading.(header) %>"><%= shown.(header, value) %></td>
               <% end %>
             </tr>
           <% end %>

data/app/views/hr_lite/admin/roles/_form.html.erb

@@ -7,8 +7,8 @@
       <% if role.system? %>
         <%= f.text_field :name, disabled: true %>
         <span class="hrl-small hrl-muted">
-          Built-in roles keep their name — the upgrade path and the seed both
-          identify them by it. The permissions below are yours to change.
+          Built-in roles keep their name, because the system finds them by it.
+          The permissions below are yours to change.
         </span>
       <% else %>
         <%= f.text_field :name %>
@@ -20,8 +20,9 @@
     </div>
 
     <% HrLite::Permissions.grouped.each do |group, keys| %>
-      <fieldset>
-        <legend><%= group %></legend>
+      <% granted = keys.count { |k| (role.grant_map[k] || "none") != "none" } %>
+      <details class="hrl-card" <%= "open" if granted.positive? %>>
+        <summary class="hrl-btn hrl-btn--block"><%= group %> · <%= granted %> of <%= keys.size %> granted</summary>
         <% keys.each do |key| %>
           <% held = role.grant_map[key] || "none" %>
           <div class="hrl-field">
@@ -36,11 +37,12 @@
                   id: "grants_#{key.tr('.', '_')}" %>
           </div>
         <% end %>
-      </fieldset>
+      </details>
     <% end %>
 
     <div class="hrl-form-actions">
-      <%= f.submit "Save", class: "hrl-btn hrl-btn--primary" %>
+      <%= f.submit "Save", class: "hrl-btn hrl-btn--primary",
+                   data: { turbo_confirm: ("Save? Everyone holding #{role.name} gets these permissions at once." if role.persisted?) } %>
       <a class="hrl-btn" href="<%= hr_lite.admin_roles_path %>">Cancel</a>
     </div>
   <% end %>

data/app/views/hr_lite/admin/roles/edit.html.erb

@@ -38,13 +38,15 @@
     </div>
   <% end %>
 
-  <%= form_with url: hr_lite.admin_role_assignments_path(@role), method: :post, local: true do %>
+  <%= form_with url: hr_lite.admin_role_assignments_path(@role), method: :post, local: true,
+                data: { turbo_confirm: "Give the #{@role.name} role to the person you picked? Their access changes at once." } do %>
     <div class="hrl-field">
       <%= label_tag :user_id, "Add somebody to this role" %>
       <% held = @role.role_assignments.map(&:user_id) %>
       <%= select_tag :user_id,
             options_for_select(HrLite.employees.reject { |u| held.include?(u.id) }
-                                     .map { |u| [ hr_display_name(u), u.id ] }) %>
+                                     .map { |u| [ hr_display_name(u), u.id ] }),
+            include_blank: "Choose…", required: true, data: { hrl_filter: "Type a name to filter" } %>
     </div>
     <div class="hrl-form-actions">
       <%= submit_tag "Add", class: "hrl-btn hrl-btn--primary" %>

data/app/views/hr_lite/admin/roles/index.html.erb

@@ -39,7 +39,7 @@
               <% end %>
             </td>
             <td data-label="">
-              <a class="hrl-small" href="<%= hr_lite.edit_admin_role_path(role) %>">Edit</a>
+              <a class="hrl-btn" href="<%= hr_lite.edit_admin_role_path(role) %>">Edit</a>
             </td>
           </tr>
         <% end %>

data/app/views/hr_lite/admin/salary_slips/show.html.erb

@@ -11,8 +11,9 @@
 <section class="hrl-card">
   <h2 class="hrl-card__title">Tax working</h2>
   <dl class="hrl-deflist">
-    <% @slip.tax_details_hash.each do |key, value| %>
-      <dt><%= key.humanize %></dt><dd><%= value %></dd>
+    <% @slip.tax_details_hash.except("high_income_review").each do |key, value| %>
+      <dt><%= key == "fy_tds_already_deducted" ? "TDS already deducted this year" : key.humanize %></dt>
+      <dd><%= key.in?(%w[regime months_remaining]) ? value.to_s.humanize : hrl_money(value) %></dd>
     <% end %>
   </dl>
   <% if @slip.tax_details_hash["high_income_review"] == "true" %>

data/app/views/hr_lite/admin/salary_structures/_form.html.erb

@@ -5,6 +5,7 @@
     <div class="hrl-field">
       <%= f.label :annual_ctc, "Annual CTC (₹ a year)" %>
       <%= f.number_field :annual_ctc, step: 1, min: 1, required: true, value: (structure.annual_ctc || breakup&.dig(:annual_ctc))&.round %>
+      <% if structure.annual_ctc %><p class="hrl-hint">= <%= hrl_money(structure.annual_ctc) %> a year</p><% end %>
       <p class="hrl-hint">The only amount you type. Saving splits it: Basic 50%, HRA 50% of Basic in a metro (40% elsewhere), employer PF, ESI and gratuity (4.81% of Basic) inside the CTC, the rest as special allowance.</p>
     </div>
     <div class="hrl-field">

data/app/views/hr_lite/admin/settings/edit.html.erb

@@ -6,11 +6,13 @@
 <section class="hrl-card">
   <%= form_with model: @setting, url: hr_lite.admin_setting_path, method: :patch, local: true do |f| %>
     <%= render "hr_lite/shared/form_errors", record: @setting %>
+    <h2 class="hrl-card__title">Working week</h2>
     <div class="hrl-field">
       <%= f.label :weekend_policy, "Weekly off policy" %>
-      <%= f.select :weekend_policy, HrLite::Setting::WEEKEND_POLICIES.map { |p| [ p.humanize, p ] } %>
+      <%= f.select :weekend_policy, { "Sunday only" => "sun_only", "Saturday and Sunday" => "sat_sun", "Sunday, 2nd and 4th Saturday" => "second_fourth_sat_sun" }.select { |_, v| v.in?(HrLite::Setting::WEEKEND_POLICIES) } %>
       <p class="hrl-hint">Drives working-day math everywhere: leave counting, attendance, payroll.</p>
     </div>
+    <h2 class="hrl-card__title hrl-mt">Breaks</h2>
     <div class="hrl-field">
       <%= f.label :lunch_break_minutes, "Lunch break limit (minutes)" %>
       <%= f.number_field :lunch_break_minutes, min: 5, max: 120 %>
@@ -20,6 +22,7 @@
       <%= f.label :short_break_minutes, "Short break limit (minutes)" %>
       <%= f.number_field :short_break_minutes, min: 1, max: 60 %>
     </div>
+    <h2 class="hrl-card__title hrl-mt">Working day</h2>
     <div class="hrl-field">
       <%= f.label :day_minutes, "Working day (minutes, break included)" %>
       <%= f.number_field :day_minutes, min: 240, max: 720, step: 15 %>
@@ -39,6 +42,7 @@
       <%= f.time_field :recheck_in_until %>
       <p class="hrl-hint">After checking out, a person can check in again once a day inside these hours.</p>
     </div>
+    <h2 class="hrl-card__title hrl-mt">Penalties and codes</h2>
     <div class="hrl-field">
       <%= f.label :penalty_leave_type_id, "Penalties come off" %>
       <%= f.collection_select :penalty_leave_type_id, HrLite::LeaveType.active, :id, :name, { include_blank: "No leave (penalties stay as time)" }, data: { hrl_filter: "Type a leave type to filter" } %>

data/app/views/hr_lite/admin/statutory_rate_cards/_form.html.erb

@@ -18,8 +18,8 @@
 
     <fieldset>
       <legend>Provident fund</legend>
-      <% { employee_rate: "Employee rate (0.12 = 12%)", employer_rate: "Employer rate",
-           eps_rate: "EPS rate", wage_ceiling: "PF wage ceiling (₹)",
+      <% { employee_rate: "Employee rate (0.12 = 12%)", employer_rate: "Employer rate (0.12 = 12%)",
+           eps_rate: "EPS rate (0.0833 = 8.33%)", wage_ceiling: "PF wage ceiling (₹)",
            eps_wage_ceiling: "EPS wage ceiling (₹)", edli_rate: "EDLI rate",
            edli_ceiling: "EDLI ceiling (₹)", admin_rate: "PF admin rate" }.each do |key, label| %>
         <div class="hrl-field">
@@ -32,7 +32,7 @@
 
     <fieldset>
       <legend>ESI</legend>
-      <% { employee_rate: "Employee rate", employer_rate: "Employer rate",
+      <% { employee_rate: "Employee rate (0.0075 = 0.75%)", employer_rate: "Employer rate (0.0325 = 3.25%)",
            gross_ceiling: "Gross ceiling (₹)" }.each do |key, label| %>
         <div class="hrl-field">
           <%= label_tag "statutory_rate_card_esi_#{key}", label %>
@@ -68,16 +68,20 @@
           the open-ended top rate.
         </p>
         <% slabs = Array(table["slabs"]) %>
-        <% (slabs + [ nil, nil ]).each_with_index do |row, index| %>
-          <div class="hrl-row">
+        <% num = ->(v) { v && number_with_precision(v, precision: 4, strip_insignificant_zeros: true) } %>
+        <div class="hrl-slabs">
+          <span class="hrl-small" aria-hidden="true">From ₹</span>
+          <span class="hrl-small" aria-hidden="true">Up to ₹</span>
+          <span class="hrl-small" aria-hidden="true">Rate (0.05 = 5%)</span>
+          <% (slabs + [ nil, nil ]).each_with_index do |row, index| %>
             <%= text_field_tag "statutory_rate_card[income_tax][#{regime}][slabs][#{index}][from]",
-                               row&.first, placeholder: "From ₹", inputmode: "decimal" %>
+                               num.(row&.first), class: "hrl-input", inputmode: "decimal", "aria-label": "Band #{index + 1} from ₹" %>
             <%= text_field_tag "statutory_rate_card[income_tax][#{regime}][slabs][#{index}][to]",
-                               row&.at(1), placeholder: "Up to ₹ (blank = top)", inputmode: "decimal" %>
+                               num.(row&.at(1)), class: "hrl-input", inputmode: "decimal", "aria-label": "Band #{index + 1} up to ₹ (blank = top)" %>
             <%= text_field_tag "statutory_rate_card[income_tax][#{regime}][slabs][#{index}][rate]",
-                               row&.last, placeholder: "Rate (0.05)", inputmode: "decimal" %>
-          </div>
-        <% end %>
+                               num.(row&.last), class: "hrl-input", inputmode: "decimal", "aria-label": "Band #{index + 1} rate" %>
+          <% end %>
+        </div>
       </fieldset>
     <% end %>
 

data/app/views/hr_lite/admin/statutory_rate_cards/index.html.erb

@@ -16,10 +16,12 @@
     <table class="hrl-table hrl-table--stack">
       <thead><tr><th>Year</th><th>PF ceiling</th><th>ESI ceiling</th><th>Verified</th><th></th></tr></thead>
       <tbody>
+        <% live = HrLite::StatutoryRateCardRecord.effective_for(Date.current.beginning_of_month) %>
         <% @cards.each do |card| %>
           <tr>
             <td data-label="Year">
               <strong>FY <%= card.financial_year %></strong>
+              <% if card == live %><span class="hrl-badge hrl-badge--ok">In force</span><% end %>
               <div class="hrl-small hrl-muted">from <%= card.effective_from.strftime("%d %b %Y") %></div>
             </td>
             <td data-label="PF ceiling" class="hrl-num"><%= HrLite::Money.format(card.to_card[:pf][:wage_ceiling]) %></td>
@@ -33,7 +35,7 @@
               <% end %>
             </td>
             <td data-label="">
-              <a class="hrl-small" href="<%= hr_lite.edit_admin_statutory_rate_card_path(card) %>">Edit</a>
+              <a class="hrl-btn" href="<%= hr_lite.edit_admin_statutory_rate_card_path(card) %>">Edit</a>
             </td>
           </tr>
         <% end %>

data/app/views/hr_lite/admin/tax_declarations/index.html.erb

@@ -26,7 +26,7 @@
               <td data-label="Regime"><%= declaration.regime.humanize %></td>
               <td data-label="Claimed" class="hrl-num"><%= hrl_money(declaration.declared_total) %></td>
               <td data-label="">
-                <a class="hrl-small" href="<%= hr_lite.admin_tax_declaration_path(declaration) %>">Check</a>
+                <a class="hrl-btn" href="<%= hr_lite.admin_tax_declaration_path(declaration) %>">Check</a>
               </td>
             </tr>
           <% end %>

data/app/views/hr_lite/admin/tax_declarations/show.html.erb

@@ -48,24 +48,28 @@
     <% end %>
     <div class="hrl-form-actions">
       <%= f.submit "Record amounts", class: "hrl-btn" %>
+      <%= f.submit "Record and verify", name: "verify", class: "hrl-btn" %>
     </div>
   </section>
 <% end %>
 
+<% if @declaration.submitted? %>
 <section class="hrl-card">
   <h2 class="hrl-card__title">Decide</h2>
   <div class="hrl-row">
+    <% unproven = @declaration.tax_declaration_items.reject { |item| item.proofs.attached? }.map { |item| item.section_label.split(" — ").first } %>
     <%= button_to "Accept all and verify", hr_lite.accept_all_admin_tax_declaration_path(@declaration),
-          method: :post, class: "hrl-btn hrl-btn--primary" %>
-    <%= button_to "Verify the amounts recorded above", hr_lite.verify_admin_tax_declaration_path(@declaration),
-          method: :post, class: "hrl-btn" %>
+          method: :post, class: "hrl-btn hrl-btn--primary",
+          form: { data: { turbo_confirm: [ "Accept every claimed amount and verify? The employee can no longer edit it.",
+                                           ("No proof attached: #{unproven.to_sentence}." if unproven.any?) ].compact.join(" ") } } %>
   </div>
   <%= form_with url: hr_lite.reject_admin_tax_declaration_path(@declaration),
                 method: :post, local: true do %>
     <div class="hrl-field">
       <%= label_tag :note, "What is missing (required to send back)" %>
-      <%= text_field_tag :note %>
+      <%= text_field_tag :note, nil, class: "hrl-input" %>
     </div>
     <%= submit_tag "Send back", class: "hrl-btn" %>
   <% end %>
 </section>
+<% end %>

data/app/views/hr_lite/admin/wfh_requests/index.html.erb

@@ -3,7 +3,22 @@
   <h1 class="hrl-page__title">Work from home</h1>
 </div>
 
-<section class="hrl-card">
+<% if @grantable.any? %>
+  <section class="hrl-card">
+    <h2 class="hrl-card__title">Give WFH</h2>
+    <p class="hrl-small hrl-muted">Approved straight away, up to <%= HrLite::WfhRequest::HR_MAX_DAYS %> days from today or later.</p>
+    <%= form_with url: hr_lite.admin_wfh_requests_path, method: :post, local: true do %>
+      <div class="hrl-field"><label for="grant-user">Employee</label><%= select_tag :user_id, options_for_select(@grantable.map { |u| [ hr_display_name(u), u.id ] }), include_blank: "Choose one", required: true, id: "grant-user", data: { hrl_filter: ("Type a name to filter" if @grantable.size > 5) } %></div>
+      <div class="hrl-field"><label for="grant-from">From</label><%= date_field_tag :start_date, Date.current, min: Date.current, required: true, id: "grant-from" %></div>
+      <div class="hrl-field"><label for="grant-to">To</label><%= date_field_tag :end_date, Date.current, min: Date.current, required: true, id: "grant-to" %></div>
+      <div class="hrl-field"><label for="grant-reason">Reason</label><%= text_area_tag :reason, nil, rows: 2, required: true, id: "grant-reason" %></div>
+      <div class="hrl-form-actions"><button type="submit" class="hrl-btn hrl-btn--primary">Give WFH</button></div>
+    <% end %>
+  </section>
+<% end %>
+
+<details class="hrl-card"<%= " open" if request.query_string.present? %>>
+  <summary class="hrl-btn hrl-btn--block">Filter</summary>
   <%= form_with url: hr_lite.admin_wfh_requests_path, method: :get, local: true do %>
     <div class="hrl-field"><label for="wfh-from">From</label><%= date_field_tag :from, @from, id: "wfh-from" %></div>
     <div class="hrl-field"><label for="wfh-to">To</label><%= date_field_tag :to, @to, id: "wfh-to" %></div>
@@ -19,7 +34,7 @@
       <a class="hrl-btn" href="<%= hr_lite.admin_wfh_requests_path %>">Reset</a>
     </div>
   <% end %>
-</section>
+</details>
 
 <div class="hrl-kpis">
   <div class="hrl-kpi"><div class="hrl-kpi__value"><%= @kpis[:requested] %></div><div class="hrl-kpi__label">Employees requested</div></div>
@@ -48,20 +63,6 @@
   <% end %>
 </section>
 
-<% if @grantable.any? %>
-  <section class="hrl-card">
-    <h2 class="hrl-card__title">Give WFH</h2>
-    <p class="hrl-small hrl-muted">Approved straight away, up to <%= HrLite::WfhRequest::HR_MAX_DAYS %> days from today or later.</p>
-    <%= form_with url: hr_lite.admin_wfh_requests_path, method: :post, local: true do %>
-      <div class="hrl-field"><label for="grant-user">Employee</label><%= select_tag :user_id, options_for_select(@grantable.map { |u| [ hr_display_name(u), u.id ] }), include_blank: "Choose one", required: true, id: "grant-user", data: { hrl_filter: ("Type a name to filter" if @grantable.size > 5) } %></div>
-      <div class="hrl-field"><label for="grant-from">From</label><%= date_field_tag :start_date, Date.current, min: Date.current, required: true, id: "grant-from" %></div>
-      <div class="hrl-field"><label for="grant-to">To</label><%= date_field_tag :end_date, Date.current, min: Date.current, required: true, id: "grant-to" %></div>
-      <div class="hrl-field"><label for="grant-reason">Reason</label><%= text_area_tag :reason, nil, rows: 2, required: true, id: "grant-reason" %></div>
-      <div class="hrl-form-actions"><button type="submit" class="hrl-btn hrl-btn--primary">Give WFH</button></div>
-    <% end %>
-  </section>
-<% end %>
-
 <section class="hrl-card">
   <h2 class="hrl-card__title">Schedules <%= @from.strftime("%-d %b") %> – <%= @to.strftime("%-d %b") %></h2>
   <% if @requests.any? %>

data/app/views/hr_lite/admin/wfh_requests/show.html.erb

@@ -1,10 +1,7 @@
 <% content_for(:page_title) { "WFH — #{hr_display_name(@request.user)}" } %>
+<a class="hrl-backlink" href="<%= hr_lite.admin_wfh_requests_path %>"><span aria-hidden="true">‹</span> Approvals</a>
 <div class="hrl-page__head">
   <h1 class="hrl-page__title">WFH — <%= hr_display_name(@request.user) %></h1>
-  <div class="hrl-page__actions">
-    <a class="hrl-btn" href="<%= hr_lite.admin_wfh_requests_path %>">All WFH</a>
-    <a class="hrl-btn" href="<%= hr_lite.admin_attendance_path(@request.user_id, month: @request.start_date.strftime("%Y-%m")) %>">Their month</a>
-  </div>
 </div>
 
 <section class="hrl-card">
@@ -19,6 +16,7 @@
         <% if @request.decision_note.present? %> · <%= @request.decision_note %><% end %></dd>
     <% end %>
   </dl>
+  <a class="hrl-card__link" href="<%= hr_lite.admin_attendance_path(@request.user_id, month: @request.start_date.strftime("%Y-%m")) %>">Their month</a>
 </section>
 
 <% if @request.pending? && @decidable %>
@@ -33,7 +31,7 @@
   <section class="hrl-card">
     <h2 class="hrl-card__title">Reject</h2>
     <%= form_with url: hr_lite.reject_admin_wfh_request_path(@request), method: :post, local: true do |f| %>
-      <div class="hrl-field"><%= f.label :decision_note, "Remark (required)", for: "reject_note" %><%= f.text_area :decision_note, id: "reject_note" %></div>
+      <div class="hrl-field"><%= f.label :decision_note, "Remark (required)", for: "reject_note" %><%= f.text_area :decision_note, id: "reject_note", required: true %></div>
       <div class="hrl-form-actions"><%= f.button "Reject", type: :submit, class: "hrl-btn hrl-btn--danger" %></div>
     <% end %>
   </section>

data/app/views/hr_lite/admin/work_locations/index.html.erb

@@ -6,7 +6,7 @@
 <div class="hrl-row hrl-row--mb">
   <span class="hrl-small hrl-muted">Status</span>
   <% HrLite::WorkLocation::STATUSES.each do |status| %>
-    <a class="hrl-btn <%= 'hrl-btn--primary' if @status == status %>" href="<%= hr_lite.admin_work_locations_path(status: status) %>"><%= status.humanize %></a>
+    <a class="hrl-btn" <%= "aria-current=true" if @status == status %> href="<%= hr_lite.admin_work_locations_path(status: status) %>"><%= status.humanize %></a>
   <% end %>
 </div>
 

data/app/views/hr_lite/admin/work_modes/index.html.erb

@@ -90,7 +90,7 @@
       <details>
         <summary class="hrl-btn">Set their mode</summary>
         <%= form_with url: hr_lite.admin_work_mode_override_path(@person.id), method: :post, local: true do %>
-          <div class="hrl-field"><label for="ov-mode">Mode</label><%= select_tag :mode, options_for_select(HrLite::WorkModeSwitch::SELECTABLE.map { |m| [ HrLite::WorkModeEvent.label(m), m ] }), id: "ov-mode" %></div>
+          <div class="hrl-field"><label for="ov-mode">Mode</label><%= select_tag :mode, options_for_select(HrLite::WorkModeSwitch::SELECTABLE.map { |m| [ HrLite::WorkModeEvent.label(m), m ] }), id: "ov-mode", data: { hrl_filter: "Type a mode" } %></div>
           <div class="hrl-field"><label for="ov-title">Title / purpose (meeting, travel, client meeting)</label><input class="hrl-input" id="ov-title" name="title" maxlength="120"></div>
           <div class="hrl-field"><label for="ov-client">Client</label><input class="hrl-input" id="ov-client" name="client" maxlength="120"></div>
           <div class="hrl-field"><label for="ov-dest">Destination</label><input class="hrl-input" id="ov-dest" name="destination" maxlength="120"></div>