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

2147
by rating

Issues

Fair Mammoth May 8, 2019 at 13:43
— Overall performance is below average.

— The app is very slow when scaling, scrolling and filtering the third graph.
— It's also slow when viewing points on graphs #3 and #5.
— Bonus goal is implemented, however the animations are slow on LG Nexus 5X.
— Pie chart is broken for the last date.
iPhone 5S, iOS 12.1; LG Nexus 5X, Android 8.1
30
Earnest Pony May 8, 2019 at 14:40
Thanks for you review.

1. On low-end devices with low battery level performance degrades to save battery. I’ve tested on Xiaomi Redmi 6A with fully charged battery + all other apps are closed — and had decent speed for all charts.
2. It depends on response speed from the server where app is hosted. Please, try the app from my server: https://ffun.io/tgcs2
3. And what about other devices? One device isn’t enough to make final decision.
4. Confirm it’s a bug that is already fixed in latest version: https://ffun.io/tgcs2
I try to zoom "App" chart at 7 April 2018 (first day of chart). I see black screen how on screenshot. The last day of chart has black screen too
Mac OS, last FireFox
21
Earnest Pony Apr 22, 2019 at 10:48
Same bug as been reported above. Fixed in latest version: https://ffun.io/tgcs2
D
Deleted Account Apr 21, 2019 at 16:29
If you change the scale then you will see a global error
Windows 8, opera
2
Earnest Pony Apr 22, 2019 at 10:58
Cannot reproduce this in any major browser in OS X. Maybe something Windows+Opera related.
Not working in IE11. At first having emotions I hurried and disliked your work, it is my mistake, but after Fancy Hare comment I reviewed it again and still your work deserves attention and & liked it instead of dislike. Good luck in competition. (+2)
Windows 8/IE11/Chrome 72
2
Earnest Pony Apr 22, 2019 at 14:27
Code is served directly from source, without Closure Compiler/Babel transpilation (as per contest requirements). IE11 is old and doesn't support modern JS. (And thank you for good words.)
Zoomed "App" chart does not show correct data. For example: I selected 9 october 2018, and moved slider to first day - 6 october 2018 (how on screen) Apple has value 45770. Then I back to normal mode and check data of 6 october 2018, Apple has value 34260, (45770 !=34260) :(

BTW, I like your app, added you +
1
Earnest Pony Apr 22, 2019 at 10:50
I confirm, it's a bug (data is shifted by 1 day)
D
Deleted Account Apr 17, 2019 at 13:52
Parts of the date in the tooltip are animated considering the difference between old and new value, but are not animated in the chart range in the same way.
Earnest Pony Apr 17, 2019 at 14:29
Good catch. I decided to animate the whole date strings in chart date range display and keep ndash " – " always visible to ease visual perception of range change and not flood with visual effects more than needed.
D
Deleted Account Apr 17, 2019 at 14:14
The lowest and the highest values of current range should always match to the lower and upper edge of the graph like you do it in mini map.
any device / any browser
Earnest Pony Apr 17, 2019 at 14:35
This is by design. If you aim to display rounded Y labels, you have to round Y range to a bigger value divisible by number of levels (6 in our case).
Wrong Y ranges for pair graph, they should be exactly the same
Earnest Pony Apr 17, 2019 at 14:40
Thanks for submitting your finding. See my previous answer to Джастин Аркадьевич — it explains the Y axis rounding that I chose to use.
Some switchers can't be set off even if where other switchers in on state
Earnest Pony Apr 17, 2019 at 14:43
This restrinction is by design for percentage charts. Why show only one serie if it's always 100%? Also, you can use longtap to keep only one serie visible — will find small easter egg in tooltip ;)
No X axis animation on detail mode, instantly snaps to another day
Earnest Pony Apr 18, 2019 at 09:59
Dark Parrot, please, see my answer to your pal Giant Parrot, regarding X axis animations.
Rounded values in bar chart looks no so good during animation, especially in preview bar
Earnest Pony Apr 17, 2019 at 14:46
"Not so good" is very subjective 🤷‍♂️
Sum of percentages in 5th chart are not equal to 100 sometimes
Earnest Pony Apr 17, 2019 at 14:48
This is made to match contest mockups. Values are rounded. That's enough for the representation of relative sizes — decimal precision isn't needed here.
For 5th chart preview bar should display bars, not areas
Earnest Pony Apr 17, 2019 at 14:50
Not always, but only in "zoomed" mode (as per mockups). And I admit that I didn't notice that — therefore didn't implement :(
Wrong min/max at local range, otherwise good job
Earnest Pony Apr 18, 2019 at 21:25
Read above, please. Y range expansion is required to maintain precision of rounded Y labels.
display total for one dataset and not display for two
Earnest Pony Apr 18, 2019 at 09:53
I confirm, that's a bug. Already fixed in https://ffun.io/tgcs2/
Lack of performance on my device (~15 fps). Y labels have wrong animations (the value that is hiding should not change, it should go smoothly into opacity 0). X labels are not animated at all. Not working in old browsers.
Everything else is fine! Good pie ;)
Chrome for Android / Nexus 6p
Earnest Pony Apr 18, 2019 at 09:56
With use of Closure Compiler / Babel — will work in older browsers. But it was requested to provide unminified/unobfuscated code. About X labels — see my answer below to Giant Parrot.
Part of JQuery library is copied.
Code will not work in browsers without es7 async await support
Earnest Pony Apr 18, 2019 at 02:44
1. I didn’t copy jquery code, just used its $ name convention. Compare my code to jquery source and you’ll see ;)
2. Source is not minified, as requested. Otherwise I would use Closure Compiler and everything will work fine.
Hi, Doge! There is no X axis animation. Also the performance is far below average.
Redmi 3 / Chrome
Earnest Pony Apr 18, 2019 at 09:43
There were no requirements in Stage 2 mockups for X axis animation. But there were new X labels layout design. First and last X labels are for showing visible date range — therefore they’re not moving. So I decided to remake X axis to match new mockups. And if we’re talking about horizontal animations on the whole chart: I decided that instant chart reaction on navigation window resize/move is better than smooth, but delayed horizontal transition.
The pie chart functionality is broken for the 3 first and 3 last dates of the chart
Chrome 73
Earnest Pony Apr 18, 2019 at 12:05
I confirm it's a bug. Already fixed in latest version: https://ffun.io/tgcs2/
I've tried to zoom 5th chart on April 6 (last day) and pie wasn't rendered at all:
MacBook Pro 15" 2015 / Chrome / MacOS 10.14.3
Earnest Pony Apr 18, 2019 at 12:05
Same as above. Fixed @ https://ffun.io/tgcs2/
Pie chart tooltip doesn't show the sum of daily values (try to set 2+ days interval):
Earnest Pony Apr 19, 2019 at 11:54
Good catch. Confirm, it's a bug. Fixed it in latest version: https://ffun.io/tgcs2/
Initial date range selection is more than 2 months unlike the contest screenshots' final month
Chrome 73
Earnest Pony Apr 19, 2019 at 10:18
Viewport was just set to last 20%. Last month would be too small default for touch navigation.
P.S. Why did you decide that mockups show initial state? 🙂
Impossible to select less than almost a month on daily graphs. Probably a minor issue and could be ignored.
Chrome 73
Earnest Pony Apr 18, 2019 at 12:17
Just a min window width restriction to ease touch navigation.
Performance no good, hangs when double clicked but animations looks perfect
Xiaomi mi3
Earnest Pony Apr 19, 2019 at 10:19
Cannot reproduce. You've double clicked on what? 🤷‍♂️
No x axes animations, otherwise good job. Like.
Iphone x
Earnest Pony Apr 19, 2019 at 17:56
Thanks. But as I understand, X axis is changed in Stage 2. So I remade it (see Stage 2 mockups).
- Changing the range on minimap not affected to chart view.
- Misalignment between minimap selected range and chart view visible range
Earnest Pony Apr 19, 2019 at 18:17
1. Cannot reproduce. Works fine.
2. Aligned properly, see mockups. Parts of the range are under left/right borders of navigation window. This is by design.
With some actions on the minimap, a bug sometimes appears: a horizontal scroll bar appears, i.e. page content gets wider than screen width
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 21:04
Confirm that this is the issue on small screens. Fixed in latest version: https://ffun.io/tgcs2/
the tooltip goes beyond the edge of the screen
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 21:05
Confirm that this is the issue on small screens. Fixed in latest version: https://ffun.io/tgcs2/
another bug with tooltip and screen borders
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 21:05
Confirm that this is the issue on small screens. Fixed in latest version: https://ffun.io/tgcs2/
one more bug with tooltip and borders of the screen, on chart 5
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 21:09
Confirm that this is the issue on small screens. Fixed in latest version: https://ffun.io/tgcs2/
it is impossible to move the slider on the minimap on chart 5, if it have min-size
Earnest Pony Apr 19, 2019 at 22:42
Small screen issue again :) Tap more precisely, or tap outside where you want viewport to be set, or resize two times to set wanted range. This happens because I added some invisible gaps to sides to simplify their selection with finger.
There are no animations of a tooltip and arcs with a focus on an zoomed 5 graph
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:08
You're testing it in mobile emulator mode and it works as expected on touch devices.
Not uniform indent between x-labels
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:11
You have to measure distance between center points of labels, because they have different width depending on date (example: "May 30" and "Jul 1")
The dates show over chart's title
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:12
Squeeze mobile view even more, like, to 100px width — and report every issue again 🤣You're trying hard.
The zoom-out's icon goes outside and part of it is not visible
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:13
Same as above. Continue 😊
The displayed part of the graph is out of bounds and is not visible.

...and again dates show over chart's title
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 20, 2019 at 00:08
Those value are out of main “visible” range. And again, continue your perfect analysis — you're doing great 🙃
When scrolling through the third graphic, white stripes appear on the left
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:17
Cannot reproduce. Try harder, my mysterious friend.
Incorrect minimum values, in this case the minimum is 272.400, but the scale shows 200.000 (200k)
Windows 10, Chrome 73.0.3683.103
Earnest Pony Apr 19, 2019 at 23:18
I already described above about this. It's by design to have precise label rounding. Please, be more attentive 😉
Answer for: You're testing it in mobile emulator mode and it works as expected on touch devices.
Earnest Pony Apr 19, 2019 at 23:21
Oh, no. You've already found some major critical bug 😱Congrats 🎉 Please, continue! I really appreciate your attention to my work.
Answer for: Squeeze mobile view even more, like, to 100px width — and report every issue again 🤣You're trying hard.

I check on the sizes with the designs from the telegram, the same bug with the icon
Earnest Pony Apr 19, 2019 at 23:24
Cannot reproduce. But I will just believe you, ok? 🤗
Answer for: I already described above about this. It's by design to have precise label rounding. Please, be more attentive

The nearest round is 250k
And why then the rest are not rounded?

867, 733, 467 and 333
Earnest Pony Apr 20, 2019 at 08:24
Because math. Try to figure out it yourself, please, — as homework 😋

P.S. I've fixed some rounding problem in latest version. Please, check here: https://ffun.io/tgcs2
The best project, You will win
Earnest Pony Apr 22, 2019 at 14:27
Thanks 🤗
D
Deleted Account Apr 21, 2019 at 16:08
On all bonuses are not visible what day is shown
All
Earnest Pony Apr 22, 2019 at 10:54
This is made to match provided mockups. There's no date info within tooltip (see attached screenshots) 🤷‍♂️
D
Deleted Account Apr 21, 2019 at 16:12
On the 5th bonus, the values don’t change
All
Earnest Pony Apr 22, 2019 at 10:55
Confirm that's a bug — same as reported above. Fixed in latest version: https://ffun.io/tgcs2/
D
Deleted Account Apr 21, 2019 at 16:40
On the 5th bonus, the extreme dates show the error
All
Earnest Pony Apr 22, 2019 at 10:59
Confirm, it's a bug. Was reported earlier and already fixed in latest version: https://ffun.io/tgcs2/
D
Deleted Account Apr 21, 2019 at 16:49
Good work, but the author saved on animation when switching to a bonus and back. while other authors spent a lot of time on this animation
Earnest Pony Apr 22, 2019 at 11:03
It was clearly stated that "you are not expected to implement the zooming transitions exactly as shown in the video demonstrations. They may be replaced with any slick and fast transition of your choice." — so it's others' problem that they had wrong priorities and spent more time on zoom animation instead of primary goals.
Nobody added any issues yet...