RESOLVED IN CODE WALKTHROUGH
Do not think this property is needed since the Label component already has this property set
RESOLVED in CODE WALKTHROUGH.
Space after if.
RESOLVED
please remove extra space before DRY_...
Make it like GridSelection.qml @ L54.
Added back
We can't jump to app if fpga ID is bad? Why not?
Removed
Done.
duty misspelled.
Removed.
Keep a blank line between banner and variables.
We do not always want to return to run state. If we were in ramp state prior to pause state, we would want to go back to ramp state here. So you need to remember your previous state in a new variable.
added
These are correct variable names based on their coefficient names in calculations of conductivity.
updated to remove background color
removed for now
RESOLVED.
updated
I should have said "before test messages" instead of "first". But where you put them is fine so I will resolve.
Will look into next branch for integration of components and backend
create a fontSize instead of alias
made changes as per comment
it looks like we already have cpp code to log it
done
use only isDefault
Replaced
its unsigned integer , no need to check -ve value
Where is len used ?
please leave a comment above describng why the visible property is being left in
Remove for now. Or any extra animation or UI cosmetics that are not a requirement.
To be consistent with the other qml files, always move this onComplete to the end of the file.
Add blank line after declarations.
Align comment.
Done
Added comment line
Removed multiple returns
Added blank lines as needed
Add @param for new param.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.