raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-13 23:37:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

Header: Improve logo, resizing and styles

Browse Source 
User 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)

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 f6e4b107b0
commit 129ce66194
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
%img{src: ContentConfig.url_for(:footer_logo), width: "144", height: "50"}/
%td{:align => "right"}
%img.middle{src: 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