Merge pull request #2148 from basecamp/revert-avatar-change

Revert avatar change

app/assets/stylesheets/avatars.css

@@ -22,11 +22,6 @@
       max-inline-size: 100%;
       object-fit: cover;
     }
-
-    span[data-creator-id] {
-      display: grid;
-      place-items: center;
-    }
   }
 
   .avatar__form {

app/controllers/my/avatars_controller.rb

@@ -1,16 +0,0 @@
-class My::AvatarsController < ApplicationController
-  def show
-    if stale? Current.user
-      if Current.user.avatar.attached?
-        redirect_to rails_blob_url(Current.user.avatar_thumbnail, disposition: "inline")
-      else
-        render_initials
-      end
-    end
-  end
-
-  private
-    def render_initials
-      render formats: :svg
-    end
-end

app/controllers/users/avatars_controller.rb

@@ -5,20 +5,17 @@ class Users::AvatarsController < ApplicationController
   before_action :ensure_permission_to_administer_user, only: :destroy
 
   def show
-    expires_in 30.minutes, public: true, stale_while_revalidate: 1.week
-
     if @user.system?
       redirect_to view_context.image_path("system_user.png")
     elsif @user.avatar.attached?
       redirect_to rails_blob_url(@user.avatar_thumbnail, disposition: "inline")
-    else
+    elsif stale? @user, cache_control: cache_control
       render_initials
     end
   end
 
   def destroy
     @user.avatar.destroy
-    @user.touch
     redirect_to @user
   end
 
@@ -31,6 +28,14 @@ class Users::AvatarsController < ApplicationController
       head :forbidden unless Current.user.can_change?(@user)
     end
 
+    def cache_control
+      if @user == Current.user
+        {}
+      else
+        { max_age: 30.minutes, stale_while_revalidate: 1.week }
+      end
+    end
+
     def render_initials
       render formats: :svg
     end

app/helpers/avatars_helper.rb

@@ -42,11 +42,6 @@ module AvatarsHelper
   end
 
   def avatar_image_tag(user, **options)
-    tag.span data: { creator_id: user.id } do
-      safe_join [
-        image_tag(my_avatar_url(script_name: user.account.slug), aria: { hidden: "true" }, size: 48, title: user.name, data: { only_visible_to_you: true }, **options),
-        image_tag(user_avatar_url(user, script_name: user.account.slug), aria: { hidden: "true" }, size: 48, title: user.name, data: { only_visible_to_others: true }, **options)
-      ]
-    end
+    image_tag user_avatar_url(user, script_name: user.account.slug), aria: { hidden: "true" }, size: 48, title: user.name, **options
   end
 end

app/views/cards/comments/_comment.html.erb

@@ -1,4 +1,5 @@
 <% cache comment do %>
+  <%# Helper Dependency Updated: avatar_image_tag 2025-12-15 %>
   <%= turbo_frame_tag comment, :container do %>
     <%# Cache bump 2025-12-14: action text attachment rendering changed for lightbox -%>
     <div id="<%= dom_id(comment) %>" data-creator-id="<%= comment.creator_id %>"  class="comment flex align-start full-width <%= "comment--system" if comment.creator.system? %>">

app/views/events/_event.html.erb

@@ -1,4 +1,5 @@
 <% cache event do %>
+  <%# Helper Dependency Updated: avatar_image_tag 2025-12-15 %>
   <% if lookup_context.exists?("events/event/eventable/_#{event.action}") %>
     <%= render "events/event/eventable/#{event.action}", event: event %>
   <% else %>

app/views/my/avatars/show.svg.erb

@@ -1 +0,0 @@
-<%= render "users/initials", user: Current.user %>

app/views/notifications/_notification.html.erb

@@ -1,4 +1,5 @@
 <% cache notification do %>
+  <%# Helper Dependency Updated: avatar_image_tag 2025-12-15 %>
   <%= notification_tag notification do %>
     <%= render "notifications/notification/header", notification: notification do %>
       <%= notification_toggle_read_button(notification, url: card_reading_path(notification.card)) %>

app/views/users/_initials.svg.erb

@@ -1,22 +0,0 @@
-<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"
-  viewBox="0 0 512 512" class="avatar" aria-hidden="true">
-  <defs>
-    <clipPath id="porthole">
-      <circle cx="50%" cy="50%" r="50%" />
-    </clipPath>
-  </defs>
-
-  <g>
-    <rect width="100%" height="100%" rx="50" fill="<%= avatar_background_color(user) %>" />
-
-    <text x="50%" y="50%" fill="#FFFFFF"
-      text-anchor="middle" dy="0.35em"
-      <%=raw 'textLength="85%" lengthAdjust="spacingAndGlyphs"' if user.initials.size >= 3 %>
-      font-family="-apple-system, BlinkMacSystemFont, Segoe UI, Roboto, Helvetica, Arial, sans-serif"
-      font-size="230"
-      font-weight="800"
-      letter-spacing="-5">
-      <%= user.initials %>
-    </text>
-  </g>
-</svg>

app/views/users/avatars/show.svg.erb

@@ -1 +1,22 @@
-<%= render "users/initials", user: @user %>
+<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"
+  viewBox="0 0 512 512" class="avatar" aria-hidden="true">
+  <defs>
+    <clipPath id="porthole">
+      <circle cx="50%" cy="50%" r="50%" />
+    </clipPath>
+  </defs>
+
+  <g>
+    <rect width="100%" height="100%" rx="50" fill="<%= avatar_background_color(@user) %>" />
+
+    <text x="50%" y="50%" fill="#FFFFFF"
+      text-anchor="middle" dy="0.35em"
+      <%=raw 'textLength="85%" lengthAdjust="spacingAndGlyphs"' if @user.initials.size >= 3 %>
+      font-family="-apple-system, BlinkMacSystemFont, Segoe UI, Roboto, Helvetica, Arial, sans-serif"
+      font-size="230"
+      font-weight="800"
+      letter-spacing="-5">
+      <%= @user.initials %>
+    </text>
+  </g>
+</svg>

config/routes.rb

@@ -164,7 +164,6 @@ Rails.application.routes.draw do
   resource :landing
 
   namespace :my do
-    resource :avatar, only: :show
     resource :identity, only: :show
     resources :access_tokens
     resources :pins

test/controllers/my/avatars_controller_test.rb

@@ -1,33 +0,0 @@
-require "test_helper"
-
-class My::AvatarsControllerTest < ActionDispatch::IntegrationTest
-  setup do
-    sign_in_as :david
-  end
-
-  test "show own initials" do
-    get my_avatar_path
-    assert_match "image/svg+xml", @response.content_type
-    assert_match "private", @response.headers["Cache-Control"]
-    assert_match "must-revalidate", @response.headers["Cache-Control"]
-  end
-
-  test "show own image redirects to the blob url" do
-    users(:david).avatar.attach(io: File.open(file_fixture("moon.jpg")), filename: "moon.jpg", content_type: "image/jpeg")
-    assert users(:david).avatar.attached?
-
-    get my_avatar_path
-
-    assert_redirected_to rails_blob_url(users(:david).avatar.variant(:thumb), disposition: "inline")
-    assert_match "private", @response.headers["Cache-Control"]
-    assert_match "must-revalidate", @response.headers["Cache-Control"]
-  end
-
-  test "requires authentication" do
-    sign_out
-
-    get my_avatar_path
-
-    assert_response :redirect
-  end
-end

test/controllers/users/avatars_controller_test.rb

@@ -12,10 +12,17 @@ class Users::AvatarsControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to ActionController::Base.helpers.image_path("system_user.png")
   end
 
-  test "show initials with public caching" do
+  test "show own initials without caching" do
     get user_avatar_path(users(:david))
     assert_match "image/svg+xml", @response.content_type
-    assert @response.cache_control[:public]
+    assert @response.cache_control[:private]
+    assert_equal "0", @response.cache_control[:max_age]
+  end
+
+  test "show other initials with caching" do
+    get user_avatar_path(users(:kevin))
+    assert_match "image/svg+xml", @response.content_type
+    assert @response.cache_control[:private]
     assert_equal 30.minutes.to_s, @response.cache_control[:max_age]
   end