Andrew Keller <andrew@xxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxx>
> ---
>
> 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;
> }
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html