sources

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-6359: Addressed code review comments

This is my comment and it is specific to the exact line below the comment. Please revert it back to be able to identify which line in the code it refers to. "This code review includes so many unre...

This is my comment and it is specific to the exact line below the comment.
Please revert it back to be able to identify which line in the code it refers to.

"This code review includes so many unrelated branched to the code review subject"
look at the list of the commits and branches included in this page
http://dvm-linux02:8060/project/UI-DEN-6349?max=100&projectKey=UI-DEN-6349&view=fe

acceptableInput is a property of the TextInput component. The TextInput component in TextEntry is accessed via the textInput alias

acceptableInput is a property of the TextInput component.
The TextInput component in TextEntry is accessed via the textInput alias

This is your change actually. It looks like you added this in 45ce6e78 The alarm colors won't work without it

This is your change actually. It looks like you added this in 45ce6e78
The alarm colors won't work without it

where the acceptableInput comes from the TextEntry component doesn't have such a property?

where the acceptableInput comes from the TextEntry component doesn't have such a property?

RESOLVED

RESOLVED

Why this is imported?

Why this is imported?

RESOLVED

RESOLVED

When I removed the keyboard styling I removed this variable as well

When I removed the keyboard styling I removed this variable as well

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

It's in the UX design here: https://app.zeplin.io/project/5e160353a7c41a9404596a70/screen/5e5060d3f75589612aee8b68
See response above

See response above

See response above

See response above

See response above

See response above

See response above

See response above

This objectName is necessary for the added alarm tests to work, which also guarantees 100% code coverage on my branch It is very helpful for Systems / SW V&V to have as many object names defined so...

This objectName is necessary for the added alarm tests to work, which also guarantees 100% code coverage on my branch
It is very helpful for Systems / SW V&V to have as many object names defined so they can write their own squishqt tests
In order for Systems / SW V&V to interact with the alarms and so the alarm tests can function, I think it should be kept

Having a separate component prevents alerts from covering up alarms and ensures the alarm takes precedence

Having a separate component prevents alerts from covering up alarms and ensures the alarm takes precedence

This isn't a change I had made. You added this in cd769413344091cea88a30861b49188c8c147cba I don't know why it's showing up here. Seems like crucible messed up Do you want me to remove it anyway?

This isn't a change I had made.
You added this in cd769413344091cea88a30861b49188c8c147cba
I don't know why it's showing up here. Seems like crucible messed up
Do you want me to remove it anyway?

Done

Done

Done

Done

This folder contains treatment related information such as parameter ranges as well as paired BLE devices. Would you prefer we store the bluetooth devices in a separate folder called bluetooth inst...

This folder contains treatment related information such as parameter ranges as well as paired BLE devices.
Would you prefer we store the bluetooth devices in a separate folder called bluetooth instead?

"connmand" is correct It's used to manage internet connections on embedded devices and causes problems if it is already running while bluetooth is starting up

"connmand" is correct
It's used to manage internet connections on embedded devices and causes problems if it is already running while bluetooth is starting up

Done

Done

Done - Updated to use iArrow now

Done - Updated to use iArrow now

Done

Done

DEN-4867: Addressed code review comments

    • -21
    • +8
    ./gui/qml/components/SettingsItem.qml
  1. … 3 more files in changeset.
same

same

Please move it after # CANBus section.

Please move it after # CANBus section.