iOS Chart – Developer Challenges

Eager Wolf Apr 27, 2019 at 14:39

Minor design issues, but very good speed. Unfortunately, no bonus goal implemented.

iPhone 6 Plus, iOS 12.1

50

Mad Leopard Apr 28, 2019 at 21:08

Thanks for the feedback. I was focused on the performance and didn't have time to implement bonus goals or polish UI.
I'd add a link to my telegram profile here https://t.me/beloal

Perfect Sheep Apr 19, 2019 at 00:58

1. Percentage chart doesnt show percentage
2. Stacked bar doesnt show total.
3. Wrong design of navigation bar in nightmode
4. on stacked bar charts the first and last bars have half of width

5. The tooltip starts following the scroll when placed either on the first point or the last, which leads to small ui bug. Otherwise it usually disappears on touch up.
For example while its left on the screen u can switch the theme and the circle points will not change background color

0:28

iphone 6 12.1.4

32

Tidy Lynx Apr 23, 2019 at 06:47

Line joints change during chart scaling

0:14

22

Mad Leopard Apr 23, 2019 at 13:16

Can be fixed in one line of code

Tidy Lynx Apr 23, 2019 at 06:47

Critical: During the scrolling charts are rasterized in very low quality

0:10

22

Mad Leopard Apr 23, 2019 at 13:05

Can be fixed in one line of code

Sergey Dikovitsky Apr 17, 2019 at 20:41

X-axis labels, y-axis labels and lines, chart lines are not animated.

0:24

iPhone 5s / iOS 12.2

13

Mad Leopard Apr 17, 2019 at 22:30

It's not clear for me how to animate Y-axis on chart #2 when blue values change and yellow don't. So, to be consistent, I've decided to disable it on all charts.
X-axis animation isn't implemented due to lack of time.

D

Deleted Account Apr 24, 2019 at 21:17

No "all" label

iPhone Xs, iOS 12.2

10

D

Deleted Account Apr 24, 2019 at 21:21

Dots have wrong design

iPhone Xs, iOS 12.2

10

D

Deleted Account Apr 24, 2019 at 21:23

Day/night switching has no animation

0:04

iPhone Xs, iOS 12.2

10

Fair Quokka Apr 27, 2019 at 20:41

— Upper bound values are cut when scaling (the attached video #1).
— Switching themes causes the screen to blink.
— Graphs blink after scrolling.
— It's possible to select dates that are not visible on the graphs (the attached screenshots #2 and #3).
— The app supports iOS 12+ only.

0:05

iPhone 6s (iOS 12.0.2); iPhone 5s (iOS 12.1)

10

Mad Leopard Apr 27, 2019 at 21:11

- this is because of layer's line join mode. Can be fixed in one line of code
- Theme switch isn't animated.
- Can't reproduce on my devices
- true
- There was no requirements to support specific iOS version. However, iOS 10 can be supported by just changing build setting (deployment target), and iOS 9 can be supported with minor code change

Tidy Lynx Apr 23, 2019 at 06:46

No table separators in cells

2

Tidy Lynx Apr 23, 2019 at 06:48

Critical: no dates and y-values animations. Not only the UX is suffering, but also it gives this contestant a major advantage in performance over other participants.

1

Mad Leopard Apr 23, 2019 at 13:10

Y values animate on filters. X-axis animation isn't implemented due to the lack of time.

Tidy Lynx Apr 23, 2019 at 06:46

Does not support iOS 8. Telegram does.

Tidy Lynx Apr 23, 2019 at 06:46

Major issue: does not support iOS 10.

Tidy Lynx Apr 23, 2019 at 06:46

Critical issue: does not support iOS 11.

Mad Leopard Apr 23, 2019 at 12:58

There was no requirements to support specific iOS version. However, iOS 10 can be supported by just changing build setting (deployment target), and iOS 9 can be supported with minor code change

Tidy Lynx Apr 23, 2019 at 06:46

maxY on chart has to be on the same position as y-coordinate of the highest y-value by design

Mad Leopard Apr 23, 2019 at 13:20

Are you sure?

Tidy Lynx Apr 23, 2019 at 06:47

Charts are clipped to the left and to the right.

Tidy Lynx Apr 23, 2019 at 06:48

Critical issue: Line joints are not rounded, even on iPhone X. So, it's not some minor optimization for old devices, rounded joints implementation on Metal require a lot more math to implement, a lot more vertices to calculate for each frame and 2-3 times more polygons to draw.

Mad Leopard Apr 23, 2019 at 13:10

I don't use metal. Can be fixed in one line

Tidy Lynx Apr 23, 2019 at 06:48

Critical issue: This code uses Metal and it will never work on 32-bit devices (including iPhone 5 on iOS 10.3).

Mad Leopard Apr 23, 2019 at 13:10

I don't use metal

Tidy Lynx Apr 23, 2019 at 06:48

Major issue: It's not so easy to port this code to iOS 8 on iPhone 5S either, because he uses help classes from MetalKit (otherwise he would support iOS 8). And you have to do a lot more work without it.

Mad Leopard Apr 23, 2019 at 13:13

I don't use metal. iOS 8 can't be supported due to use of UIStackView, NSLayoutAnchor and other api from iOS 9

Tidy Lynx Apr 23, 2019 at 06:48

Critical issue: line drawing quality is poor. It is not easy to implement good line quality on bare Metal, MSAA is not enough. Qualitative line drawing requires a lot more calculations and tricks. And such implementations may still be worse than Core Animation / Core Graphics quality and would definitely affect the performance. You can read more about line quality in Metal projects in my comments on entry228.

Mad Leopard Apr 23, 2019 at 13:15

Again, I don't use metal.

D

Deleted Account Apr 24, 2019 at 21:14

Chart max line is to higher above the last y-axis line

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:14

Wrong design

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:14

Wrong design

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:15

Chart must be beyond the borders

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:16

Wrong x-axis

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:16

Wrong end date

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:17

Charts loses quality while scrolling

0:10

0:08

0:05

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:18

Wrong mini-map

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:18

Numbers must be shorten

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:20

All charts, except percentage chart, must have exactly 6 lines on y-axis

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:21

Charts cutted off

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:22

Wrong space

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:22

Percentage chart must have 25% step

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:22

Dancing peaks. See right side

0:07

iPhone Xs, iOS 12.2

D

Deleted Account Apr 24, 2019 at 21:23

Completely no animation on x-axis and y-axis

0:08

iPhone Xs, iOS 12.2

Info

Testing and Issues

Voting

Issues

Info

Testing and Issues

Voting

Issues

Log In