Commit c1e94479 authored by Alexis Reigel's avatar Alexis Reigel

restrict height of the custom brand logo in emails

parent 26da2c45
...@@ -54,7 +54,10 @@ module EmailsHelper ...@@ -54,7 +54,10 @@ module EmailsHelper
def header_logo def header_logo
if brand_item && brand_item.header_logo? if brand_item && brand_item.header_logo?
brand_header_logo image_tag(
brand_item.header_logo,
style: 'height: 50px'
)
else else
image_tag( image_tag(
image_url('mailers/gitlab_header_logo.gif'), image_url('mailers/gitlab_header_logo.gif'),
......
...@@ -52,7 +52,7 @@ describe EmailsHelper do ...@@ -52,7 +52,7 @@ describe EmailsHelper do
) )
expect(header_logo).to eq( expect(header_logo).to eq(
%{<img src="/uploads/appearance/header_logo/#{appearance.id}/dk.png" alt="Dk" />} %{<img style="height: 50px" src="/uploads/appearance/header_logo/#{appearance.id}/dk.png" alt="Dk" />}
) )
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment