Info

Open Website

Testing and Issues

You can test this app and submit issues during the testing period of the JavaScript Chart contest.

Entries with serious issues will not be able to win the contest, but even minor issues might be important for overall results.

Voting

3320
by rating

Issues

Fair Quokka May 8, 2019 at 15:00
— Overall performance is above average.
— Slightly sloppy on launch.
— Blinking when switching themes.
— Long click is not implemented on desktop (all ok on mobile).
— Some minor design issues (the colors when the night theme applied).
Samsung Galaxy J7 (2017), Android 8.1.0; iPhone 5S, iOS 12.1
32
Regal Mink May 8, 2019 at 15:42
Thank you for the review!

> Long click is not implemented on desktop
According to the task condition "A long tap on any data filter should uncheck all other filters." as well as relying on user experience, long press is only applicable on mobile devices with touch screen. I'm not sure that long click is actually present somewhere in the Desktop Experience. This was intentional and adding long press is not a problem.

> Some minor design issues (the colors when the night theme applied).
I still don't understand what the wrong colors are. I double-checked it. As I wrote above If you're talking about line colors, I took the colors from the overview.json, because in accordance with condition of the competition it was necessary to make a tool that builds appropriate graphics based on the input data.

> Blinking when switching themes.
Switching themes done without background animation intentionally to make switching fast on slow devices.
Nexus 6p / Android 8.1.0
Glitches on horizontal scrolling.
Samsung Galaxy S9 , Android 8.0.0, Chrome 73.0.3683.90
31
Regal Mink May 10, 2019 at 13:16
Thanks for the comment. 

I understand that it looks like a glitch, but it's a simplification of lines for a large amount of data. The default is 2 pixels to start simplifying the data. If these two pixels contain more than 1 point, they are combined in a special way. This behavior can be easily removed by configuring this library. I'm sorry if that caused any confusion.

I hope that this will not affect the result in any way.

Please see my screenshot (2019-05-10 (3).png) how to configure the library to avoid this behavior.

-----
UPD: Kindly see the solution with the changed configuration here: https://iprit.github.io/telechart2/
Large Beaver Apr 18, 2019 at 15:58
Invalid polygons in area and bar chart but text change and axis animations is beautifull
Xiaomi mi 3
10
Regal Mink Apr 18, 2019 at 17:29
Thank you for the comment here!

What browser did u use? Is it a webview? Can you test it in Chrome for Android for example? I think it's kinda device problems because there is no extraordinary using of simple HTML5 canvas.
Invalid colors for night mode
10
Regal Mink Apr 18, 2019 at 16:45
Thank you so much for the comment!

If you're talking about line colors, I took the colors from the overview.json, because in accordance with condition of the competition it was necessary to make a tool that builds appropriate graphics based on the input data. If we're talking about other UI colors, I can't find any inconsistencies. I checked every color in every mode. What exactly doesn't match and where?
Dark Parrot Apr 18, 2019 at 20:51
Voted up. Very good performance. Slightly faster then mine on very old devices. But on iphone 4s my app is faster, don't know why, on 5th your is faster.
2
Regal Mink Apr 20, 2019 at 18:00
Thank you, good to hear!
60 fps on 6x throttling on #3 - #5 charts it's godlike man, I applaud you standing :) But low performance on line charts is sad. Also 40KB for library after gzip is too much :( Anyway, great job!
1
Regal Mink Apr 20, 2019 at 18:07
Thank you for such a pleasing comment! 

I'm surprised about line chart isn't working fast. What device & browser did you use for the test? 

As for the file size - because in this version I made support for drawing in WebWorkers together with newest OffscreenCanvas. This feature is enabled at the bottom of the page (only for modern browsers but with backward compatibility). Also, this script includes all data from different overview.json.

UPD (20.04):
If you're very interested, I accelerated this app in another ~1.55 times (in terms of FPS) and making several different comparisons with my current application!
You can test it here: https://iprit.github.io/telechart2/
Slim Jaguar Apr 19, 2019 at 13:37
Webworkers support. Cool.
1
Regal Mink Apr 20, 2019 at 18:11
Thank you!

If you're very interested, I made this app even more faster in ~1.55 times (in terms of FPS) and made several different comparisons with my current application!
You can test it here: https://iprit.github.io/telechart2/
No devicepixelratio support
ios 12.1
Regal Mink Apr 20, 2019 at 18:04
It's not true, it supports devicePixelRatio (DPR later). 

The application has several layers (Chart, Minimap, UI layer, etc) and each has its own modifier for DPR. For example, for a minimap, you might not have the maximum DPR value supported by the device, since all the lines/bars/areas are barely visible, so this modifier allows you to set how much scale the DPR of the layer. 
Another example is UI layer. There is no need to reduce anything and all UI elements must be with excellent quality.

The effect of this logic pays off and allows you to find the best option between quality and performance. These modifiers are easily adjustable and this is not a big problem, so I have already increased modifier for the main layer (in my repository) for better retina support.

Thank you so much for the comment here!
About zooming: if you dont make zoom, so you shouldnt add little arrow in tooltip.
Regal Mink Apr 18, 2019 at 00:52
I tried to match design specification and moqups as much as possible, even if I didn't make the bonus goal. In the real application these icons should be done, so I made them. 

I have adhered to what needs to be done for great speed and performance of main 5 charts. They should be thoroughly thought out and maximum convenient for use on both Desktop and Mobile devices, as well as with full support for old browsers (IE 9+). Even more, I have done for modern browsers drawings in separate threads on newest OffscreenCanvas.
Дмитрий Потапов Apr 17, 2019 at 16:38
Administrators Telegram, where my graphics? Yesterday I sent a zip archive, but I do not see my work in the list
Regal Mink Apr 29, 2019 at 00:07
Thank you for the comment here. Unfortunately, I can't reproduce your issue.
Sleek Shark Apr 18, 2019 at 16:19
long press switch work only for touch, exclude mouse
Regal Mink Apr 18, 2019 at 16:26
Thank you for the comment here!

According to the task condition "A long tap on any data filter should uncheck all other filters." as well as relying on user experience, long press is only applicable on mobile devices with touch screen. I'm not sure that long press is actually present somewhere in the Desktop Experience.
No bonus tasks, but that is not an issue, if not planned.
Regal Mink Apr 18, 2019 at 19:34
Thank you for the comment here!

As I explained above, I aimed to the speed and performance of the app on old devices, as well as the usability on all platforms and browsers (old and new).
Slim Jaguar Apr 19, 2019 at 13:34
On scrollbar, I am able to move right handle while holding left

Check also mine https://contest.dev/chart-js/entry9
Regal Mink Apr 19, 2019 at 15:39
Thank for the comment here!

This is intended behavior. The main purpose is to not restrict the movement, while keeping the minimum width of the slider.
Kingly Chicken Apr 19, 2019 at 15:14
Incorrect brush design
Iphone x
Regal Mink Apr 19, 2019 at 16:26
Thank you for your issue here!

This is intentional. I added a small gap between the charts and the slider so that the slider would not blended with the minimap. It even has on moqups with line charts.
Nobody added any issues yet...