application / sources / gui / qml

pages

Clone Tools

Stats

Commits this week: 0
Total Committers: 11
Last Commit: 18 Jun
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 14 Mar 2023
2:15 am

Behrouz NematiPour

DEN-13152 : Flow Rate Settings Display "--" or "0" When Treatment is Paused

- Messages 0x0500 and 0x0800 has been updated and the Prescribed Flow Rate is being used.

- The diagnostics screen updated.

- The treatment main screen updated.

- The treatment flow adjustment screen updated.

- the Flows on the commit has 10f39b63ce8a992280ab123f05a5eb44a27c121b has been separated which has to be reverted back since these two messages almost all the time have the same payloads.

Options
    • -2
    • +2
    ./treatment/adjustments/TreatmentAdjustmentFlow.qml
    • -2
    • +2
    ./treatment/sections/TreatmentFlows.qml
  3. … 10 more files in changeset.
Monday 13 Mar 2023
11:49 pm

Behrouz NematiPour

DEN-14640 : Change Request: BP Set Flow Rate Shown during Blood Prime/Rinseback

- Added the message 0x0500#P1

|0x0500| 5 |0x040 |1 Hz |N |HD |All |Blood Flow Data Flow Set Pt (S32)|

regarding our conversation with @Sean.

Options
    • -1
    • +0
    ./endtreatment/rinseback/EndTreatmentRinseback.qml
    • -0
    • +10
    ./treatment/TreatmentBloodPrime.qml
2:08 pm

Behrouz NematiPour

RESOLVED. Thanks for the test, that is what I remembered. Fine then

RESOLVED.

Thanks for the test, that is what I remembered.
Fine then

2:02 pm

Behrouz NematiPour

1 - The component instance has been used in the same file, and that is the new feature, and previously we have to create a file to use the component instance, or directly return the created compone...

1 - The component instance has been used in the same file, and that is the new feature, and previously we have to create a file to use the component instance, or directly return the created component for one instance use.
2 - The Loader is used inside of the component, which doesn't seem right, the idea is to use inline components so, if possible, avoid loaders.
3 - Multiple components for the line tickmarks and round ones are created since it seems only a property change to make it rounded, since QML doesn't have a line or circle and all are Rectangle, it makes sense to update the underlying rectangle properties to get the desired shape.

1:43 pm

Behrouz NematiPour

Thanks, Sorry for the confusion. Yes, please, as we talked, the active progress should be orange, and the background I think can stay as it was. Then you can remove that extra Rectangle.

Thanks,
Sorry for the confusion.
Yes, please, as we talked, the active progress should be orange, and the background I think can stay as it was.
Then you can remove that extra Rectangle.

1:00 pm

vduong  (deleted user)

This may be a new concept for the project, but it's something that I've used and have seen used before in other QML projects. This also guarantees that you are only loading the object when you nee...

This may be a new concept for the project, but it's something that I've used and have seen used before in other QML projects.

This also guarantees that you are only loading the object when you need it when you use it with the Loader.

The Component itself, if you look at the Qt documentation, you will find "The component encapsulates the QML types within, as if they were defined in a separate QML file, and is not loaded until requested (in this case, by the two Loader objects)."

So, I am sure this is efficient.

12:56 pm

vduong  (deleted user)

We had this discussion this morning, I have already tested it also. It is working as we expected. It only prints out the round tick's Component.onCompleted log messages of the round tick marks and ...

We had this discussion this morning, I have already tested it also. It is working as we expected. It only prints out the round tick's Component.onCompleted log messages of the round tick marks and not both the round and lined tick marks. This is operating as we expected.

12:55 pm

vduong  (deleted user)

Ok, easy change over.

Ok, easy change over.

12:55 pm

vduong  (deleted user)

We had a discussion about what needed to be done and you had mentioned that the progressRect would be transparent. In order for that to be transparent, we needed a background rectangle color. BUT w...

We had a discussion about what needed to be done and you had mentioned that the progressRect would be transparent. In order for that to be transparent, we needed a background rectangle color. BUT we had talked this morning and I will remove this rectangle and just set the progress rect's color as w/e the active color is.

12:54 pm

vduong  (deleted user)

I believe this is leftover code that I missed from the slider bug fixes. I'll remove it for this branch.

I believe this is leftover code that I missed from the slider bug fixes. I'll remove it for this branch.

5:19 am

Behrouz NematiPour

Why was this Rectangle added? the ProgressRect is a Rectangle and can be used for the active color, and the _progressRectBackground seems redundant.

Why was this Rectangle added?
the ProgressRect is a Rectangle and can be used for the active color, and the _progressRectBackground seems redundant.

5:18 am

Behrouz NematiPour

Why was this property added?

Why was this property added?

4:45 am

Behrouz NematiPour

Regarding our new design change requests all tick marks shall be round by default.

Regarding our new design change requests all tick marks shall be round by default.

4:33 am

Behrouz NematiPour

Why In-Line Component is used. The idea behind the loader is to set them active if only it is set active. Does the in-line component have the same efficiency? Please test and provide proof of conce...

Why In-Line Component is used.
The idea behind the loader is to set them active if only it is set active.
Does the in-line component have the same efficiency?
Please test and provide proof of concept, since this feature is very new and I haven't tested it.
If can't be proven, use the loader only, please.

4:24 am

Behrouz NematiPour

In C++ the trinary "?" operator will call both true and false sections and then will decide which to return. If that is the case for the QML/JS I suggest using a function with an if block to return...

In C++ the trinary "?" operator will call both true and false sections and then will decide which to return.
If that is the case for the QML/JS I suggest using a function with an if block to return the decided object.
to ensure the trinary "?" operator behavior, you can use the "Component.onCompleted: {", if the new Qt 5.15 in-line Component declaration supports it.

Saturday 11 Mar 2023
5:24 am

Vy <vduong@sunriselabs.com>

DEN-14912 Quick Start : Sliders - Refinement of the rangeSlider's look for quick start

Options
    • -0
    • +10
    ./pretreatment/create/PreTreatmentCreate.qml
  2. … 1 more file in changeset.
Friday 10 Mar 2023
11:55 pm

Vy <vduong@sunriselabs.com>

DEN-14912: Quick Start Sliders -Improvement to the rangeSlider's look and feel for quick start

Options
    • -2
    • +2
    ./pretreatment/create/PreTreatmentCreate.qml
  2. … 4 more files in changeset.
6:01 pm

Dara Navaei

UI-DEN-14909_UI VD S92 QS Arrows
UI-DEN-14909_UI VD S92 QS Arrows
6:01 pm

Dara Navaei

UI-DEN-14909_UI VD S92 QS Sliders
UI-DEN-14909_UI VD S92 QS Sliders
11:14 am

Vy <vduong@sunriselabs.com>

DEN-14838 Revert PretreatmentCreate file again due to miscommit

Options
    • -57
    • +38
    ./pretreatment/create/PreTreatmentCreate.qml
10:56 am

Vy <vduong@sunriselabs.com>

DEN-14838 Revert changes from QuickStart WIP

Options
    • -38
    • +57
    ./pretreatment/create/PreTreatmentCreate.qml
Thursday 09 Mar 2023
11:33 am

vduong  (deleted user)

Can you check on the latest commit if this space was already removed? I can't tell which line this comment is in regards to. Thanks!

Can you check on the latest commit if this space was already removed? I can't tell which line this comment is in regards to. Thanks!

9:41 am

vduong  (deleted user)

I think we wanted the alias and not the bool. I vaguely remember suggesting the change to alias. This could've been a revert change. I am not sure. I will change it back to alias

I think we wanted the alias and not the bool. I vaguely remember suggesting the change to alias. This could've been a revert change. I am not sure. I will change it back to alias

9:39 am

vduong  (deleted user)

This is after the text property. I removed the newline between them as a change

This is after the text property. I removed the newline between them as a change

9:39 am

vduong  (deleted user)

Moved onTextChanged line and removed extra newline

Moved onTextChanged line and removed extra newline

9:37 am

vduong  (deleted user)

Will make note. Changed comment to reflect.

Will make note. Changed comment to reflect.

9:35 am

vduong  (deleted user)

Updated.

Updated.

9:35 am

vduong  (deleted user)

It will display two same values with one tick. I have removed the check.

It will display two same values with one tick.

I have removed the check.

9:34 am

vduong  (deleted user)

Updated, thank you

Updated, thank you

Tuesday 07 Mar 2023
6:47 pm

Behrouz NematiPour

please remove an extra empty line which is not necessary.

please remove an extra empty line which is not necessary.