consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use the dir attribute in dashboard and mailer layouts

Browse Source 
We forgot to do so in commit d827768c0. In order to avoid the same
mistake in the future, we're extracting a method to get these
attributes. We're also adding tests, since we didn't have any tests to
check that the `dir` attribute was properly set.
This commit is contained in:
Javi Martín
2022-10-06 18:10:35 +02:00
parent 314ce70000
commit 2c0ede3aaa
11 changed files with 63 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/components/layout/common_html_attributes_component.html.erb Normal file
View File

@@ -0,0 +1 @@
<%= attributes -%>

17
app/components/layout/common_html_attributes_component.rb Normal file
View File

@@ -0,0 +1,17 @@
class Layout::CommonHTMLAttributesComponent < ApplicationComponent
delegate :rtl?, to: :helpers
private
def attributes
sanitize([dir, lang].compact.join(" "))
end
def dir
'dir="rtl"' if rtl?
end
def lang
"lang=\"#{I18n.locale}\""
end
end

4
app/helpers/layouts_helper.rb
View File

@@ -9,4 +9,8 @@ module LayoutsHelper
link_to(text, path, options.merge(title: title))
end
end
def common_html_attributes
render Layout::CommonHTMLAttributesComponent.new
end
end

4
app/mailers/application_mailer.rb
View File

@@ -1,7 +1,5 @@
class ApplicationMailer < ActionMailer::Base
helper :settings
helper :application
helper :mailer
helper :application, :layouts, :mailer, :settings
default from: proc { "#{Setting["mailer_from_name"]} <#{Setting["mailer_from_address"]}>" }
layout "mailer"
before_action :set_asset_host

3
app/views/layouts/admin.html.erb
View File

@@ -1,6 +1,5 @@
<!DOCTYPE html>
<html <%= "dir=rtl" if rtl? %> lang="<%= I18n.locale %>">
<html <%= common_html_attributes %>>
<head>
<%= render "layouts/common_head", default_title: "Admin" %>
<%= content_for :head %>

2
app/views/layouts/application.html.erb
View File

@@ -1,5 +1,5 @@
<!DOCTYPE html>
<html <%= "dir=rtl" if rtl? %> lang="<%= I18n.locale %>" data-current-user-id="<%= current_user&.id %>">
<html <%= common_html_attributes %> data-current-user-id="<%= current_user&.id %>">
<head>
<%= render "layouts/common_head", default_title: setting["org_name"] %>
<%= render "layouts/meta_tags" %>

2
app/views/layouts/dashboard.html.erb
View File

@@ -1,5 +1,5 @@
<!DOCTYPE html>
<html lang="<%= I18n.locale %>" data-current-user-id="<%= current_user&.id %>">
<html <%= common_html_attributes %> data-current-user-id="<%= current_user&.id %>">
<head>
<%= render "layouts/common_head", default_title: setting["org_name"] %>
<%= render "layouts/meta_tags" %>

2
app/views/layouts/devise.html.erb
View File

@@ -1,5 +1,5 @@
<!DOCTYPE html>
<html <%= "dir=rtl" if rtl? %> lang="<%= I18n.locale %>">
<html <%= common_html_attributes %>>
<head>
<%= render "layouts/common_head", default_title: setting["org_name"] %>
<%= render "layouts/meta_tags" %>

2
app/views/layouts/mailer.html.erb
View File

@@ -1,5 +1,5 @@
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html lang="<%= I18n.locale %>">
<html <%= common_html_attributes %>>
<head>
<title><%= t("mailers.title") %></title>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">

3
app/views/layouts/management.html.erb
View File

@@ -1,6 +1,5 @@
<!DOCTYPE html>
<html <%= "dir=rtl" if rtl? %> lang="<%= I18n.locale %>">
<html <%= common_html_attributes %>>
<head>
<%= render "layouts/common_head", default_title: "Management" %>
<%= stylesheet_link_tag "print", media: "print" %>

34
spec/components/layout/common_html_attributes_component_spec.rb Normal file
View File

@@ -0,0 +1,34 @@
require "rails_helper"
describe Layout::CommonHTMLAttributesComponent do
let(:component) { Layout::CommonHTMLAttributesComponent.new }
it "includes the default language by default" do
render_inline component
expect(page.text).to eq 'lang="en"'
end
it "includes the current language" do
I18n.with_locale(:es) { render_inline component }
expect(page.text).to eq 'lang="es"'
end
context "RTL languages" do
let!(:default_enforce) { I18n.enforce_available_locales }
before do
I18n.enforce_available_locales = false
allow(I18n).to receive(:available_locales).and_return(%i[ar en es])
end
after { I18n.enforce_available_locales = default_enforce }
it "includes the dir attribute" do
I18n.with_locale(:ar) { render_inline component }
expect(page.text).to eq 'dir="rtl" lang="ar"'
end
end
end