Drop boosts

Killed by design.

Author: dhh

app/assets/stylesheets/cards.css

@@ -471,37 +471,6 @@
     }
   }
 
-  .input.boost__input {
-    --hover-size: 0;
-    --input-border-radius: 0;
-    --input-border-size: 0;
-    --input-padding: 0;
-    --outline-size: 0;
-
-    color: inherit;
-    font-size: inherit;
-    font-weight: inherit;
-    inline-size: min-content;
-    max-inline-size: 3ch;
-    min-inline-size: 1ch;
-    outline: none;
-
-    @supports (field-sizing: content) {
-      field-sizing: content;
-      max-inline-size: unset;
-    }
-
-    &:focus {
-      background-color: var(--color-highlight);
-    }
-
-    &::-webkit-outer-spin-button,
-    &::-webkit-inner-spin-button {
-      -webkit-appearance: none;
-      margin: 0;
-    }
-  }
-
   .card__closed {
     align-items: center;
     aspect-ratio: 7/4;

app/controllers/cards/boosts_controller.rb

@@ -19,13 +19,9 @@ def set_activity_day
     end
 
     def events_for_activity_day
-      user_events.where(created_at: @activity_day.all_day).
-        group_by { |event| [ event.created_at.hour, helpers.event_column(event) ] }.
-        map { |hour_col, events|
-          [ hour_col,
-            events.uniq { |e| e.action == "boosted" ? [ e.creator_id, e.card_id ] : e.id }
-          ]
-        }
+      user_events.where(created_at: @activity_day.all_day)
+        .group_by { |event| [ event.created_at.hour, helpers.event_column(event) ] }
+        .map { |hour_col, events| [ hour_col, events.uniq { |e| e.id } ] }
     end
 
     def latest_event_before_activity_day
@@ -38,7 +34,7 @@ def user_events
 
     def user_cards
       Current.user.accessible_cards
-            .published_or_drafted_by(Current.user)
-            .where(collection_id: collection_filter)
+        .published_or_drafted_by(Current.user)
+        .where(collection_id: collection_filter)
     end
 end

app/controllers/concerns/events_timeline.rb

@@ -72,8 +72,6 @@ def event_action_sentence(event)
       end
     when "unassigned"
       "#{ event.creator == Current.user ? "You" : event.creator.name } unassigned #{ event.assignees.include?(Current.user) ? "yourself" : event.assignees.pluck(:name).to_sentence } from <span style='color: var(--card-color)'>#{ card_title(event.card) }</span>".html_safe
-    when "boosted"
-      "#{ event.creator == Current.user ? "You" : event.creator.name } boosted <span style='color: var(--card-color)'>#{ card_title(event.card) }</span>".html_safe
     when "commented"
       "#{ event.creator == Current.user ? "You" : event.creator.name } commented on <span style='color: var(--card-color)'>#{ card_title(event.card) }</span>".html_safe
     when "published"
@@ -99,8 +97,6 @@ def event_action_icon(event)
     case event.action
     when "assigned"
       "assigned"
-    when "boosted"
-      "thumb-up"
     when "staged"
       "bolt"
     when "unstaged"

app/helpers/events_helper.rb

@@ -1,5 +1,5 @@
 class Card < ApplicationRecord
-  include Assignable, Boostable, Colored, DraftCommenting, Engageable, Eventable, Golden,
+  include Assignable, Colored, DraftCommenting, Engageable, Eventable, Golden,
     Messages, Notifiable, Pinnable, Closeable, Scorable, Searchable, Staged,
     Statuses, Taggable, Watchable
 
@@ -22,7 +22,6 @@ class Card < ApplicationRecord
   scope :indexed_by, ->(index) do
     case index
     when "most_active"    then ordered_by_activity
-    when "most_boosted"   then ordered_by_boosts
     when "newest"         then reverse_chronologically
     when "oldest"         then chronologically
     when "latest"         then latest

app/models/card.rb

@@ -36,7 +36,6 @@ def event_score(event)
 
     def event_weight(event)
       case
-      when event.boosted? then 1
       when event.comment&.first_by_author_on_card? then 20
       when event.comment&.follows_comment_by_another_author? then 15
       when event.commented? then 10
@@ -60,6 +59,6 @@ def last_scorable_activity_at
     end
 
     def scorable_events
-      events.where(action: [ :commented, :boosted ])
+      events.where(action: :commented)
     end
 end

app/models/card/boostable.rb

@@ -9,15 +9,9 @@ class Event < ApplicationRecord
   has_one :comment, through: :message, source: :messageable, source_type: "Comment"
 
   scope :chronologically, -> { order created_at: :asc, id: :desc }
-  scope :non_boosts, -> { where.not action: :boosted }
-  scope :boosts, -> { where action: :boosted }
 
   after_create -> { card.touch_last_active_at }
 
-  def boosted?
-    action == "boosted"
-  end
-
   def commented?
     action == "commented"
   end

app/models/card/scorable.rb

@@ -5,16 +5,12 @@ class EventSummary < ApplicationRecord
 
   # FIXME: Consider persisting the body and compute at write time.
   def body
-    "#{main_summary} #{boosts_summary}".squish
+    events.map { |event| summarize(event) }.join(" ")
   end
 
   private
     delegate :time_ago_in_words, to: "ApplicationController.helpers"
 
-    def main_summary
-      events.non_boosts.map { |event| summarize(event) }.join(" ")
-    end
-
     def summarize(event)
       case event.action
       when "published"
@@ -39,12 +35,4 @@ def summarize(event)
         "#{event.creator.name} changed title from '#{event.particulars.dig('particulars', 'old_title')}' to '#{event.particulars.dig('particulars', 'new_title')}'."
       end
     end
-
-    def boosts_summary
-      if tally = events.boosts.group(:creator).count.presence
-        tally.map do |creator, count|
-          "#{creator.name} +#{count}"
-        end.to_sentence + "."
-      end
-    end
 end

app/models/event.rb

@@ -1,7 +1,7 @@
 module Filter::Fields
   extend ActiveSupport::Concern
 
-  INDEXES = %w[ most_boosted newest oldest latest stalled ]
+  INDEXES = %w[ newest oldest latest stalled ]
 
   delegate :default_value?, to: :class