hr_lite 0.35.0 → 0.36.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 874b64768a50e28c793779db9c5ff36e35f0b266cb318f34ca94df74e4de4bd7
-  data.tar.gz: f959b7a94d511c5ebf1f712cb29c1261df6207dc0c9a3bc6938dffad170900f8
+  metadata.gz: 91b052360d5658195be8a557e080ddcaea5660de77265ff6cb4d030b879f23a8
+  data.tar.gz: 32a1e1624df566187143af64b995b0af4a2db0c44463ff7600d0f6d7ed0f2e32
 SHA512:
-  metadata.gz: 482b65659ee54aec1e96bb34972028e428839f09925c7278cc97d3652a5ac54a622fcceb43575810561e1fd4dcb66037e2083d7f8856ae29ba71404c976d4dc6
-  data.tar.gz: 8d3540851748ca4677db0503a4f47e140bc7e27aa60ab7816280acfc317ae3fbc9ed0f7028719edf6a209932743aa24fb13ea94036b9f120296b0e576a089bac
+  metadata.gz: 0caced16fcf281925ba4ec901631dcffcd20e07aad5124781c67751298f1dfde737eb9ca60eaf4f3c100d47129a76a83bfb2ff93bd72d21db6de2834319dbc57
+  data.tar.gz: 8002305bba4b795098b2499a6765809a2a1aebf3c281fe715e7fc72f349ef3630cc653f2abbda98c32d03df4f5c223cbd4bf17e4ddb8b8404085aaa080a80f40

data/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.36.0] - 2026-10-13
+
+- Pick how long a mode lasts: Busy, Meeting and Out of office offer 15 min, 30 min, 1 h, 2 h, 4 h or 1 day (30 min preselected); Travelling and Client meeting also offer "Until I change it" (the default). Lunch and short break offer only lengths within their company limit — lunch 15 or 30 min by default — and one tap still takes the full limit; a longer break is refused. Every mode ends on its own at the picked time; the longest is a day. The choices are real radio buttons shown as 44px chips with 8px gaps.
+
+## [0.35.1] - 2026-10-13
+
+- Spacing: forms, buttons and expandable sections stacked in a card no longer touch (only form fields carried spacing, so "Set a status" sat on "Change my mode", "Save status" on "Check in", "Out of office" on "Check out"). One card-level rule gives them a 12px gap. The emoji grid, lead picker, check-in badges, punch-card links (now wrapping as whole items, no dangling dots) and the admin month's In/Out map links all keep at least the 8px gap between touch targets. Measured at 360 px and 1280 px across Home, Attendance, WFH, WFH locations, My work time, Team and the HR work-mode, WFH, locations, work-time, attendance, holidays and employee screens: no stacked targets closer than 8px, no horizontal scroll.
+
 ## [0.35.0] - 2026-10-13
 
 - Today's check-ins on Home: everyone in today, split into on time, late and not in yet (leave, holidays and weekends left out). "On time" uses each person's own slot plus their HR buffer and the grace minutes, the same rule as the late penalty. Hover or tap a person for their check-in, how late, check-out and hours worked (or hours so far).

data/app/assets/stylesheets/hr_lite/hr_lite.css

@@ -234,6 +234,18 @@ html:has(.hrl-body) { scroll-padding-top: 4rem; scroll-padding-bottom: calc(4.4r
 .hrl-card--empty { text-align: center; color: var(--hrl-muted); padding: 1.8rem 1rem; }
 .hrl-card--warn { background: var(--hrl-warn-bg); border-color: transparent; }
 .hrl-card > p:first-child { margin-top: 0; }
+/* Vertical rhythm: forms, buttons and disclosures stacked in a card never
+   touch (only .hrl-field carried spacing, so a button above a form, or an
+   open disclosure above the next control, sat flush). 12px >= the 8px
+   minimum gap between touch targets. */
+.hrl-card > :is(form, details, .button_to) + *,
+.hrl-card > * + :is(form, details, .button_to) { margin-top: .75rem; }
+details[open] > summary.hrl-btn { margin-bottom: .75rem; }
+.hrl-links { display: flex; flex-wrap: wrap; gap: var(--hrl-gap-target) 1rem; margin-bottom: var(--hrl-gap-target); }
+[data-hrl-crm] select { margin-top: var(--hrl-gap-target); }
+.hrl-checkins + .hrl-card__link { margin-top: var(--hrl-gap-target); }
+.hrl-punch-line + .hrl-punch-line { margin-top: var(--hrl-gap-target); }
+details > form > button[type=submit], details > form > .hrl-btn { margin-top: .25rem; }
 .hrl-card__link, .hrl-backlink { display: inline-flex; gap: var(--hrl-s-1); align-items: center; min-height: var(--hrl-target); min-width: var(--hrl-target); font-size: var(--hrl-t-small); font-weight: 600; }
 .hrl-listrow { display: grid; grid-template-columns: 1fr auto; align-items: start; gap: var(--hrl-s-1) var(--hrl-s-3); padding: var(--hrl-s-3) 0; border-bottom: 1px solid var(--hrl-line); min-height: var(--hrl-target); color: var(--hrl-ink); overflow-wrap: anywhere; }
 .hrl-listrow:last-child { border-bottom: 0; }
@@ -608,7 +620,7 @@ a.hrl-tree__name { display: inline-flex; align-items: center; min-height: var(--
 .hrl-chart-tip { position: absolute; z-index: 30; max-width: 16rem; padding: .4rem .6rem; border-radius: var(--hrl-radius-sm); background: var(--hrl-ink); color: var(--hrl-card); font-size: var(--hrl-t-small); pointer-events: none; }
 [data-hrl-tip]:focus { outline: 2px solid var(--hrl-accent); outline-offset: 1px; }
 .hrl-custom-status { display: inline-flex; gap: .3rem; align-items: center; overflow-wrap: anywhere; }
-.hrl-emoji-grid { display: grid; grid-template-columns: repeat(auto-fill, minmax(2.75rem, 1fr)); gap: .25rem; margin-top: .4rem; }
+.hrl-emoji-grid { display: grid; grid-template-columns: repeat(auto-fill, minmax(2.75rem, 1fr)); gap: var(--hrl-gap-target); margin-top: var(--hrl-gap-target); }
 .hrl-emoji { min-height: var(--hrl-target); border: 1px solid var(--hrl-line); border-radius: var(--hrl-radius-sm); background: var(--hrl-card); font-size: 1.25rem; cursor: pointer; }
 /* Today's check-ins: one initials badge per person; hover/tap shows times. */
 .hrl-checkins + .hrl-checkins { margin-top: .6rem; }

data/app/controllers/hr_lite/admin/work_modes_controller.rb

@@ -43,7 +43,7 @@ module HrLite
         raise ActiveRecord::RecordNotFound if employee.id == hr_current_user.id
 
         result = WorkModeSwitch.call(user: employee, mode: params[:mode], source: "hr", actor: hr_current_user,
-                                     params: params.permit(:title, :client, :destination, :reason, :minutes, :until, :category, :crm_ref))
+                                     params: params.permit(:title, :client, :destination, :reason, :minutes, :until, :category, :crm_ref, :duration))
         flash_for = result.ok? ? { notice: "#{hr_display_name(employee)} is now: #{result.event.label}." } : { alert: result.error }
         redirect_back fallback_location: admin_attendance_path(employee.id), **flash_for
       end

data/app/controllers/hr_lite/work_modes_controller.rb

@@ -5,7 +5,7 @@ module HrLite
 
     def create
       result = WorkModeSwitch.call(user: hr_current_user, mode: params[:mode], source: "employee", actor: hr_current_user,
-                                   params: params.permit(:title, :client, :destination, :reason, :minutes, :until, :category, :crm_ref))
+                                   params: params.permit(:title, :client, :destination, :reason, :minutes, :until, :category, :crm_ref, :duration))
       if result.ok?
         redirect_to attendance_path, notice: "You are now: #{result.event.label}."
       else

data/app/services/hr_lite/work_mode_switch.rb

@@ -13,6 +13,17 @@ module HrLite
 
     def self.call(...) = new(...).call
 
+    # How long a mode lasts, picked per use (owner, 2026-10-13): 15 min up to a day.
+    DURATIONS = { "15" => 15.minutes, "30" => 30.minutes, "60" => 1.hour, "120" => 2.hours,
+                  "240" => 4.hours, "day" => 1.day }.freeze
+    DURATION_LABELS = { "15" => "15 min", "30" => "30 min", "60" => "1 h", "120" => "2 h", "240" => "4 h", "day" => "1 day" }.freeze
+
+    # The lengths offered for a mode: breaks never past their company limit.
+    def self.duration_choices(mode)
+      cap = { "lunch" => Setting.instance.lunch_break_minutes, "short_break" => Setting.instance.short_break_minutes }[mode.to_s]
+      cap ? DURATIONS.keys.select { |k| DURATIONS[k] <= cap.minutes } : DURATIONS.keys
+    end
+
     CLEAR_AFTER = { "30" => 30.minutes, "60" => 1.hour, "240" => 4.hours, "today" => :today, "never" => nil }.freeze
 
     # A person's own status, Slack-style: works with or without a check-in
@@ -120,10 +131,18 @@ module HrLite
       day = @now.in_time_zone(HrLite.config.time_zone).all_day
       return "You have already taken lunch today." if WorkModeEvent.where(user_id: @user.id, mode: "lunch", started_at: day).exists?
 
-      base.merge(ends_at: @now + Setting.instance.lunch_break_minutes.minutes)
+      capped_break(Setting.instance.lunch_break_minutes)
     end
 
-    def short_break_attrs = base.merge(ends_at: @now + Setting.instance.short_break_minutes.minutes)
+    def short_break_attrs = capped_break(Setting.instance.short_break_minutes)
+
+    # A break runs the picked length (never past the limit), or the full limit.
+    def capped_break(limit)
+      return base.merge(ends_at: @now + limit.minutes) unless @p[:duration]
+      return "That break length is over the #{limit}-minute limit." unless self.class.duration_choices(@mode).include?(@p[:duration])
+
+      base.merge(ends_at: @now + DURATIONS[@p[:duration]])
+    end
 
     def parse_until
       Time.zone.parse("#{@now.to_date} #{@p[:until]}") if @p[:until]
@@ -131,12 +150,18 @@ module HrLite
       nil
     end
 
-    # minutes (30/60/…) or until "HH:MM" today; must be in the future, today.
+    # A picked length (15 min … 1 day), or minutes / an "HH:MM" end time;
+    # always in the future and at most a day away.
     def timed(attrs, required: true)
-      ends_at = @p[:minutes] ? @now + @p[:minutes].to_i.minutes : parse_until
+      return "Pick how long from the choices." if @p[:duration] && !DURATIONS.key?(@p[:duration])
+
+      ends_at = if @p[:duration] then @now + DURATIONS[@p[:duration]]
+      elsif @p[:minutes] then @now + @p[:minutes].to_i.minutes
+      else parse_until
+      end
       return attrs unless ends_at || required
-      return "Pick when it ends." unless ends_at
-      return "The end time must be later today." unless ends_at > @now && ends_at <= @now.end_of_day
+      return "Pick how long." unless ends_at
+      return "The end time must be later than now, and at most a day away." unless ends_at > @now && ends_at <= @now + 1.day
 
       attrs.merge(ends_at: ends_at)
     end

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

@@ -39,7 +39,7 @@
           <% [ [ "In", record.check_in_at, record.check_in_lat, record.check_in_lng ],
                [ "Out", record.check_out_at, record.check_out_lat, record.check_out_lng ] ].each do |label, at, lat, lng| %>
             <% next unless at %>
-            <div><%= label %> <%= at.strftime("%H:%M") %> · <%= record.remote? ? record.work_mode_label : (HrLite::OfficeLocation.place(lat, lng, @offices) || "No GPS") %>
+            <div class="hrl-punch-line"><%= label %> <%= at.strftime("%H:%M") %> · <%= record.remote? ? record.work_mode_label : (HrLite::OfficeLocation.place(lat, lng, @offices) || "No GPS") %>
               <% if lat && lng %>
                 · <a class="hrl-card__link" href="https://www.google.com/maps?q=<%= lat %>,<%= lng %>" target="_blank" rel="noopener noreferrer">Map<span class="hrl-sr-only"> of <%= label.downcase %> <%= record.date.strftime("%-d %b") %></span></a>
               <% end %>

data/app/views/hr_lite/attendance/_duration_chips.html.erb

@@ -0,0 +1,16 @@
+<%# How long a mode lasts: real radios styled as chips (selected state exposed).
+    locals: mode, id, optional (adds "Until I change it", the default) %>
+<% choices = HrLite::WorkModeSwitch.duration_choices(mode)
+   optional = local_assigns[:optional]
+   default = optional ? "" : (choices.include?("30") ? "30" : choices.last) %>
+<fieldset class="hrl-field">
+  <legend class="hrl-small"><strong>How long</strong></legend>
+  <div class="hrl-chiprow" role="radiogroup" aria-label="How long">
+    <% if optional %>
+      <label><input type="radio" name="duration" value="" <%= "checked" if default == "" %>><span>Until I change it</span></label>
+    <% end %>
+    <% choices.each do |key| %>
+      <label><input type="radio" name="duration" id="<%= id %>-<%= key %>" value="<%= key %>" <%= "checked" if key == default %>><span><%= HrLite::WorkModeSwitch::DURATION_LABELS[key] %></span></label>
+    <% end %>
+  </div>
+</fieldset>

data/app/views/hr_lite/attendance/_punch_card.html.erb

@@ -91,12 +91,15 @@
     </div>
   <% end %>
 
-  <a class="hrl-card__link" href="<%= hr_lite.attendance_path %>">View my month</a>
-  <% if HrLite.config.office_geofence %>
-    · <a class="hrl-card__link" href="<%= hr_lite.wfh_requests_path %>">Work from home</a>
-    · <a class="hrl-card__link" href="<%= hr_lite.work_locations_path %>">WFH locations</a>
-    · <a class="hrl-card__link" href="<%= hr_lite.work_time_path %>">My work time</a>
-  <% end %>
+  <%# Links wrap as whole items; no separator dots left dangling at a line end. %>
+  <div class="hrl-links">
+    <a class="hrl-card__link" href="<%= hr_lite.attendance_path %>">View my month</a>
+    <% if HrLite.config.office_geofence %>
+      <a class="hrl-card__link" href="<%= hr_lite.wfh_requests_path %>">Work from home</a>
+      <a class="hrl-card__link" href="<%= hr_lite.work_locations_path %>">WFH locations</a>
+      <a class="hrl-card__link" href="<%= hr_lite.work_time_path %>">My work time</a>
+    <% end %>
+  </div>
 
   <% if local_assigns.key?(:attention) %>
     <div class="hrl-row">

data/app/views/hr_lite/attendance/_work_mode.html.erb

@@ -22,22 +22,37 @@
   <details>
     <summary class="hrl-btn hrl-btn--block">Change my mode</summary>
     <div class="hrl-mode-switch">
-      <% %w[lunch short_break office].each do |mode| %>
-        <%= button_to hr_lite.work_mode_path, params: { mode: mode }, method: :post, class: "hrl-btn hrl-btn--block" do %>
-          <%= hrl_mode_icon(HrLite::WorkStatus::Entry.new(mode: mode, label: HrLite::WorkModeEvent.label(mode))) %>
-          <%= HrLite::WorkModeEvent.label(mode) %>
-        <% end %>
+      <%= button_to hr_lite.work_mode_path, params: { mode: "office" }, method: :post, class: "hrl-btn hrl-btn--block" do %>
+        <%= hrl_mode_icon(HrLite::WorkStatus::Entry.new(mode: "office", label: HrLite::WorkModeEvent.label("office"))) %>
+        <%= HrLite::WorkModeEvent.label("office") %>
       <% end %>
       <% if HrLite::WfhRequest.approved_for(hr_current_user.id, Date.current) || HrLite::EmployeeProfile.where(user_id: hr_current_user.id).pick(:permanent_wfh) %>
         <%= button_to "Work from home", hr_lite.work_mode_path, params: { mode: "wfh" }, method: :post, class: "hrl-btn hrl-btn--block" %>
       <% end %>
+      <%# Breaks: one tap for the full limit, or pick a shorter length when the limit allows a choice. %>
+      <% %w[lunch short_break].each do |mode| %>
+        <% if HrLite::WorkModeSwitch.duration_choices(mode).size > 1 %>
+          <details>
+            <summary class="hrl-btn hrl-btn--block"><%= HrLite::WorkModeEvent.label(mode) %></summary>
+            <%= form_with url: hr_lite.work_mode_path, method: :post, local: true do %>
+              <input type="hidden" name="mode" value="<%= mode %>">
+              <%= render "hr_lite/attendance/duration_chips", mode: mode, id: "dur-#{mode}" %>
+              <button type="submit" class="hrl-btn hrl-btn--primary">Start <%= HrLite::WorkModeEvent.label(mode).downcase %></button>
+            <% end %>
+          </details>
+        <% else %>
+          <%= button_to hr_lite.work_mode_path, params: { mode: mode }, method: :post, class: "hrl-btn hrl-btn--block" do %>
+            <%= hrl_mode_icon(HrLite::WorkStatus::Entry.new(mode: mode, label: HrLite::WorkModeEvent.label(mode))) %>
+            <%= HrLite::WorkModeEvent.label(mode) %>
+          <% end %>
+        <% end %>
+      <% end %>
 
       <details>
         <summary class="hrl-btn hrl-btn--block">Busy / focus</summary>
         <%= form_with url: hr_lite.work_mode_path, method: :post, local: true do %>
           <input type="hidden" name="mode" value="busy">
-          <div class="hrl-field"><label for="busy-minutes">For</label>
-            <%= select_tag :minutes, options_for_select([ [ "30 minutes", 30 ], [ "1 hour", 60 ], [ "2 hours", 120 ] ]), id: "busy-minutes" %></div>
+          <%= render "hr_lite/attendance/duration_chips", mode: "busy", id: "dur-busy" %>
           <button type="submit" class="hrl-btn hrl-btn--primary">Set busy</button>
         <% end %>
       </details>
@@ -48,7 +63,7 @@
           <div class="hrl-field"><label for="mt-title">Meeting</label><input class="hrl-input" id="mt-title" name="title" maxlength="120" required></div>
           <div class="hrl-field"><label for="mt-client">Client (optional)</label><input class="hrl-input" id="mt-client" name="client" maxlength="120"></div>
           <%= render "hr_lite/attendance/crm_picker", id: "mt-crm" %>
-          <div class="hrl-field"><label for="mt-until">Ends at</label><input class="hrl-input" type="time" id="mt-until" name="until" required></div>
+          <%= render "hr_lite/attendance/duration_chips", mode: "meeting", id: "dur-meeting" %>
           <button type="submit" class="hrl-btn hrl-btn--primary">Start meeting</button>
         <% end %>
       </details>
@@ -60,7 +75,7 @@
           <div class="hrl-field"><label for="tr-why">Purpose</label><input class="hrl-input" id="tr-why" name="title" maxlength="120" required></div>
           <div class="hrl-field"><label for="tr-client">Client (optional)</label><input class="hrl-input" id="tr-client" name="client" maxlength="120"></div>
           <%= render "hr_lite/attendance/crm_picker", id: "tr-crm" %>
-          <div class="hrl-field"><label for="tr-until">Back by (optional)</label><input class="hrl-input" type="time" id="tr-until" name="until"></div>
+          <%= render "hr_lite/attendance/duration_chips", mode: "travelling", id: "dur-travelling", optional: true %>
           <button type="submit" class="hrl-btn hrl-btn--primary">Start travelling</button>
         <% end %>
       </details>
@@ -72,7 +87,7 @@
           <div class="hrl-field"><label for="om-client">Client or prospect<%= " (or link a lead above)" if HrLite.config.crm_search %></label><input class="hrl-input" id="om-client" name="client" maxlength="120"<%= " required" unless HrLite.config.crm_search %>></div>
           <div class="hrl-field"><label for="om-why">Purpose</label><input class="hrl-input" id="om-why" name="title" maxlength="120" required></div>
           <div class="hrl-field"><label for="om-where">Where (optional)</label><input class="hrl-input" id="om-where" name="destination" maxlength="120"></div>
-          <div class="hrl-field"><label for="om-until">Ends at (optional)</label><input class="hrl-input" type="time" id="om-until" name="until"></div>
+          <%= render "hr_lite/attendance/duration_chips", mode: "outside_meeting", id: "dur-om", optional: true %>
           <button type="submit" class="hrl-btn hrl-btn--primary">Start client meeting</button>
         <% end %>
       </details>
@@ -82,7 +97,7 @@
           <input type="hidden" name="mode" value="out_of_office">
           <div class="hrl-field"><label for="oo-why">Reason</label>
             <%= select_tag :category, options_for_select(HrLite::WorkModeSwitch::OUT_OF_OFFICE_REASONS.invert), id: "oo-why" %></div>
-          <div class="hrl-field"><label for="oo-until">Back by</label><input class="hrl-input" type="time" id="oo-until" name="until" required></div>
+          <%= render "hr_lite/attendance/duration_chips", mode: "out_of_office", id: "dur-oo" %>
           <p class="hrl-small hrl-muted">Away for a full day? <a href="<%= hr_lite.new_leave_request_path %>">Apply for leave</a> instead.</p>
           <button type="submit" class="hrl-btn hrl-btn--primary">Set out of office</button>
         <% end %>

data/lib/hr_lite/version.rb

@@ -1,3 +1,3 @@
 module HrLite
-  VERSION = "0.35.0"
+  VERSION = "0.36.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hr_lite
 version: !ruby/object:Gem::Version
-  version: 0.35.0
+  version: 0.36.0
 platform: ruby
 authors:
 - kshitiz sinha
@@ -350,6 +350,7 @@ files:
 - app/views/hr_lite/attendance/_calendar_connection.html.erb
 - app/views/hr_lite/attendance/_crm_picker.html.erb
 - app/views/hr_lite/attendance/_custom_status.html.erb
+- app/views/hr_lite/attendance/_duration_chips.html.erb
 - app/views/hr_lite/attendance/_month_grid.html.erb
 - app/views/hr_lite/attendance/_penalties.html.erb
 - app/views/hr_lite/attendance/_punch_card.html.erb