application / sources / gui / qml

pages

Clone Tools

Stats

Commits this week: 0
Total Committers: 11
Last Commit: 03 Apr
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
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.

6:45 pm

Behrouz NematiPour

Didn't we find an issue with changing this to bool instead of alias while were testing?

Didn't we find an issue with changing this to bool instead of alias while were testing?

6:33 pm

Behrouz NematiPour

in general, if the comment is for the entire if block put it in front of the if block. also for the else at line 66.

in general, if the comment is for the entire if block put it in front of the if block.
also for the else at line 66.

6:30 pm

Behrouz NematiPour

please move after the text property and remove the extra empty line.

please move after the text property and remove the extra empty line.

6:30 pm

Behrouz NematiPour

please move after the text property.

please move after the text property.

6:29 pm

Behrouz NematiPour

Please let the following calculation of the overlap do the separation. Otherwise will completely overlap and eventually displays one value which would be confusing. And also creates a position jump.

Please let the following calculation of the overlap do the separation.
Otherwise will completely overlap and eventually displays one value which would be confusing.
And also creates a position jump.

6:28 pm

Behrouz NematiPour

typo: ajust => adjust

typo: ajust => adjust

6:25 pm

Behrouz NematiPour

Please change the minimum gap to 4. I think when tested, 4 was sufficient.

Please change the minimum gap to 4.
I think when tested, 4 was sufficient.

Monday 06 Mar 2023
2:52 pm

Vy <vduong@sunriselabs.com>

DEN-14838 Revert PreTreatmentCreate.qml file changes

Options
    • -57
    • +38
    ./pretreatment/create/PreTreatmentCreate.qml
Sunday 05 Mar 2023
7:46 pm

Behrouz NematiPour

7:45 pm

vduong  (deleted user)

RESOLVED

RESOLVED

7:39 pm

vduong  (deleted user)

RESOLVED

RESOLVED

7:39 pm

vduong  (deleted user)

RESOLVED

RESOLVED

7:38 pm

Behrouz NematiPour

The reason is separation and keeping the logic of the Slider and its ON/OFF switch clear and non-confusing, and independent. Like right now that I had a wrong fix and I easily just changed the conn...

The reason is separation and keeping the logic of the Slider and its ON/OFF switch clear and non-confusing, and independent.
Like right now that I had a wrong fix and I easily just changed the connection related to the fix I had not changed the Slider implementation itself.

Saturday 04 Mar 2023
9:09 am

vduong  (deleted user)

Friday 03 Mar 2023
5:57 pm

vduong  (deleted user)

I disagree. If we bind the value here to the "active" property, the value it takes is whatever "active" has when the component gets created.

I disagree. If we bind the value here to the "active" property, the value it takes is whatever "active" has when the component gets created.