application

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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.

Why was this property added?

Why was this property added?

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.

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.

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.

UI-DEN-14909_UI VD S92 QS Arrows
UI-DEN-14909_UI VD S92 QS Arrows
UI-DEN-14909_UI VD S92 QS Sliders
UI-DEN-14909_UI VD S92 QS Sliders
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!

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

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

Moved onTextChanged line and removed extra newline

Moved onTextChanged line and removed extra newline

Will make note. Changed comment to reflect.

Will make note. Changed comment to reflect.

Updated.

Updated.

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.

Updated, thank you

Updated, thank you

please remove an extra empty line which is not necessary.

please remove an extra empty line which is not necessary.

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?

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.

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

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

please move after the text property.

please move after the text property.

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.

typo: ajust => adjust

typo: ajust => adjust

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.

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

RESOLVED

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.

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.