raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-16 04:24:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

Header: Improve logo, resizing and styles

Browse Source 
Use the sites main logo instead of footer logo to allow for better contrast on gray background (dark text on gray instead of white text on gray)

The table's background color is defined in the parent table already

Limit the max width and max height of the logo and resize the logo accordingly instead of forcing it to squeeze into a fixed size

Update specs
This commit is contained in:
drummer83
2023-12-25 22:02:24 +01:00
parent 48688311d1
commit a27e89f898
4 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/assets/stylesheets/mail/email.scss
View File

@@ -121,6 +121,10 @@ table.order-summary {
text-align: right;
}
.middle {
vertical-align: middle;
}
.social .soc-btn {
padding: 3px 7px;
font-size: 12px;

6
app/views/layouts/mailer.html.haml
View File

@@ -13,11 +13,11 @@
%td
%td.header.container
.content
%table{:bgcolor => "#f2f2f2"}
%table
%tr
%td
= image_tag ContentConfig.url_for(:footer_logo), width: "144", height: "50"
%td{:align => "right"}
= image_tag ContentConfig.url_for(:logo), style: "max-height: 50px; max-width: 300px;"
%td.text-right
%h6.collapse
= Spree::Config[:site_name]
%td

4
spec/mailers/order_mailer_spec.rb
View File

@@ -46,7 +46,7 @@ RSpec.describe Spree::OrderMailer do
end
it "display the OFN header by default" do
expect(email.body).to include(ContentConfig.url_for(:footer_logo))
expect(email.body).to include(ContentConfig.url_for(:logo))
end
context 'when hide OFN navigation is enabled for the distributor of the order' do
@@ -55,7 +55,7 @@ RSpec.describe Spree::OrderMailer do
end
it 'does not display the OFN navigation' do
expect(email.body).not_to include(ContentConfig.url_for(:footer_logo))
expect(email.body).not_to include(ContentConfig.url_for(:logo))
end
end
end

4
spec/mailers/subscription_mailer_spec.rb
View File

@@ -117,7 +117,7 @@ RSpec.describe SubscriptionMailer, type: :mailer do
end
it "display the OFN header by default" do
expect(email.body).to include(ContentConfig.url_for(:footer_logo))
expect(email.body).to include(ContentConfig.url_for(:logo))
end
describe "linking to order page" do
@@ -162,7 +162,7 @@ RSpec.describe SubscriptionMailer, type: :mailer do
end
it 'does not display the OFN navigation' do
expect(email.body).not_to include(ContentConfig.url_for(:footer_logo))
expect(email.body).not_to include(ContentConfig.url_for(:logo))
end
end
end