JavaScript Chart – Developer Challenges

Fair Quokka May 8, 2019 at 14:03

— Overall performance is slightly below average.
— The app is slow when filtering and scaling the third and the fifth graphs.
— Bonus goals is completed, slow when zooming in and out.
— Blinking when filtering and scrolling the graphs.

Samsung Galaxy J7 (2017), Android 8.1.0; iPhone 5S, iOS 12.1

30

Merry Ant Apr 17, 2019 at 13:17

No retina support (blurry on retina screen)

All devices with pixelRatio > 1

21

Earnest Hedgehog Apr 22, 2019 at 06:04

I agree. I think this isn't a fatal bug. easy fix: add one line to the code

I spent less than 2 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Yuriy Reznik Apr 17, 2019 at 15:21

no line disappearing animation in minimap

0:08

21

Earnest Hedgehog Apr 19, 2019 at 20:28

I agree

Fancy Hare Apr 18, 2019 at 10:38

The values displayed in the tooltip for pie chart are different from those displayed for the same date in the standard mode of the stacked percentage chart.

0:41

Chrome 73

11

Earnest Hedgehog Apr 22, 2019 at 06:55

I think this isn't a fatal bug. easy fix: change one line to the code

I spent less than 2 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Funny Owl Apr 17, 2019 at 12:08

If disable graph during animation artifact appears

10

Earnest Hedgehog Apr 18, 2019 at 07:05

I tried to repeat this mistake, 15 minutes, I did not succeed. My app is working correctly. show on video or is it fake.

Merry Ant Apr 17, 2019 at 14:20

No percent numbers on small pieces

10

Earnest Hedgehog Apr 18, 2019 at 07:35

Yessss. this question created the biggest argument with my girlfriend. but I decided not to write the text in small pieces.
I didn't like any solution to this problem

Yuriy Reznik Apr 17, 2019 at 15:15

While draging range selector - y axis disappearing at all

0:05

10

Earnest Hedgehog Apr 20, 2019 at 09:03

You wrote it all right. the old disappears, the new appears. maybe, I could improve this function.

Funny Owl Apr 17, 2019 at 12:09

BTW awesome work, great

3

Earnest Hedgehog Apr 18, 2019 at 07:05

thank you

Funny Owl Apr 17, 2019 at 12:04

A wrong starting point for zoom on a 4th graph

Earnest Hedgehog Apr 22, 2019 at 05:53

I used local time zone methods. I think this isn't a fatal bug. easy fix: replace several methods, getMonth() -> getUTCMonth(), getDate() -> getUTCDay() ...

I spent less than 5 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Funny Owl Apr 17, 2019 at 12:04

Zoom out doesn't work with disabled data

Earnest Hedgehog Apr 18, 2019 at 06:59

This is not a bug, this is my setting. if the data is disabled then the functionality is blocked

Yuriy Reznik Apr 17, 2019 at 15:18

while moving range selector from start to end (2th chart) - lines fully disappearing from view area

0:05

Earnest Hedgehog Apr 18, 2019 at 07:56

You wrote it all right, I don't think it's a bug. controversial question how best to do.

Yuriy Reznik Apr 17, 2019 at 15:22

in night mode we can see artifacts in y axis while moving cursor

0:08

Earnest Hedgehog Apr 18, 2019 at 08:35

I don't think it's a bug. for resource savings. I don't redraw the canvas and I put a layer(mask) on top of the canvas

Yuriy Reznik Apr 17, 2019 at 15:23

no line disappearing animation in minimap at all

0:07

Earnest Hedgehog Apr 19, 2019 at 20:26

duplicate

Yuriy Reznik Apr 17, 2019 at 15:24

informer should contains average value of selected period, not sum of values

Earnest Hedgehog Apr 22, 2019 at 06:23

I think this isn't a fatal bug. easy fix: change one line to the code

I spent about 1 minute to correct this bug. You can test: http://markhel.ru/t/index.html

Yuriy Reznik Apr 17, 2019 at 15:28

desktop view is very cumbersome

Earnest Hedgehog Apr 18, 2019 at 08:59

maximum width one chart 500px. On most PCs there will be 2 charts on the line. on my opinion this is a good view

Sleek Shark Apr 17, 2019 at 18:00

After the zoom there should be no preview on this chart.

Earnest Hedgehog Apr 22, 2019 at 06:44

you're right. little nuance

I spent less than 10 minutes to correct this nuance. You can test: http://markhel.ru/t/index.html

Witty Owl Apr 17, 2019 at 21:21

bad scale

Earnest Hedgehog Apr 22, 2019 at 06:51

Look at the last point already the line went to the top

Fancy Hare Apr 18, 2019 at 10:00

Initial time range selection on zoommed in single column chart is less than 100%

0:13

Chrome 73

Earnest Hedgehog Apr 22, 2019 at 05:55

Duplicate. I used local time zone methods. I think this isn't a fatal bug. easy fix: replace several methods, getMonth() -> getUTCMonth(), getDate() -> getUTCDay() ...

I spent less than 5 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Fancy Hare Apr 18, 2019 at 10:31

In rare cases when y_scaled chart lines differ in maximum top on relative scale - the lines are not rescaled properly after selecting/deselecting the filters (chips, buttons)

1:16

Chrome 73

Earnest Hedgehog Apr 22, 2019 at 06:53

at the last point already the line went to the top

Fancy Hare Apr 18, 2019 at 10:41

The dates start from 6 Apr and end on 5 Apr unlike the data, which is 7 Apr to 6 Apr - not sure, might be just wrong date processing with local timezones or something like that.
Probably a minor issue and should be ignored.

Chrome 73

Earnest Hedgehog Apr 22, 2019 at 05:56

Duplicate. I used local time zone methods. I think this isn't a fatal bug. easy fix: replace several methods, getMonth() -> getUTCMonth(), getDate() -> getUTCDay() ...

I spent less than 5 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Fancy Hare Apr 18, 2019 at 10:45

Impossible to select less than almost a month on daily graphs

Chrome 73

Earnest Hedgehog Apr 18, 2019 at 12:21

This is not a bug, this is my setting.

Hip Hyena Apr 18, 2019 at 15:30

Unable to select time range by dragging in the preview area (outside of the current window).

Earnest Hedgehog Apr 22, 2019 at 07:08

Unfortunately I did not have enough time to create this function.

I'd like to note, that you (Hip Hyena) created this terrible function. no animation, range change. better not to do than do bad

Gentle Squid Apr 18, 2019 at 15:31

Wrong hour start values when zooming (1# chart). you display these hour points in absolute values, thats why the selected week is shown very small (when it starts to zoom)

Earnest Hedgehog Apr 22, 2019 at 05:56

Duplicate. I used local time zone methods. I think this isn't a fatal bug. easy fix: replace several methods, getMonth() -> getUTCMonth(), getDate() -> getUTCDay() ...

I spent less than 5 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Gentle Squid Apr 19, 2019 at 11:44

I'm talking about the zoom transition. few moment after it starts and still in progress

Giant Parrot Apr 19, 2019 at 13:39

The minimap corners are not round. Also the sizes of elements are absurdly large on desktop.

MacBook Pro 2015 / Safari 12.1

Earnest Hedgehog Apr 20, 2019 at 08:14

I think this is fake. Or show the video where you refresh the page and show the bug.
on your screenshot: CSS argument "border-radius" works well in other places

Kingly Chicken Apr 19, 2019 at 20:03

No retina support, blurry at high dens devices

Earnest Hedgehog Apr 22, 2019 at 06:05

Duplicate. I think this isn't a fatal bug. easy fix: add one line to the code

I spent less than 2 minutes to correct this bug. You can test: http://markhel.ru/t/index.html

Large Beaver Apr 23, 2019 at 20:50

Blurred picture (devicePixelRatio) and not good text change animation (subjective) but zoom animations is awesome and all bonus goals completed. Voted.

Windows 8/Chrome 73

Giant Parrot Apr 25, 2019 at 00:19

«I think this is fake. Or show the video where you refresh the page and show the bug.»

Safari handles border-radius not like it does Chrome. Try it on an iPhone and you’ll see.

0:17

iPhone SE / Safari 12.1

Info

Testing and Issues

Voting

Issues

Info

Testing and Issues

Voting

Issues

Log In