From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 19 Feb 2014 19:32:55 GMT
Subject: Re: [PATCH] gitweb: Avoid overflowing page body frame with large images
Message-ID: <xmqq38jeaoew.fsf@gitster.dls.corp.google.com>
In-Reply-To: <C66B1A3C-7148-4514-B385-537E0BB6E187@kellerfarm.com>

Andrew Keller <andrew@kellerfarm.com> writes:

> When displaying a blob in gitweb, if it's an image, specify constraints for
> maximum display width and height to prevent the image from overflowing the
> frame of the enclosing page_body div.
>
> This change assumes that it is more desirable to see the whole image without
> scrolling (new behavior) than it is to see every pixel without zooming
> (previous behavior).
>
> Signed-off-by: Andrew Keller <andrew@kellerfarm.com>
> ---
>
> This is an updated copy of this patch.
>
> Could I request a thumbs up, thumbs down, or thumbs sideways from those who develop gitweb?

I do not develop gitweb, but the change looks reasonable to me.

>  gitweb/gitweb.perl       |    2 +-
>  gitweb/static/gitweb.css |    5 +++++
>  2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 3bc0f0b..79057b7 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -7094,7 +7094,7 @@ sub git_blob {
>  	git_print_page_path($file_name, "blob", $hash_base);
>  	print "<div class=\"page_body\">\n";
>  	if ($mimetype =~ m!^image/!) {
> -		print qq!<img type="!.esc_attr($mimetype).qq!"!;
> +		print qq!<img class="blob" type="!.esc_attr($mimetype).qq!"!;
>  		if ($file_name) {
>  			print qq! alt="!.esc_attr($file_name).qq!" title="!.esc_attr($file_name).qq!"!;
>  		}
> diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
> index 3b4d833..3212601 100644
> --- a/gitweb/static/gitweb.css
> +++ b/gitweb/static/gitweb.css
> @@ -32,6 +32,11 @@ img.avatar {
>  	vertical-align: middle;
>  }
>  
> +img.blob {
> +	max-height: 100%;
> +	max-width: 100%;
> +}
> +
>  a.list img.avatar {
>  	border-style: none;
>  }