[qml] Clean-up: Harmonize syntax across all files

This commit is contained in:
Candice Bentéjac 2023-10-17 15:42:51 +02:00
parent e9d80611c7
commit e463f0dce2
6 changed files with 99 additions and 94 deletions

View file

@ -14,7 +14,11 @@ Dialog {
// Fade in transition
enter: Transition {
NumberAnimation { property: "opacity"; from: 0.0; to: 1.0 }
NumberAnimation {
property: "opacity"
from: 0.0
to: 1.0
}
}
modal: true
@ -173,14 +177,14 @@ Dialog {
// try to load the local file
var url = Filepath.stringToUrl(modelData.localUrl)
// fallback to the online url if file is not found
if(!Filepath.exists(url))
if (!Filepath.exists(url))
url = modelData.onlineUrl
Request.get(url,
function(xhr){
if(xhr.readyState === XMLHttpRequest.DONE)
function(xhr) {
if (xhr.readyState === XMLHttpRequest.DONE)
{
// status is OK
if(xhr.status === 200)
if (xhr.status === 200)
textArea.text = MeshroomApp.markdownToHtml(xhr.responseText)
else
textArea.text = "Could not load license file. Available online at <a href='" + url + "'>"+ url + "</a>."

View file

@ -93,7 +93,7 @@ Panel {
enabled: liveSfmManager ? liveSfmManager.running || folderPath.text.trim() != '' : false
checked: liveSfmManager ? liveSfmManager.running : false
onClicked: {
if(!liveSfmManager.running)
if (!liveSfmManager.running)
liveSfmManager.start(folderPath.text, minImg_SB.value)
else
liveSfmManager.stop()

View file

@ -19,14 +19,13 @@ ToolButton {
}
background: Rectangle {
color: {
if(pressed || checked || hovered)
{
if(pressed || checked)
if (pressed || checked || hovered) {
if (pressed || checked)
return Qt.darker(parent.palette.base, 1.3)
if(hovered)
if (hovered)
return Qt.darker(parent.palette.base, 0.6)
}
return "transparent";
return "transparent"
}
border.color: checked ? Qt.darker(parent.palette.base, 1.4) : "transparent"

View file

@ -36,14 +36,13 @@ ToolButton {
}
background: Rectangle {
color: {
if(pressed || checked || hovered)
{
if(pressed || checked)
if (pressed || checked || hovered) {
if (pressed || checked)
return Qt.darker(parent.palette.base, 1.3)
if(hovered)
if (hovered)
return Qt.darker(parent.palette.base, 0.6)
}
return "transparent";
return "transparent"
}
border.color: checked ? Qt.darker(parent.palette.base, 1.4) : "transparent"

View file

@ -32,15 +32,15 @@ Item {
// Load a 3D media file in the 3D viewer
function load3DMedia(filepath, label = undefined) {
if(panel3dViewerLoader.active) {
panel3dViewerLoader.item.viewer3D.load(filepath, label);
if (panel3dViewerLoader.active) {
panel3dViewerLoader.item.viewer3D.load(filepath, label)
}
}
Connections {
target: reconstruction
function onGraphChanged() {
if(panel3dViewerLoader.active) {
if (panel3dViewerLoader.active) {
panel3dViewerLoader.item.viewer3D.clear()
}
}
@ -51,11 +51,11 @@ Item {
// Load reconstruction's current SfM file
function viewSfM() {
var activeNode = _reconstruction.activeNodes ? _reconstruction.activeNodes.get('sfm').node : null;
if(!activeNode)
return;
if(panel3dViewerLoader.active) {
panel3dViewerLoader.item.viewer3D.view(activeNode.attribute('output'));
var activeNode = _reconstruction.activeNodes ? _reconstruction.activeNodes.get('sfm').node : null
if (!activeNode)
return
if (panel3dViewerLoader.active) {
panel3dViewerLoader.item.viewer3D.view(activeNode.attribute('output'))
}
}

View file

@ -34,9 +34,9 @@ ApplicationWindow {
onClosing: {
// make sure document is saved before exiting application
close.accepted = false
if(!ensureNotComputing())
if (!ensureNotComputing())
return
ensureSaved(function(){ Qt.quit() })
ensureSaved(function() { Qt.quit() })
}
palette: _PaletteManager.palette
@ -134,7 +134,7 @@ ApplicationWindow {
onAccepted: {
// save current file
if(saveAction.enabled)
if (saveAction.enabled)
{
saveAction.trigger()
fireCallback()
@ -144,7 +144,7 @@ ApplicationWindow {
{
saveFileDialog.open()
function _callbackWrapper(rc) {
if(rc === Platform.Dialog.Accepted)
if (rc === Platform.Dialog.Accepted)
fireCallback()
saveFileDialog.closed.disconnect(_callbackWrapper)
}
@ -155,7 +155,7 @@ ApplicationWindow {
function fireCallback()
{
// call the callback and reset it
if(_callback)
if (_callback)
_callback()
_callback = undefined
}
@ -215,7 +215,7 @@ ApplicationWindow {
false
function compute(node, force) {
if(!force && warnIfUnsaved && !_reconstruction.graph.filepath)
if (!force && warnIfUnsaved && !_reconstruction.graph.filepath)
{
unsavedComputeDialog.currentNode = node;
unsavedComputeDialog.open();
@ -226,7 +226,7 @@ ApplicationWindow {
}
catch (error) {
const data = ErrorHandler.analyseError(error)
if(data.context === "COMPUTATION")
if (data.context === "COMPUTATION")
computeSubmitErrorDialog.openError(data.type, data.msg, node)
}
}
@ -241,7 +241,7 @@ ApplicationWindow {
}
catch (error) {
const data = ErrorHandler.analyseError(error)
if(data.context === "SUBMITTING")
if (data.context === "SUBMITTING")
computeSubmitErrorDialog.openError(data.type, data.msg, node)
}
}
@ -255,10 +255,18 @@ ApplicationWindow {
function openError(type, msg, node) {
errorType = type
switch(type) {
case "Already Submitted": this.setupPendingStatusError(msg, node); break
case "Compatibility Issue": this.setupCompatibilityIssue(msg); break
default: this.onlyDisplayError(msg)
switch (type) {
case "Already Submitted": {
this.setupPendingStatusError(msg, node)
break
}
case "Compatibility Issue": {
this.setupCompatibilityIssue(msg)
break
}
default: {
this.onlyDisplayError(msg)
}
}
this.open()
@ -291,7 +299,7 @@ ApplicationWindow {
text: ""
onAccepted: {
switch(errorType) {
switch (errorType) {
case "Already Submitted": {
close()
_reconstruction.graph.clearSubmittedNodes()
@ -337,7 +345,11 @@ ApplicationWindow {
standardButton(Dialog.Discard).text = "Continue without Saving"
}
onDiscarded: { close(); computeManager.compute(currentNode, true) }
onDiscarded: {
close()
computeManager.compute(currentNode, true)
}
onAccepted: saveAsAction.trigger()
}
@ -363,8 +375,7 @@ ApplicationWindow {
title: "Open File"
nameFilters: ["Meshroom Graphs (*.mg)"]
onAccepted: {
if(_reconstruction.loadUrl(fileUrl))
{
if (_reconstruction.loadUrl(fileUrl)) {
MeshroomApp.addRecentProjectFile(fileUrl.toString())
}
}
@ -413,13 +424,9 @@ ApplicationWindow {
function ensureSaved(callback)
{
var saved = _reconstruction.undoStack.clean
// If current document is modified, open "unsaved dialog"
if(!saved)
{
if (!saved) { // If current document is modified, open "unsaved dialog"
unsavedDialog.prompt(callback)
}
else // otherwise, directly call the callback
{
} else { // Otherwise, directly call the callback
callback()
}
return saved
@ -438,7 +445,7 @@ ApplicationWindow {
// Check and return whether no local computation is in progress
function ensureNotComputing()
{
if(_reconstruction.computingLocally)
if (_reconstruction.computingLocally)
{
// Open a warning dialog to ask for computation to be stopped
computingAtExitDialog.open()
@ -578,23 +585,23 @@ ApplicationWindow {
let folder = "";
if (imagesFolder.toString() === "" && workspaceView.imageGallery.galleryGrid.itemAtIndex(0) !== null) {
imagesFolder = Filepath.stringToUrl(Filepath.dirname(workspaceView.imageGallery.galleryGrid.itemAtIndex(0).source));
imagesFolder = Filepath.stringToUrl(Filepath.dirname(workspaceView.imageGallery.galleryGrid.itemAtIndex(0).source))
}
if (_reconstruction.graph && _reconstruction.graph.filepath) {
folder = Filepath.stringToUrl(Filepath.dirname(_reconstruction.graph.filepath));
folder = Filepath.stringToUrl(Filepath.dirname(_reconstruction.graph.filepath))
} else {
var projects = MeshroomApp.recentProjectFiles;
if (projects.length > 0 && Filepath.exists(projects[0])) {
folder = Filepath.stringToUrl(Filepath.dirname(projects[0]));
folder = Filepath.stringToUrl(Filepath.dirname(projects[0]))
}
}
if (importImages && imagesFolder.toString() !== "" && Filepath.exists(imagesFolder)) {
folder = imagesFolder;
folder = imagesFolder
}
dialog.folder = folder;
dialog.folder = folder
}
header: MenuBar {
@ -609,8 +616,8 @@ ApplicationWindow {
property int fullWidth: {
var result = 0;
for (var i = 0; i < count; ++i) {
var item = itemAt(i);
result = Math.max(item.implicitWidth + item.padding * 2, result);
var item = itemAt(i)
result = Math.max(item.implicitWidth + item.padding * 2, result)
}
return result;
}
@ -641,8 +648,8 @@ ApplicationWindow {
text: "Open"
shortcut: "Ctrl+O"
onTriggered: ensureSaved(function() {
initFileDialogFolder(openFileDialog);
openFileDialog.open();
initFileDialogFolder(openFileDialog)
openFileDialog.open()
})
}
Menu {
@ -653,10 +660,10 @@ ApplicationWindow {
property int fullWidth: {
var result = 0;
for (var i = 0; i < count; ++i) {
var item = itemAt(i);
result = Math.max(item.implicitWidth + item.padding * 2, result);
var item = itemAt(i)
result = Math.max(item.implicitWidth + item.padding * 2, result)
}
return result;
return result
}
implicitWidth: fullWidth
Repeater {
@ -664,14 +671,11 @@ ApplicationWindow {
model: MeshroomApp.recentProjectFiles
MenuItem {
onTriggered: ensureSaved(function() {
openRecentMenu.dismiss();
if(_reconstruction.loadUrl(modelData))
{
MeshroomApp.addRecentProjectFile(modelData);
}
else
{
MeshroomApp.removeRecentProjectFile(modelData);
openRecentMenu.dismiss()
if (_reconstruction.loadUrl(modelData)) {
MeshroomApp.addRecentProjectFile(modelData)
} else {
MeshroomApp.removeRecentProjectFile(modelData)
}
})
@ -692,13 +696,11 @@ ApplicationWindow {
shortcut: "Ctrl+S"
enabled: _reconstruction ? (_reconstruction.graph && !_reconstruction.graph.filepath) || !_reconstruction.undoStack.clean : false
onTriggered: {
if(_reconstruction.graph.filepath) {
_reconstruction.save();
}
else
{
initFileDialogFolder(saveFileDialog);
saveFileDialog.open();
if (_reconstruction.graph.filepath) {
_reconstruction.save()
} else {
initFileDialogFolder(saveFileDialog)
saveFileDialog.open()
}
}
}
@ -707,8 +709,8 @@ ApplicationWindow {
text: "Save As..."
shortcut: "Ctrl+Shift+S"
onTriggered: {
initFileDialogFolder(saveFileDialog);
saveFileDialog.open();
initFileDialogFolder(saveFileDialog)
saveFileDialog.open()
}
}
MenuSeparator { }
@ -717,8 +719,8 @@ ApplicationWindow {
text: "Import Images"
shortcut: "Ctrl+I"
onTriggered: {
initFileDialogFolder(importImagesDialog, true);
importImagesDialog.open();
initFileDialogFolder(importImagesDialog, true)
importImagesDialog.open()
}
}
@ -743,8 +745,8 @@ ApplicationWindow {
onActivated: saveAsTemplateAction.triggered()
}
onTriggered: {
initFileDialogFolder(saveTemplateDialog);
saveTemplateDialog.open();
initFileDialogFolder(saveTemplateDialog)
saveTemplateDialog.open()
}
}
@ -763,8 +765,8 @@ ApplicationWindow {
onActivated: importProjectAction.triggered()
}
onTriggered: {
initFileDialogFolder(importProjectDialog);
importProjectDialog.open();
initFileDialogFolder(importProjectDialog)
importProjectDialog.open()
}
}
@ -896,7 +898,7 @@ ApplicationWindow {
function onGraphChanged() {
// open CompatibilityManager after file loading if any issue is detected
if(compatibilityManager.issueCount)
if (compatibilityManager.issueCount)
compatibilityManager.open()
// trigger fit to visualize all nodes
graphEditor.fit()
@ -984,24 +986,25 @@ ApplicationWindow {
function viewNode(node, mouse) {
// 2D viewer
viewer2D.tryLoadNode(node);
viewer2D.tryLoadNode(node)
// 3D viewer
for (var i = 0; i < node.attributes.count; i++) {
var attr = node.attributes.at(i)
if(attr.isOutput && attr.desc.semantic !== "image" && workspaceView.viewIn3D(attr, mouse))
break;
if (attr.isOutput && attr.desc.semantic !== "image" && workspaceView.viewIn3D(attr, mouse))
break
}
}
function viewIn3D(attribute, mouse) {
if(!panel3dViewer || !attribute.node.has3DOutput)
return false;
var loaded = panel3dViewer.viewer3D.view(attribute);
if (!panel3dViewer || !attribute.node.has3DOutput)
return false
var loaded = panel3dViewer.viewer3D.view(attribute)
// solo media if Control modifier was held
if(loaded && mouse && mouse.modifiers & Qt.ControlModifier)
panel3dViewer.viewer3D.solo(attribute);
return loaded;
if (loaded && mouse && mouse.modifiers & Qt.ControlModifier)
panel3dViewer.viewer3D.solo(attribute)
return loaded
}
}
}
@ -1203,8 +1206,8 @@ ApplicationWindow {
readOnly: node ? node.locked : false
onUpgradeRequest: {
var n = _reconstruction.upgradeNode(node);
_reconstruction.selectedNode = n;
var n = _reconstruction.upgradeNode(node)
_reconstruction.selectedNode = n
}
}
}