Show staff cards' primary sector first with the starred chip

Event staff cards listed sectors as plain compact badges with no primary
distinction. Use the shared tagging_label chip — primary first, starred
and darker — so the staff cards match the recipients page and the rest of
the app.

Also realign the people "Other" response spec with the profile's single
combined Sectors list (the two-column layout is gone): assert the Other
answer renders as a free-text "(other)" chip rather than the starred
primary, instead of keying off the removed "Additional sectors" heading.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: maebeale

app/views/events/staff.html.erb

@@ -28,7 +28,9 @@
         <% active_affiliation = person.affiliations.find { |a| !a.inactive? } %>
         <% org = active_affiliation&.organization %>
         <% bio = event_staff.public_bio %>
-        <% sectors = person.profile_show_sectors? ? person.sectors.sort_by(&:name) : [] %>
+        <%# Primary sector first (starred, darker chip), then the rest alphabetically —
+            the same treatment as the recipients page. %>
+        <% sector_items = person.profile_show_sectors? ? person.sectorable_items.includes(:sector).sort_by { |si| [ si.is_primary? ? 0 : 1, si.sector&.name.to_s.downcase ] } : [] %>
         <%# Facilitator tenure → one experience badge, computed from already-loaded
             affiliations (avoids the decorator's query-heavy badges call). Thresholds
             match PersonDecorator#compute_badges. %>
@@ -93,13 +95,13 @@
                 </span>
               <% end %>
 
-              <% if sectors.any? %>
+              <% if sector_items.any? %>
                 <div class="mt-1.5 flex flex-wrap justify-center gap-1">
-                  <% sectors.first(3).each do |sector| %>
-                    <span class="px-2 py-0.5 rounded-md text-xs font-medium <%= DomainTheme.bg_class_for(:sectors, intensity: 100) %> <%= DomainTheme.text_class_for(:sectors, intensity: 700) %>"><%= sector.name %></span>
+                  <% sector_items.first(3).each do |si| %>
+                    <%= render "sectors/tagging_label", sector: si.sector, is_primary: si.is_primary? %>
                   <% end %>
-                  <% if sectors.size > 3 %>
-                    <span class="px-2 py-0.5 rounded-md text-xs font-medium text-gray-400">+<%= sectors.size - 3 %></span>
+                  <% if sector_items.size > 3 %>
+                    <span class="px-2 py-0.5 rounded-md text-xs font-medium text-gray-400">+<%= sector_items.size - 3 %></span>
                   <% end %>
                 </div>
               <% end %>

spec/requests/people_other_responses_spec.rb

@@ -14,16 +14,17 @@ def answer(identifier, value)
   before { sign_in admin }
 
   describe "profile page" do
-    it "shows the Other service area as an additional sector" do
+    it "shows the Other service area as a free-text chip, not the primary sector" do
       person.update!(profile_show_sectors: true)
       answer("primary_service_area", "Other: Equine therapy")
 
       get person_path(person)
 
       expect(response.body).to include("Equine therapy")
-      # The Other chip sits in the "Additional sectors" column, not "Primary sector".
-      additional_section = response.body.split("Additional sectors").last
-      expect(additional_section).to include("Equine therapy")
+      # The combined Sectors list renders it via the free-text "(other)" chip rather
+      # than promoting it to the starred primary sector.
+      equine_chip = response.body[/Equine therapy.{0,80}/m]
+      expect(equine_chip).to include("(other)")
     end
   end