New axis control
Should land D15224 before merging.
No Linters Available |
No Unit Test Coverage |
Buildable 2702 | |
Build 2720: arc lint + arc unit |
not possible since we will not know if the machine is using metric or imperial units. that is why i just used "units"
can this be configured? (the metrics or imperial?)
src/widgets/axiscontrol.h | ||
---|---|---|
52 | sorry, patrick's wrong here. |
src/widgets/axiscontrol.h | ||
---|---|---|
52 | The documentation says that is a class with functions. struct QLatin1Char { public: Q_DECL_CONSTEXPR inline explicit QLatin1Char(char c) Q_DECL_NOTHROW : ch(c) {} Q_DECL_CONSTEXPR inline char toLatin1() const Q_DECL_NOTHROW { return ch; } Q_DECL_CONSTEXPR inline ushort unicode() const Q_DECL_NOTHROW { return ushort(uchar(ch)); } private: char ch; }; |
src/widgets/axiscontrol.h | ||
---|---|---|
52 | so? |
src/widgets/axiscontrol.h | ||
---|---|---|
52 | Sith, tomaz is correct, I just talked with him. |
! In D15281#323497, @tcanabrava wrote:
can this be configured? (the metrics or imperial?)
Yes we can set it to " mm" by default, Then expose the QDoubleSpinBox::setSufix() method. Then Client will be able to change units if the user changes the mode..
src/widgets/axiscontrol.h | ||
---|---|---|
52 | Ok no const ref for QLatin1Char. I Will "fix" this for the rest of atcore tonight |
Where will the user be able to set the units? it should have a combobox on this widget to choose those.