mirror of
https://git.mirrors.martin98.com/https://github.com/Ultimaker/Cura
synced 2025-07-31 00:21:59 +08:00
Apply suggestions from code review
Co-authored-by: Casper Lamboo <c.lamboo@ultimaker.com>
This commit is contained in:
parent
90ce031b27
commit
cac9d67352
@ -21,14 +21,14 @@ Row
|
|||||||
{
|
{
|
||||||
id: leftLabel
|
id: leftLabel
|
||||||
text: catalog.i18nc("@action:label", "Type")
|
text: catalog.i18nc("@action:label", "Type")
|
||||||
width: (parent.width / 4) | 0
|
width: Math.round(parent.width / 4)
|
||||||
wrapMode: Text.WordWrap
|
wrapMode: Text.WordWrap
|
||||||
}
|
}
|
||||||
UM.Label
|
UM.Label
|
||||||
{
|
{
|
||||||
id: rightLabel
|
id: rightLabel
|
||||||
text: manager.machineType
|
text: manager.machineType
|
||||||
width: (parent.width / 3) | 0
|
width: Math.round(parent.width / 3)
|
||||||
wrapMode: Text.WordWrap
|
wrapMode: Text.WordWrap
|
||||||
}
|
}
|
||||||
}
|
}
|
@ -50,7 +50,7 @@ Item
|
|||||||
Item
|
Item
|
||||||
{
|
{
|
||||||
id: comboboxTooltip
|
id: comboboxTooltip
|
||||||
width: (parent.width / 2.5) | 0
|
width: Math.round(parent.width / 2.5)
|
||||||
height: visible ? UM.Theme.getSize("default_margin").height : 0
|
height: visible ? UM.Theme.getSize("default_margin").height : 0
|
||||||
anchors.top: parent.top
|
anchors.top: parent.top
|
||||||
anchors.right: parent.right
|
anchors.right: parent.right
|
||||||
|
@ -91,7 +91,7 @@ Item
|
|||||||
fixedWidthMode: true
|
fixedWidthMode: true
|
||||||
// The maximum width of the button is half of the total space, minus the padding of the parent, the right
|
// The maximum width of the button is half of the total space, minus the padding of the parent, the right
|
||||||
// padding of the component and half the spacing because of the space between buttons.
|
// padding of the component and half the spacing because of the space between buttons.
|
||||||
width: parent.width / 2 - rightPadding * 1.5
|
width: Math.round(parent.width / 2 - rightPadding * 1.5)
|
||||||
onClicked:
|
onClicked:
|
||||||
{
|
{
|
||||||
machineSelection.toggleContent()
|
machineSelection.toggleContent()
|
||||||
|
Loading…
x
Reference in New Issue
Block a user