napchart icon indicating copy to clipboard operation
napchart copied to clipboard

Improve readability of charts on thumbnails created by crawlers

Open Theleruby opened this issue 7 years ago • 6 comments

I am generating napchart thumbnails using this API: https://napchart.com/api/getImage?chartid=e1sml&width=600&height=600&shape=circle

Before, when inserting these thumbnails inline into Discord using MessageEmbed, the text on the length of time allotted for naps was readable. This allowed the napchart to be understood even by looking at the thumbnail. Now, it is only very barely legible. In order to comfortably see the times you are forced to open the chart in your browser. This is quite a large usability regression.

Side by side comparison: Thumbnail comparison

It would be great for you to increase the font size at least to the point where the thumbnail is readable.

Theleruby avatar Jan 28 '18 18:01 Theleruby

I definitely aggree. Should do something about text size. Do you think dynamic text size based on size of image would work? Like for a thumb you get larger size?

larskarbo avatar Jan 28 '18 21:01 larskarbo

Yes that should probably be OK.

Theleruby avatar Jan 28 '18 21:01 Theleruby

fix is rolled out

https://github.com/larskarbo/napchart/commit/56ba3a35db317e2d030bf4ed515b3acf559be2a4

blog post: https://blog.napchart.com/update-on-text-size/

larskarbo avatar Jan 29 '18 10:01 larskarbo

Thanks, this looks much better now And thanks for the blog mention 👍

Theleruby avatar Jan 29 '18 14:01 Theleruby

Here's a preview of how it looks now:

thumbnail

I think maybe the font size for the text in the middle on the thumbnails should be reduced a bit, because it's overlapping the chart by quite a lot. But the text size on the sleep blocks is now perfect.

Trying to balance between appearance and function might be a little challenging here 😛

Theleruby avatar Jan 29 '18 15:01 Theleruby

Definitely need some work on this, I will leave the issue open

larskarbo avatar Feb 02 '18 14:02 larskarbo