yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use RowLayout for the video, configure and about buttons

Browse source 
Simplifies the code as we don't have to anchor the buttons.

REVIEW: 114432
This commit is contained in:
Martin Gräßlin 2013-12-13 12:24:18 +01:00
parent 9e00284158
commit 5f1e8ec2bd
1 changed files with 25 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

50
kcmkwin/kwincompositing/qml/Effect.qml
View file

@ -182,32 +182,32 @@ Item {
} }
} }
Button { RowLayout {
id: videoButton
anchors.right: effectConfig.effectUiConfigExists(model.ServiceNameRole) ? configureButton.left : aboutButton.left
visible: model.VideoRole.toString() !== ""
iconName: "video"
onClicked: videoItem.showHide()
}
Button {
id: configureButton
anchors.right: aboutButton.left
visible: effectConfig.effectUiConfigExists(model.ServiceNameRole)
enabled: effectStatusCheckBox.checked
iconName: "configure"
onClicked: {
effectConfig.openConfig(model.NameRole);
}
}
Button {
id: aboutButton
anchors.right: parent.right anchors.right: parent.right
iconName: "dialog-information" Button {
onClicked: { id: videoButton
animationAbout.running = true; visible: model.VideoRole.toString() !== ""
animationAboutSpacing.running = true; iconName: "video"
desc.wrapDescription(); onClicked: videoItem.showHide()
}
Button {
id: configureButton
visible: effectConfig.effectUiConfigExists(model.ServiceNameRole)
enabled: effectStatusCheckBox.checked
iconName: "configure"
onClicked: {
effectConfig.openConfig(model.NameRole);
}
}
Button {
id: aboutButton
iconName: "dialog-information"
onClicked: {
animationAbout.running = true;
animationAboutSpacing.running = true;
desc.wrapDescription();
}
} }
} }