application

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-17157: Update_Denali_Application_for_Dry_Demo

- Addressed the code review comments.

Done

Done

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

DEN-17157: Update_Denali_Application_for_Dry_Demo

- Addressed the code review comments.

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

please add //DEBUG: to comment this out.

please add //DEBUG: to comment this out.

please align arguments.

please align arguments.

please remove extra space before DRY_...

please remove extra space before DRY_...

I believe you should correct the other z-orders.

I believe you should correct the other z-orders.

same here as above. *same here as above.
  • same here as above.
  • same here as above.
is there a specific reason the notSetVariable function is not used and replaced with "__"?

is there a specific reason the notSetVariable function is not used and replaced with "__"?

please do the same as the comment in the following link: http://devapps.diality.us:8060/cru/UI-DEN-17157-1#c21568

please do the same as the comment in the following link:
http://devapps.diality.us:8060/cru/UI-DEN-17157-1#c21568

please align the " ,2"

please align the " ,2"

please define these numbers (on other lines as well) as a: readonly property <type of the value> <name of the propery> example: on top: readonly property int bloodFlowRateDef : 400 replace with: d...

please define these numbers (on other lines as well) as a:
readonly property <type of the value> <name of the propery>
example:
on top:

readonly property int bloodFlowRateDef : 400

replace with:

defaultValue: _GuiView.dryDemoMode ? _root.bloodFlowRateDef : vTreatmentRanges.bloodFlowRateDef