Info

Kind Shark
Author
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

633
by rating

Issues

Fair Quokka May 8, 2019 at 13:08
— Overall performance is below average.
— The app is slow when filtering, scrolling and scaling all the graphs.
— Values on Y-axis are not correct on the fifth graph.
— Total is missing for graph #3, percentage for #5. Long tap is not implemented.
— Bonus goal not completed, zoom is missing, pie chart is implemented as a separate graph.
Samsung Galaxy J7 (2017), Android 8.1.0; iPhone 5S, iOS 12.1
30
Kind Shark May 8, 2019 at 17:40
Thanks for your issue!
- This solution was tested on my iphone 5s with ios 12 and it worked nice, very strange that performance are below average.

- Long tap was implemented reversively(long tap reset all buttons to initial state, my bad), but anyway, it works😅

- Yeah, I did not have enough time to implement all features
D
Deleted Account Apr 17, 2019 at 13:10
no comments
11
Kind Shark Apr 17, 2019 at 19:43
Thanks for the message, it is a small bug and it is fixing by adding three lines of code, but i was not add it because I don't know about that bug, because I do not scroll my charts so fast, really fast) and drop the control fast too, it is not so real world usage bug, people do not sroll this so fast, but yeah, bug is a bug:) thanks for the issue😁

It is only one bug that you found?
Fancy Hare Apr 22, 2019 at 11:58
Several small issues
The 'views' button is displayed for single column chart

The values displayed in tooltips are not in comma-separated format

No percentages displayed in the tooltip for the stacked percentage chart

The mask for bar charts is used on the focused bar rather than on all non-focused ones.

Wrong type of the pie chart timeline (navigator)

In general we can clearly see that the author did a great job and tried hard - even implemented a separate chart for pie diagram. Probably just didn't have enough time to finish and polish everything. To my mind it doesn't deserve so many dislikes. No dislike from me.
Chrome 73
1
Kind Shark Apr 22, 2019 at 12:15
Thank you very much for the issue!)
Д
Дмитрий Apr 17, 2019 at 13:48
Administrators Telegram, where's my graphics? Yesterday I sent a zip archive, but I do not see my work in the list
Kind Shark Apr 17, 2019 at 19:45
Thanks for the issue, but I am not an administrator:)
Dandy Bat Apr 18, 2019 at 05:22
https://jschart.usercontent.dev/entry211/build/
Authors, please move my app's data one dir down. Nobody can view my app since it placed not at '/entry211/' and at '/entry211/build/'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Kind Shark Apr 18, 2019 at 06:45
Thanks for the issue.

There was single IMPORTANT RULE: The archive must include index.html and all dependent resources (such as .js and .css files). Please make sure that all the paths you use are relative.
Dandy Bat Apr 18, 2019 at 07:26
hmm, so what kind of rule did i break? My archive includes html file with relative paths. There was not mark that it should placed on the top level. I just put it in the 'build' directory. What is the crime?
Kind Shark Apr 18, 2019 at 08:13
ARCHIVE must include index.html, not BUILD directory must include index.html
Witty Owl Apr 19, 2019 at 11:44
red line enters the range area and the points do not match the scale
Kind Shark Apr 19, 2019 at 13:48
Thanks for your issue!)

Line on control is a known bug(first issue)

The second one is strange, thanks!)
Crazy Pug Apr 24, 2019 at 21:38
Long press not work
MBP 2018 15, Chrome, Mojave
Kind Shark Apr 25, 2019 at 07:40
Thanks for the issue< but long press is working, but it works differently, it reset filters(enable all disabled data) just like in telegram requirements(maby i do not understand right)

upd: sorry, my bad, it works reversive
Crazy Pug Apr 24, 2019 at 21:40
Line charts: after moving trimmer there is trace from from main chart on preview
0:07
MBP 2018 15, Chrome, Mojave
Kind Shark Apr 25, 2019 at 07:39
Thanks for the issue, but this use case looks not usefull in real life, you don't need to move and drop control so fast, it looks strange
Crazy Pug Apr 24, 2019 at 21:41
Stacked, bars: selecting bar is not by design. Other bars should be semi-transparent
MBP 2018 15, Chrome, Mojave
Crazy Pug Apr 24, 2019 at 21:42
Large numbers not in human readable format
MBP 2018 15, Chrome, Mojave
Crazy Pug Apr 24, 2019 at 21:43
Preview: charts not rendered from start to end
MBP 2018 15, Chrome, Mojave
Kind Shark Apr 25, 2019 at 07:44
Thanks for the issue! This is not a bug, chart draws in range from left and right control, it is more correct, range need to be calculated from visible part of line, it is strange to draw it from start(behind the control)

it has better UX then on design
Crazy Pug Apr 24, 2019 at 21:45
Tooltip rendered outside of chart
MBP 2018 15, Chrome, Mojave
Kind Shark Apr 25, 2019 at 07:45
Thanks for the issue! My bad :)
Crazy Pug Apr 24, 2019 at 21:46
Stacked: red data is not visible
MBP 2018 15, Chrome, Mojave
Kind Shark Apr 25, 2019 at 07:45
Thanks for the issue!

Red data is too small to be visible, it is not a bug, just disable other data to look at red data
Nobody added any issues yet...