UI for "Image too large" errors

This commit is contained in:
Jérémie Astori 2017-12-28 16:31:19 -05:00
parent 335bd803af
commit 91e3ca88bb
No known key found for this signature in database
GPG Key ID: B9A4F245CD67BDE8
5 changed files with 42 additions and 6 deletions

View File

@ -1328,6 +1328,10 @@ kbd {
overflow: hidden; overflow: hidden;
} }
#chat .toggle-type-error {
background: transparent;
}
#chat .toggle-content img { #chat .toggle-content img {
max-width: 100%; max-width: 100%;
max-height: 128px; max-height: 128px;

View File

@ -0,0 +1,10 @@
"use strict";
const sizes = ["Bytes", "KB", "MB", "GB", "TB", "PB", "EB", "ZB", "YB"];
module.exports = function(size) {
// Loosely inspired from https://stackoverflow.com/a/18650828/1935861
const i = size > 0 ? Math.floor(Math.log(size) / Math.log(1024)) : 0;
const fixedSize = parseFloat((size / Math.pow(1024, i)).toFixed(1));
return `${fixedSize} ${sizes[i]}`;
};

View File

@ -29,10 +29,14 @@
</a> </a>
{{/equal}} {{/equal}}
{{#equal type "error"}} {{#equal type "error"}}
<a class="toggle-text" href="{{link}}" target="_blank" rel="noopener"> {{#equal error "image-too-big"}}
<div class="head" title="{{head}}">{{head}}</div> <em>
<div class="body" title="{{body}}">{{body}}</div> This image is larger than {{friendlysize maxSize}} and cannot be
</a> previewed.
<a href="{{link}}" target="_blank" rel="noopener">Click here</a>
to open it in a new window.
</em>
{{/equal}}
{{/equal}} {{/equal}}
</div> </div>
{{/preview}} {{/preview}}

View File

@ -100,8 +100,8 @@ function parse(msg, preview, res, client) {
case "image/webp": case "image/webp":
if (res.size > (Helper.config.prefetchMaxImageSize * 1024)) { if (res.size > (Helper.config.prefetchMaxImageSize * 1024)) {
preview.type = "error"; preview.type = "error";
preview.head = "Large image"; preview.error = "image-too-big";
preview.body = "Image is greater than your max image size. Click to view."; preview.maxSize = Helper.config.prefetchMaxImageSize * 1024;
} else { } else {
preview.type = "image"; preview.type = "image";
preview.thumb = preview.link; preview.thumb = preview.link;

View File

@ -0,0 +1,18 @@
"use strict";
const expect = require("chai").expect;
const friendlysize = require("../../../../../client/js/libs/handlebars/friendlysize");
describe("friendlysize Handlebars helper", function() {
it("should render big values in human-readable version", function() {
expect(friendlysize(51200)).to.equal("50 KB");
});
it("should round with 1 digit", function() {
expect(friendlysize(1234567)).to.equal("1.2 MB");
});
it("should render special case 0 as 0 Bytes", function() {
expect(friendlysize(0)).to.equal("0 Bytes");
});
});