Parachute, Introductu]ion bug fixed

Authored by rkaur on Apr 4 2016, 11:56 AM.

Description

Parachute, Introductu]ion bug fixed

Details

Committed
rkaurApr 4 2016, 11:56 AM
Parents
R2:f923db1e0295: Parachute, rotation from -6 to 6 added in parachute moment
Branches
Unknown
Tags
Unknown
Restricted Application added a subscriber: jjazeix. ยท View Herald TranscriptApr 4 2016, 11:57 AM
bcoudoin added inline comments.
/src/activities/parachute/Parachute.qml
81

Don't forget that we expect a space after a :

159

Need spaces around the ===. You can achieve this in declarative mode, see bellow my explanation.

284

Try to stick to 80 columns.

476

spaces

611

this is too much nested if. Maybe a better form is to make a linear equation close to what you want like:

11000 + 300 * bar.level

Or with an array like:

[0, 14000, 15000, 11000][bar.level]

(I put a 0 item in the list to accept uninitialized bar.

/src/activities/parachute/parachute.js
155

It is best to use the declarative form like adding this in InstuctionTwo item:
visible: items.bar.level === 1 && tuxImageStatus === 1