kosmos/mastodon
kosmos/mastodon
8
0
Fork 0
Code Pull Requests Activity

Remove body_class_string helper (#33072)

Browse Source
This commit is contained in:
Matt Jankowski 2024-11-26 02:19:20 -05:00 committed by GitHub
parent 6d62581da1
commit 0ea9d8164b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 14 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/controllers/application_controller.rb
View File

@ -22,7 +22,6 @@ class ApplicationController < ActionController::Base
helper_method :use_seamless_external_login? helper_method :use_seamless_external_login?
helper_method :sso_account_settings helper_method :sso_account_settings
helper_method :limited_federation_mode? helper_method :limited_federation_mode?
helper_method :body_class_string
helper_method :skip_csrf_meta_tags? helper_method :skip_csrf_meta_tags?
rescue_from ActionController::ParameterMissing, Paperclip::AdapterRegistry::NoHandlerError, with: :bad_request rescue_from ActionController::ParameterMissing, Paperclip::AdapterRegistry::NoHandlerError, with: :bad_request
@ -158,10 +157,6 @@ class ApplicationController < ActionController::Base
current_user.setting_theme current_user.setting_theme
end end
def body_class_string
@body_classes || ''
end
def respond_with_error(code) def respond_with_error(code)
respond_to do |format| respond_to do |format|
format.any { render "errors/#{code}", layout: 'error', status: code, formats: [:html] } format.any { render "errors/#{code}", layout: 'error', status: code, formats: [:html] }

2
app/helpers/application_helper.rb
View File

@ -143,7 +143,7 @@ module ApplicationHelper
end end
def body_classes def body_classes
output = body_class_string.split output = []
output << content_for(:body_classes) output << content_for(:body_classes)
output << "theme-#{current_theme.parameterize}" output << "theme-#{current_theme.parameterize}"
output << 'system-font' if current_account&.user&.setting_system_font_ui output << 'system-font' if current_account&.user&.setting_system_font_ui

20
spec/helpers/application_helper_spec.rb
View File

@ -5,12 +5,20 @@ require 'rails_helper'
RSpec.describe ApplicationHelper do RSpec.describe ApplicationHelper do
describe 'body_classes' do describe 'body_classes' do
context 'with a body class string from a controller' do context 'with a body class string from a controller' do
before { helper.extend controller_helpers } before do
user = Fabricate :user
user.settings['web.use_system_font'] = true
user.settings['web.reduce_motion'] = true
user.save
it 'uses the controller body classes in the result' do helper.extend controller_helpers
end
it 'uses the current theme and user settings classes in the result' do
expect(helper.body_classes) expect(helper.body_classes)
.to match(/modal-layout compose-standalone/) .to match(/theme-default/)
.and match(/theme-default/) .and match(/system-font/)
.and match(/reduce-motion/)
end end
it 'includes values set via content_for' do it 'includes values set via content_for' do
@ -24,10 +32,8 @@ RSpec.describe ApplicationHelper do
def controller_helpers def controller_helpers
Module.new do Module.new do
def body_class_string = 'modal-layout compose-standalone'
def current_account def current_account
@current_account ||= Fabricate(:account) @current_account ||= Fabricate(:account, user: User.last)
end end
def current_theme = 'default' def current_theme = 'default'